]> git.ipfire.org Git - thirdparty/squid.git/commit
Exit without asserting when helper process startup fails (#1543)
authorAlex Rousskov <rousskov@measurement-factory.com>
Fri, 27 Oct 2023 21:27:20 +0000 (21:27 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Fri, 27 Oct 2023 21:27:30 +0000 (21:27 +0000)
commit7386e1fea0a7176c468ec954ec1f3bfb4b7f9dbb
tree417121459093294d8aecd154c60758978800c762
parent8275c50cbd1505ab532480e83ce6058beac0e42e
Exit without asserting when helper process startup fails (#1543)

... to dup() after fork() and before execvp().

Assertions are for handling program logic errors. Helper initialization
code already handled system call errors correctly (i.e. by exiting the
newly created helper process with an error), except for a couple of
assert()s that could be triggered by dup(2) failures.

This bug was discovered and detailed by Joshua Rogers at
https://megamansec.github.io/Squid-Security-Audit/ipc-assert.html
where it was filed as 'Assertion in Squid "Helper" Process Creator'.
src/ipc.cc