]> git.ipfire.org Git - thirdparty/squid.git/commit - src/http.cc
Always call processReplyBody() after adaptOrFinalizeReply().
authorDmitry Kurochkin <dmitry.kurochkin@measurement-factory.com>
Wed, 29 Jun 2011 05:59:07 +0000 (09:59 +0400)
committerDmitry Kurochkin <dmitry.kurochkin@measurement-factory.com>
Wed, 29 Jun 2011 05:59:07 +0000 (09:59 +0400)
commitef85ab2f777c5d44cd2d6d769b4bda47e821f170
treecbe30f9e0ab893626ba59b8e4820b752ee88385a
parent90110b9645849db8e9fae42ffb5827a17d22c152
Always call processReplyBody() after adaptOrFinalizeReply().

The recent fix for "store_status == STORE_PENDING" added a check
with return from HttpStateData::processReply() before
processReplyBody() call.  But processReplyBody() does proper
server state cleanup.  We could end up with server state that
would not be cleaned (until connection is closed).  That could
also introduce other issues.  The patch moves the isAccepting()
check from processReply() to processReplyBody().  We do not write
to an entry that is not accepting, but allow processReplyBody()
to do proper cleanup.
src/http.cc