]> git.ipfire.org Git - thirdparty/squid.git/commit
Nil request dereference in ACLExtUser and SourceDomainCheck ACLs (#1931)
authorEduard Bagdasaryan <eduard.bagdasaryan@measurement-factory.com>
Tue, 5 Nov 2024 14:21:19 +0000 (14:21 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Tue, 5 Nov 2024 14:28:17 +0000 (14:28 +0000)
commit0ef767a500ad47f2791fb1f065c29c08543cd119
tree64584a35ab034a0a0d6f918b67fc82248d3b5724
parentfff4502b16fdc4941fa24b1eb429788286ecd106
Nil request dereference in ACLExtUser and SourceDomainCheck ACLs (#1931)

ACLExtUser-based ACLs (i.e. ext_user and ext_user_regex) dereferenced a
nil request pointer when they were used in a context without a request
(e.g., when honoring on_unsupported_protocol).

SourceDomainCheck-based ACLs (i.e. srcdomain and srcdom_regex) have a
similar bug, although we do not know whether broken slow ACL code is
reachable without a request (e.g., on_unsupported_protocol tests cannot
reach that code until that directive starts supporting slow ACLs). This
change does not start to require request presence for these two ACLs to
avoid breaking any existing configurations that "work" without one.
src/acl/ExtUser.h
src/acl/SourceDomain.cc