]> git.ipfire.org Git - thirdparty/squid.git/commit
Honor the "deny" part of "foobar deny ACL" options - Temporary patch BumpSslServerFirst.take07
authorChristos Tsantilas <chtsanti@users.sourceforge.net>
Mon, 12 Mar 2012 18:15:44 +0000 (20:15 +0200)
committerChristos Tsantilas <chtsanti@users.sourceforge.net>
Mon, 12 Mar 2012 18:15:44 +0000 (20:15 +0200)
commit737c703d5d5711ed802580d06c601a2865902937
treea3647a6d51af629a32c68e592db3eb46a3444b52
parent2bd84e5f2b52b9190bc5760916ab41be13d44f4f
Honor the "deny" part of "foobar deny ACL" options - Temporary patch

When AuthenticateAcl() and aclMatchExternal() were converted to use extended
authentication ACL states (r11644 and r11645 dated 2011-08-14), the result of
those function calls was set as the current checklist answer.  This was
incorrect because those functions do not make allow/deny decisions. They only
tell us whether the ACL part of the allow/deny rule matches. If there is a
match, the ACCESS_ALLOWED/ACCESS_DENIED answer depends on whether it is an
allow or deny rule.

For example, "http_access deny BadGuys" should deny access when the BadGuys
ACL matches, but it was allowing access instead.
src/auth/AclMaxUserIp.cc
src/auth/AclProxyAuth.cc
src/external_acl.cc