From: Simon Wilkinson <sxw@your-file-system.com>
Date: Tue, 19 Feb 2013 15:11:33 +0000 (+0000)
Subject: kauth: Remove unused assignments to bp
X-Git-Tag: upstream/1.8.0_pre1^2~1504
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=95a05f49ea303b2a25c4465bf1bbe333fb869911;p=packages%2Fo%2Fopenafs.git

kauth: Remove unused assignments to bp

We never use the character pointer once we've advanced it past the
slash, so don't bother doing the work of advancing it.

Caught by clang-analyzer

Change-Id: I8ea561d6756685a13892f0635fa2e19b3f1233ab
Reviewed-on: http://gerrit.openafs.org/9171
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/kauth/kkids.c b/src/kauth/kkids.c
index 3439f633e..90d9b1e5e 100644
--- a/src/kauth/kkids.c
+++ b/src/kauth/kkids.c
@@ -178,14 +178,12 @@ find_me(char *arg, char *parent_dir)
 	 */
 	strncpy(parent_dir, truename, bp - truename);
 	parent_dir[bp - truename] = 0;
-	bp++;			/*Skip the slash */
     } else {
 	/*
 	 * No slash appears in the given file name.  Set parent_dir to the current
 	 * directory, and the last component as the given name.
 	 */
 	strcpy(parent_dir, ".");
-	bp = truename;
     }
 
     return 1;			/* found it */