]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug 4919: master commit b599471 leaks memory (#364) (#368)
authorsquidadm <squidadm@users.noreply.github.com>
Sun, 17 Feb 2019 22:55:02 +0000 (11:55 +1300)
committerAmos Jeffries <yadij@users.noreply.github.com>
Sun, 17 Feb 2019 22:55:02 +0000 (11:55 +1300)
commit607b68b7108f3bef8fbda0f89db93c092462b9d6
tree54cbda0f2e3a3faffec1aed688eb0886144409cb
parent1e3da85d14dbda4389f93c822a1e3123e69aedfc
Bug 4919: master commit b599471 leaks memory (#364) (#368)

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