]> git.ipfire.org Git - people/pmueller/ipfire-2.x.git/commit
squid-asnbl: Fix for bug#13023 - squid-asnbl-helper segfaulting and shutdown squid
authorAdolf Belka <adolf.belka@ipfire.org>
Wed, 22 Mar 2023 18:28:52 +0000 (19:28 +0100)
committerPeter Müller <peter.mueller@ipfire.org>
Sat, 1 Jul 2023 19:30:03 +0000 (19:30 +0000)
commitdf30842927aac053cd669cee8105d2f39345c3b1
treed7499ab39caf2cb1b44143abdbdeb64ceb36157f
parent81ed7573b10b07884346d3fa4336e05944fd48a7
squid-asnbl: Fix for bug#13023 - squid-asnbl-helper segfaulting and shutdown squid

- Patch provided by bug reporter. Here is the description of the problem from the bug.
   First I discovered that the helper only sometimes throwing the error and quits even
   for the same values and queries. Also the timespan until the error happens was quite
   different for every restart of squid  (minutes to hours). And it does not depend on
   the traffic on the proxy, even one connection could cause a crash while ten or
   hundrets won't. After a few days of testing different solutions and done a lot of
   debugging, redesigning the function did not fully solve the problem. Such standard
   things like checking the result variable for NULL (or it's equivalent "is None" in
   python) before evaluating it's subfunction produces the exact same error message. But
   with that knowledge it more and more turns out that python3 sometimes 'detects' the
   local return variable if it was a misused global. So for a full fix, the return
   variable also has to be initialized that python3 won't detect it's usage as an
   'UnboundLocalError' to succesfully fix this bug.
- LFS file updated to run patch before copying helper into place.
- Update of rootfile not needed.
- Bug reporter has been requested to raise this issue at the git repo for squid-asnbl.

Fixes: Bug#13023
Tested-by: Nicolas Pӧhlmann <business@hardcoretec.com>
Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
lfs/squid-asnbl
src/patches/squid/squid-asnbl-0.2.4_initialise_global_variables_to_make_compatible_with_python_3.patch [new file with mode: 0644]