From: Derrick Brashear <shadow@dementia.org>
Date: Mon, 22 Feb 2010 17:15:15 +0000 (-0500)
Subject: analyze paranoia
X-Git-Tag: openafs-devel-1_5_73~153
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=48908e2a932420fe67fe1065d55f7e458b5cd1c2;p=packages%2Fo%2Fopenafs.git

analyze paranoia

don't pass a null pointer to FindVolume in case one shows up

Change-Id: I76258d8f8ca18c536e3faa776df3e06005fe2bc2
Reviewed-on: http://gerrit.openafs.org/1360
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/afs_analyze.c b/src/afs/afs_analyze.c
index 443fae378..e7e07b40d 100644
--- a/src/afs/afs_analyze.c
+++ b/src/afs/afs_analyze.c
@@ -458,8 +458,9 @@ afs_Analyze(register struct afs_conn *aconn, afs_int32 acode,
     if ((acode < 0) && (acode != VRESTARTING)) {
 	if (acode == RX_CALL_TIMEOUT) {
 	    serversleft = afs_BlackListOnce(areq, afid, tsp);
-	    tvp = afs_FindVolume(afid, READ_LOCK);
-	    if (!tvp || (tvp->states & VRO))
+	    if (afid)
+		tvp = afs_FindVolume(afid, READ_LOCK);
+	    if (!afid || !tvp || (tvp->states & VRO))
 		areq->idleError++;
 	    if ((serversleft == 0) && tvp &&
 		((tvp->states & VRO) || (tvp->states & VBackup))) {