From: Andrew Deason <adeason@sinenomine.net>
Date: Fri, 3 Jul 2009 14:03:57 +0000 (+0000)
Subject: viced-ubik-clientdestroy-null-20090703
X-Git-Tag: openafs-devel-1_5_61~183
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=6ef1234402435306cb3935d1eadc5fcf332bd632;p=packages%2Fo%2Fopenafs.git

viced-ubik-clientdestroy-null-20090703

LICENSE IPL10
FIXES 125020

null ubik client point when it's freed in hpr_End to avoid a null deref later
---

diff --git a/src/viced/afsfileprocs.c b/src/viced/afsfileprocs.c
index 5e202c186..db375223d 100644
--- a/src/viced/afsfileprocs.c
+++ b/src/viced/afsfileprocs.c
@@ -340,8 +340,10 @@ CallPreamble(register struct rx_call *acall, int activecall,
 
 	/* Is it still necessary to drop this? We hit the net, we should... */
 	H_UNLOCK;
-	if (uclient) 
+	if (uclient) {
 	    hpr_End(uclient);
+	    uclient = NULL;
+	}
 	code = hpr_Initialize(&uclient);
 
 	if (!code)