From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Thu, 21 Jun 2012 23:37:25 +0000 (-0400)
Subject: Windows: cm_GetROVolumeID empty server list check
X-Git-Tag: upstream/1.8.0_pre1^2~2278
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=4785361388d21c46dd30332d89bb201522ee1a6c;p=packages%2Fo%2Fopenafs.git

Windows: cm_GetROVolumeID empty server list check

Instead of testing for a non-NULL server list, check to see
that the list itself is non-empty.

Change-Id: I1d8be33cd5f0713677d6013f5d7344400f6b1ae6
Reviewed-on: http://gerrit.openafs.org/7640
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsd/cm_volume.c b/src/WINNT/afsd/cm_volume.c
index 6a87a9ced..00960360d 100644
--- a/src/WINNT/afsd/cm_volume.c
+++ b/src/WINNT/afsd/cm_volume.c
@@ -1194,7 +1194,7 @@ long cm_GetROVolumeID(cm_volume_t *volp)
     long id;
 
     lock_ObtainRead(&volp->rw);
-    if (volp->vol[ROVOL].ID && volp->vol[ROVOL].serversp)
+    if (volp->vol[ROVOL].ID && !cm_IsServerListEmpty(volp->vol[ROVOL].serversp))
 	id = volp->vol[ROVOL].ID;
     else
 	id = volp->vol[RWVOL].ID;