------------------------------------------------------------
revno: 12665
revision-id: squid3@treenet.co.nz-20131130091707-jo49nw1xcvozbz5r
parent: squid3@treenet.co.nz-20131130090657-i3lxe25svrigl3xy
fixes bug(s): http://bugs.squid-cache.org/show_bug.cgi?id=3972
author: Nathan Hoad <nathan@getoffmalawn.com>
committer: Amos Jeffries <squid3@treenet.co.nz>
branch nick: 3.3
timestamp: Sat 2013-11-30 02:17:07 -0700
message:
  Bug 3972: Segfault when getting the deny info page ID after a reconfigure
  
  Older ACL code was using a stale AclMatchedName value. More recent code resets
  the AclMatchedName global to NULL to avoid leaking the stale value, but that
  may crash strcmp() in aclGetDenyInfoPage(). Long-term, the global should be
  removed, of course.
------------------------------------------------------------
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: squid3@treenet.co.nz-20131130091707-jo49nw1xcvozbz5r
# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.3
# testament_sha1: 75738e2eea1c5d1dfd93a6cfb084564dbbcadf67
# timestamp: 2013-11-30 09:53:24 +0000
# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.3
# base_revision_id: squid3@treenet.co.nz-20131130090657-\
#   i3lxe25svrigl3xy
# 
# Begin patch
=== modified file 'src/acl/Gadgets.cc'
--- src/acl/Gadgets.cc	2012-09-05 14:49:29 +0000
+++ src/acl/Gadgets.cc	2013-11-30 09:17:07 +0000
@@ -53,6 +53,11 @@
 err_type
 aclGetDenyInfoPage(AclDenyInfoList ** head, const char *name, int redirect_allowed)
 {
+    if (!name) {
+        debugs(28, 3, "ERR_NONE due to a NULL name");
+        return ERR_NONE;
+    }
+
     AclDenyInfoList *A = NULL;
 
     debugs(28, 8, HERE << "got called for " << name);