auth/authcon.c: fix likely mistake
authorNickolai Zeldovich <nickolai@csail.mit.edu>
Sun, 6 Jan 2013 04:45:21 +0000 (23:45 -0500)
committerJeffrey Altman <jaltman@your-file-system.com>
Sun, 6 Jan 2013 16:34:49 +0000 (08:34 -0800)
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>

src/auth/authcon.c

index 8ede547..9148624 100644 (file)
@@ -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;