From 6ef1234402435306cb3935d1eadc5fcf332bd632 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Fri, 3 Jul 2009 14:03:57 +0000 Subject: [PATCH 1/1] 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 --- src/viced/afsfileprocs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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) -- 1.9.4