From: Simon Wilkinson <sxw@your-file-system.com>
Date: Tue, 19 Feb 2013 17:18:13 +0000 (+0000)
Subject: fstrace: Remove redundant variable assignment
X-Git-Tag: upstream/1.8.0_pre1^2~1460
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=b7baf62d56cb8c1f207b885fb7c7a7b7105edef5;p=packages%2Fo%2Fopenafs.git

fstrace: Remove redundant variable assignment

Don't reset the value of 'code' when nothing else references it.

Caught by clang-analyzer

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

diff --git a/src/venus/fstrace.c b/src/venus/fstrace.c
index 56daa9392..9020e6968 100644
--- a/src/venus/fstrace.c
+++ b/src/venus/fstrace.c
@@ -661,7 +661,6 @@ icl_DumpKernel(FILE *outFilep, char *setname)
 			    i, (long)tname, sizeof(tname), 0, 0);
 	    if (code) {
 		if (errno == EBADF) {
-		    code = 0;
 		    continue;	/* missing slot, nothing to worry about */
 		}
 		break;