From 1b3749335165a0d6ff7882cea4ee3e16f2055b4e Mon Sep 17 00:00:00 2001 From: Nickolai Zeldovich Date: Sat, 5 Jan 2013 23:45:21 -0500 Subject: [PATCH] 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 Reviewed-by: Jeffrey Altman Tested-by: BuildBot Reviewed-by: Simon Wilkinson --- src/auth/authcon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; -- 2.39.5