libafs: don't crash on no addresses in afs_Conn
authorDerrick Brashear <shadow@dementix.org>
Thu, 26 Apr 2012 13:24:25 +0000 (09:24 -0400)
committerDerrick Brashear <shadow@dementix.org>
Thu, 26 Apr 2012 15:37:16 +0000 (08:37 -0700)
we try to avoid a crash on a missing serverHost; do the same
on missing addrs in that host

FIXES 130714

Change-Id: Ic6bab8654cf7bf85351a9482b9ba5d51e5a768d5
Reviewed-on: http://gerrit.openafs.org/7287
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementix.org>

src/afs/afs_conn.c

index 09d7d8f..8b2dd73 100644 (file)
@@ -301,6 +301,7 @@ afs_Conn(struct VenusFid *afid, struct vrequest *areq,
 
     /* First is always lowest rank, if it's up */
     if ((tv->status[0] == not_busy) && tv->serverHost[0]
+       && tv->serverHost[0]->addr
        && !(tv->serverHost[0]->addr->sa_flags & SRVR_ISDOWN) &&
        !(((areq->idleError > 0) || (areq->tokenError > 0))
          && (areq->skipserver[0] == 1)))