]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
vol: nuke() doesn't need to maintain next inode ptr
authorSimon Wilkinson <sxw@your-file-system.com>
Tue, 19 Feb 2013 14:47:56 +0000 (14:47 +0000)
committerJeffrey Altman <jaltman@your-file-system.com>
Thu, 21 Feb 2013 17:06:41 +0000 (09:06 -0800)
ni is unused on the true side of the if expression in nuke() - so
just don't bother populating it.

Caught by clang-analyzer

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

index 1618b44ea69397ba626b1cb79025a78e4d3e7017..3399de57e6ac81e0a3336a1a8cfc2ec9f16d7fb7 100644 (file)
@@ -186,7 +186,6 @@ nuke(char *aname, VolumeId avolid)
                IH_RELEASE(fileH);
 #endif /* AFS_NAMEI_ENV */
            }
-           ni = ti->next;
            if (li) free(li);
            li = ti;
        }