DEVEL15-viced-ubik-clientdestroy-null-20090703
authorAndrew Deason <adeason@sinenomine.net>
Fri, 3 Jul 2009 14:04:07 +0000 (14:04 +0000)
committerDerrick Brashear <shadow@dementia.org>
Fri, 3 Jul 2009 14:04:07 +0000 (14:04 +0000)
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)

src/viced/afsfileprocs.c

index 5e202c1..db37522 100644 (file)
@@ -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)