From: Nickolai Zeldovich <nickolai@csail.mit.edu>
Date: Sun, 6 Jan 2013 04:45:21 +0000 (-0500)
Subject: auth/authcon.c: fix likely mistake
X-Git-Tag: upstream/1.8.0_pre1^2~1675
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=1b3749335165a0d6ff7882cea4ee3e16f2055b4e;p=packages%2Fo%2Fopenafs.git

auth/authcon.c: fix likely mistake

It seems likely the original developer intended to check
*scIndex==RX_SECIDX_NULL rather than scIndex==RX_SECIDX_NULL.

Change-Id: I8a33d3cb177a9212917613581957073392b08314
Reviewed-on: http://gerrit.openafs.org/8881
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
---

diff --git a/src/auth/authcon.c b/src/auth/authcon.c
index 8ede547a5..914862432 100644
--- a/src/auth/authcon.c
+++ b/src/auth/authcon.c
@@ -331,7 +331,7 @@ afsconf_PickClientSecObj(struct afsconf_dir *dir, afsconf_secflags flags,
 	     * explicitly requested. Check for this, and bail out if we
 	     * get one. Note that this leaks a security object at present
 	     */
-	    if (scIndex == RX_SECIDX_NULL) {
+	    if (*scIndex == RX_SECIDX_NULL) {
 		sc = NULL;
 		code = AFSCONF_NOTFOUND;
 		goto out;