vlserver: Remove dead assignment
authorSimon Wilkinson <sxw@your-file-system.com>
Thu, 21 Feb 2013 21:16:40 +0000 (21:16 +0000)
committerDerrick Brashear <shadow@your-file-system.com>
Sun, 24 Feb 2013 13:22:13 +0000 (05:22 -0800)
Don't assign the serverNumber to j, when we're just about to try
and get it from a multi homed extent record

Caught by clang-analyzer.

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

src/vlserver/vlprocs.c

index 715e2a0..baa2a4c 100644 (file)
@@ -3107,7 +3107,6 @@ vlentry_to_vldbentry(struct vl_ctx *ctx, struct nvlentry *VlEntry,
     for (i = 0; i < OMAXNSERVERS; i++) {
        if (VlEntry->serverNumber[i] == BADSERVERID)
            break;
-       j = VlEntry->serverNumber[i];
        code = multiHomedExtent(ctx, VlEntry->serverNumber[i], &exp);
        if (code)
            return code;