From: Andrew Deason <adeason@sinenomine.net>
Date: Fri, 3 Jul 2009 14:04:17 +0000 (+0000)
Subject: STABLE14-viced-ubik-clientdestroy-null-20090703
X-Git-Tag: openafs-stable-1_4_11pre3~2
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=0c5509ad2fdc5a4c0d9fe65be8e3f6faa0fcf250;p=packages%2Fo%2Fopenafs.git

STABLE14-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 555f0f53d..987c63320 100644
--- a/src/viced/afsfileprocs.c
+++ b/src/viced/afsfileprocs.c
@@ -336,8 +336,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)