jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-19 15:30:14 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

callpostamble-deal-with-null-host-20060905

Browse Source 
FIXES 38566

don't dereference garbage when no host was set
This commit is contained in:
Derrick Brashear 2006-09-05 17:30:33 +00:00
parent 2c103b8fb3
commit 56bbd155db
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/viced/afsfileprocs.c
View File

@ -311,6 +311,8 @@ CallPreamble(register struct rx_call *acall, int activecall,
char hoststr[16], hoststr2[16];
struct ubik_client *uclient;
*ahostp = NULL;
if (!tconn) {
ViceLog(0, ("CallPreamble: unexpected null tconn!\n"));
return -1;
@ -430,14 +432,19 @@ CallPostamble(register struct rx_connection *aconn, afs_int32 ret,
held = h_Held_r(thost);
if (held)
h_Release_r(thost);
if (ahost != thost) {
if (ahost && ahost != thost) {
char hoststr[16], hoststr2[16];
ViceLog(0, ("CallPostamble: ahost %s:%d (%x) != thost %s:%d (%x)\n",
afs_inet_ntoa_r(thost->host, hoststr), ntohs(thost->port),
afs_inet_ntoa_r(ahost->host, hoststr), ntohs(ahost->port),
ahost,
afs_inet_ntoa_r(thost->host, hoststr2), ntohs(thost->port),
thost));
h_Release_r(ahost);
} else if (!ahost) {
char hoststr[16];
ViceLog(0, ("CallPostamble: null ahost for thost %s:%d (%x)\n",
afs_inet_ntoa_r(thost->host, hoststr), ntohs(thost->port),
thost));
}
H_UNLOCK;
return (translate ? sys_error_to_et(ret) : ret);