From: Andrew Deason <adeason@sinenomine.net>
Date: Fri, 3 Jul 2009 14:04:07 +0000 (+0000)
Subject: DEVEL15-viced-ubik-clientdestroy-null-20090703
X-Git-Url: http://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=3866da1c89b611f9e36a0d15543315e1a6134782

DEVEL15-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


(cherry picked from commit 6ef1234402435306cb3935d1eadc5fcf332bd632)
---

diff --git a/src/viced/afsfileprocs.c b/src/viced/afsfileprocs.c
index 5e202c1..db37522 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)