budb: Don't double initialise iterator

The blockaddr iterator is initialised as part of the for loop.
Don't also initialise it immediately before.

Caught by clang-analyzer

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

diff --git a/src/budb/ol_verify.c b/src/budb/ol_verify.c
index 2c887177688e8f2d5d32a0260973579aefaa16df..aeb3cdf81ea29755617427abd795961a6414ff27 100644 (file)
--- a/src/budb/ol_verify.c
+++ b/src/budb/ol_verify.c
@@ -1188,7 +1188,6 @@ verifyTextChain(struct ubik_trans *ut, struct textBlock *tbPtr)
 
     for (new = 0; new < 2; new++) {
        size = 0;
-       blockAddr = ntohl(tbPtr->textAddr);
 
        for (blockAddr =
             (new ? ntohl(tbPtr->newTextAddr) : ntohl(tbPtr->textAddr));