From: Chaskiel M Grundman <cg2v@andrew.cmu.edu>
Date: Sun, 29 Jun 2008 04:38:59 +0000 (+0000)
Subject: afs-blacklist-initialize-volume-ptr-always-20080628
X-Git-Tag: openafs-devel-1_5_61~1021
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=dff0d834a7c03d19d572d74849599ac89c2a058e;p=packages%2Fo%2Fopenafs.git

afs-blacklist-initialize-volume-ptr-always-20080628

LICENSE IPL10

always initialize this
---

diff --git a/src/afs/afs_analyze.c b/src/afs/afs_analyze.c
index 44745ccb2..78d9a5da1 100644
--- a/src/afs/afs_analyze.c
+++ b/src/afs/afs_analyze.c
@@ -247,20 +247,21 @@ afs_BlackListOnce(struct vrequest *areq, struct VenusFid *afid,
     afs_int32 i;
     afs_int32 serversleft = 0;
 
-    if (afid)
+    if (afid) {
 	tvp = afs_FindVolume(afid, READ_LOCK);
-    if (tvp) {
-	for (i = 0; i < MAXHOSTS; i++) {
-	    if (tvp->serverHost[i] == tsp) {
-		areq->skipserver[i] = 1;
-	    }
-	    if (tvp->serverHost[i] &&
-		!(tvp->serverHost[i]->addr->sa_flags & 
-		  SRVR_ISDOWN)) {
-		areq->skipserver[i] = 1;
+	if (tvp) {
+	    for (i = 0; i < MAXHOSTS; i++) {
+		if (tvp->serverHost[i] == tsp) {
+		    areq->skipserver[i] = 1;
+		}
+		if (tvp->serverHost[i] &&
+		    !(tvp->serverHost[i]->addr->sa_flags & 
+		      SRVR_ISDOWN)) {
+		    areq->skipserver[i] = 1;
+		}
 	    }
+	    afs_PutVolume(tvp, READ_LOCK);
 	}
-	afs_PutVolume(tvp, READ_LOCK);
     }
     for (i = 0; i < MAXHOSTS; i++) {
 	if (areq->skipserver[i] == 0) {