h-gethost-r-race-20030401
authorDerrick Brashear <shadow@dementia.org>
Wed, 2 Apr 2003 01:22:16 +0000 (01:22 +0000)
committerDerrick Brashear <shadow@dementia.org>
Wed, 2 Apr 2003 01:22:16 +0000 (01:22 +0000)
commit5967b22698c8aeb51131a62c56a2f7fbf1f8e79e
treeb3e9366eaa15898fc34789a62c1a632cffd1cc6c
parentf14686ea2e4d10381554bd466d236dd9b9479e4a
h-gethost-r-race-20030401

FIXES 1308

Thanks to Chaskiel Grundman for explaining what was happening:
- the connection is old and pre-existing, but has no host structure.
- 2 calls come in
- the first one enters h_GetHost_r, and h_Lookup_r returns null (but identP is
non-null, since rx keeps it around until it gc's the connection)
  The first thread calls WhoAreYou, which succeeds, it then calls
InitCallBackState3 (after H_UNLOCK) note that the host has been inserted into
the hashtable
- the second thread enters h_GetHost_r, and calls rx_GetSpecific. it then
calls h_Lookup_r. h_Lookup_r will block (new host is locked), but eventually
returns the new host
- InitCallBackState3 returns, and the frees the old identP, replaces it, and
unlocks the host.
- the first thread returns from h_Lookup_r. boom.

the changes:
-call rx_GetSpecific after h_Lookup_r returns (and potentially slept)
-removes an if wrapping which always is true (since !interfValid is always
 true)
-don't realloc identP if it exists
-don't free an old one by calling rx_SetSpecific either
src/viced/host.c