From: Michael Meffie <mmeffie@sinenomine.net>
Date: Fri, 8 Feb 2013 22:08:28 +0000 (-0500)
Subject: libafs: cleanup redundant assignment
X-Git-Tag: upstream/1.8.0_pre1^2~1548
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=85bac1ae40123810fd6bd30625999e276af5ab75;p=packages%2Fo%2Fopenafs.git

libafs: cleanup redundant assignment

Cleanup redundant clearing of the avcp pointer in lookup, which
looks to be a cut and paste error from the ibm afs days.

Change-Id: Ib554d862bd8485be2a4dbeb38fcaacac57c1e818
Reviewed-on: http://gerrit.openafs.org/9085
Reviewed-by: Chas Williams - CONTRACTOR <chas@cmf.nrl.navy.mil>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

diff --git a/src/afs/VNOPS/afs_vnop_lookup.c b/src/afs/VNOPS/afs_vnop_lookup.c
index 7e16e3a22..0faae09de 100644
--- a/src/afs/VNOPS/afs_vnop_lookup.c
+++ b/src/afs/VNOPS/afs_vnop_lookup.c
@@ -1430,11 +1430,8 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
     if (code)
 	goto done;
 
-    *avcp = NULL;		/* Since some callers don't initialize it */
-
     /* come back to here if we encounter a non-existent object in a read-only
      * volume's directory */
-
   redo:
     *avcp = NULL;		/* Since some callers don't initialize it */
     bulkcode = 0;