git://git.openafs.org
/
openafs.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
(parent:
22196a7
)
viced-ubik-clientdestroy-null-20090703
author
Andrew Deason
<adeason@sinenomine.net>
Fri, 3 Jul 2009 14:03:57 +0000 (14:03 +0000)
committer
Derrick Brashear
<shadow@dementia.org>
Fri, 3 Jul 2009 14:03:57 +0000 (14:03 +0000)
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
patch
|
blob
|
history
diff --git
a/src/viced/afsfileprocs.c
b/src/viced/afsfileprocs.c
index
5e202c1
..
db37522
100644
(file)
--- 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)