From: Christos Tsantilas <christos@chtsanti.net>
Date: Sat, 29 Jun 2019 07:15:34 +0000 (+0000)
Subject: Bug 4842: Memory leak when http_reply_access uses external_acl (#424)
X-Git-Tag: SQUID_5_0_1~76
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=db85103692c087a301e89e03de0a0d5368a5cf23;p=thirdparty%2Fsquid.git

Bug 4842: Memory leak when http_reply_access uses external_acl (#424)

Http::One::Server::handleReply() sets AccessLogEntry::reply which may
already be set. It is already set, for example, when the ACL code
has already called syncAle() because external ACLs require an ALE.

This bug was introduced by commit fbbea6620.

This is a Measurement Factory project.
---

diff --git a/src/servers/Http1Server.cc b/src/servers/Http1Server.cc
index 7bb6190f7f..a0ee371b81 100644
--- a/src/servers/Http1Server.cc
+++ b/src/servers/Http1Server.cc
@@ -312,6 +312,7 @@ Http::One::Server::handleReply(HttpReply *rep, StoreIOBuffer receivedData)
     }
 
     assert(rep);
+    HTTPMSGUNLOCK(http->al->reply);
     http->al->reply = rep;
     HTTPMSGLOCK(http->al->reply);
     context->sendStartOfMessage(rep, receivedData);