]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug #2058: deny_info TCP_RESET crashes squid
authorhno <>
Thu, 30 Aug 2007 19:50:24 +0000 (19:50 +0000)
committerhno <>
Thu, 30 Aug 2007 19:50:24 +0000 (19:50 +0000)
commit2e216b1d94f7e97fb8c506574f15062c5b37db0b
treeee2db7b6679490eb8bb3397844192377945ffe39
parent0a628adacf8f97411c53b83784a2afaf4e95fc56
Bug #2058: deny_info TCP_RESET crashes squid

There was a race condition in request processing, easily triggered by using
deny_info TCP_RESET and denying access. It's very likely the problem could
also be triggered by other conditions where Squid very quickly closes the
client connection.

Was caused by a malplaced isClosed() condition. Moved this down to after the
request processing where it belongs.

This patch also adds some safeguards make further request processing stop and
avoid risking referencing the fd as valid after close.

Additionally connStateFree was renamed to the more appropriate connStateClosed
as all it does is to make the connState aware that the underlying fd has been
closed.
src/client_side.cc