From: Christos Tsantilas <chtsanti@users.sourceforge.net>
Date: Sat, 28 Mar 2009 16:39:14 +0000 (+0200)
Subject: In ACLFilledChecklist::ACLFilledChecklist constructor the HttpRequest
X-Git-Tag: SQUID_3_2_0_1~1092
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f4462b38e240eeb2fb566e5df348f3c3c9ddc03f;p=thirdparty%2Fsquid.git

In ACLFilledChecklist::ACLFilledChecklist constructor the HttpRequest
parameter hides the ACLFilledChecklist::request member.
As a result acl's do not work at all and some cases squid crashes
(eg when dstdomain acls used)
---

diff --git a/src/acl/FilledChecklist.cc b/src/acl/FilledChecklist.cc
index 8c92a639b1..ed76ceb77d 100644
--- a/src/acl/FilledChecklist.cc
+++ b/src/acl/FilledChecklist.cc
@@ -229,7 +229,7 @@ ACLFilledChecklist::markSourceDomainChecked()
  *    *not* delete the list.  After the callback function returns,
  *    checkCallback() will delete the list (i.e., self).
  */
-ACLFilledChecklist::ACLFilledChecklist(const acl_access *A, HttpRequest *request, const char *ident):
+ACLFilledChecklist::ACLFilledChecklist(const acl_access *A, HttpRequest *http_request, const char *ident):
     dst_peer(NULL),
     request(NULL),
     reply(NULL),
@@ -255,8 +255,8 @@ ACLFilledChecklist::ACLFilledChecklist(const acl_access *A, HttpRequest *request
     if (A)
         accessList = cbdataReference(A);
 
-    if (request != NULL) {
-        request = HTTPMSGLOCK(request);
+    if (http_request != NULL) {
+        request = HTTPMSGLOCK(http_request);
 #if FOLLOW_X_FORWARDED_FOR
         if (Config.onoff.acl_uses_indirect_client)
             src_addr = request->indirect_client_addr;