]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug 4919: master commit b599471 leaks memory (#364) M-staged-PR364
authorAlex Rousskov <rousskov@measurement-factory.com>
Thu, 14 Feb 2019 18:48:56 +0000 (18:48 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Thu, 14 Feb 2019 21:21:32 +0000 (21:21 +0000)
commit664dc2670410b8498b550b3c0402ab61992e615b
treeff029b1515914bf38cc33fe819cedf11049106c6
parenta56fcf0b804aebbea6dcc59d99448dcc84925ac7
Bug 4919: master commit b599471 leaks memory (#364)

Restored the natural order of the following two notifications:
* BodyConsumer::noteMoreBodyDataAvailable() and
* BodyConsumer::noteBodyProductionEnded() or noteBodyProducerAborted().

Commit b599471 unintentionally reordered those two notifications. Client
kids (and possibly other BodyConsumers) relied on the natural order to
end their work. If an HttpStateData job was done with the Squid-to-peer
connection and only waiting for the last adapted body bytes, it would
get stuck and leak many objects. This use case was not tested during
b599471 work.
src/BodyPipe.cc