Fix socks4 and socks4a support under glibc's source fortification

With glib'c source fortification, gcc offers the compilation warning

    gateway.c: In function 'socks4_connect':
    gateway.c:840:4: warning: 'strlcpy' writing 4988 bytes into a region of size 1 overflows the destination
      840 |    strlcpy(&(c->userid), socks_userid, sizeof(buf) - sizeof(struct socks_op));
  |    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    gateway.c:112:9: note: destination object 'userid' of size 1
      112 |    char userid;               /* first byte of userid */
  |         ^~~~~~

resulting in a runtime abort() when using a socks4 or socks4a upstream proxy:

    $ privoxy --no-daemon <(echo 'forward-socks4 / 127.0.0.1:9050 .')
    7fef77483740 Info: Privoxy version 3.0.34
    7fef77483740 Info: Program name: privoxy
    *** buffer overflow detected ***: terminated
    rc: line 3: 321835 Aborted    (core dumped) privoxy --no-daemon <(echo 'forward-socks4 / 127.0.0.1:9050 .')

Despite the warning, the strlcpy() call in question is fine: gcc
misidentifies the size of the destination buffer, estimating to hold
only a single char while in fact the buffer stretches beyond the end of
the struct socks_op.

This commit fixes this issue in a way which is in line with the second
strlcpy() call in the socks4_connect(). Alternatively, we could also
remove the padding member and promote userid to a trailing flexible array
member. However, this would necessitate further adjustments because that
way the size of struct socks_op would change.

The issue was originally reported in the NixOS issue tracker at

    https://github.com/NixOS/nixpkgs/issues/265654

prompted by an upgrade of glibc from 2.37-39 to 2.38-0, and the fix is
joint work with @esclear and @richi235.

create_client_ssl_connection(): Make it more obviours from an error message that a function failed

privoxy-log-parser: Highlight: 'Couldn't deliver the error message for [...]'

Bump copyright

privoxy-log-parser: Bump copyright

privoxy-log-parser: Highlight 'Failed to accept() incoming connection: Software caused connection abort'

listen_loop(): Improve an error message

Bump copyright

Use stringify() instead of section_target()

... and remove section_target(). Like the XXX comment suggested
this could be done my moving the hash into the templates which
seems preferable anyway.

Block requests to secure-eu.nmrodam.com/

Block requests to o2.mouseflow.com/

uagen: Bump copyright

uagen: Bump version

uagen: Bump BROWSER_VERSION and BROWSER_REVISION

... to match Firefox ESR 115.

I don't know why the BROWSER_REVISION is still at 109
but that's what Firefox ESR 115 uses when compiled
on ElectroBSD.

Disable fast-redirects for /wp-content/plugins/pdf-viewer-for-elementor

Bump MAX_FILTER_TYPES

... which should have been done in d128e6aa41 when introducing
the client-body-tagger{} action.

Prevents an assertion in cgi_edit_actions_for_url() from triggering
after e32d03e0 when using the CGI editor with assertions enabled.

Add Aaron Li as contributor

Add missing client-body-tagger data to the action_type_info[] struct

... so lookups based on the action index work correctly again.

Prevents assertion failures or segfaults when trying to edit
an action file with the CGI editor.

The type of failure depended on whether or not assertions
were enabled and on whether or not Privoxy had been compiled
with FEATURE_EXTERNAL_FILTERS.

Regression introduced in Privoxy 3.0.34.

Patch submitted by Aaron Li in #940.

cgi_edit_actions_for_url(): Wrap line sooner

action_render_string_actions_template(): Assert that the multi action index is valid

cgi_edit_actions_for_url(): Assert that the multi action index is valid

cgi_edit_actions_for_url(): Add missing space

action_render_string_actions_template: Fix spelling in description and re-flow

action_render_string_actions_template(): Adjust space around function parameters

Add a regression test for left-anchored path patterns

pcre2 compile_pattern(): Actually pass the anchored pattern to pcre2_compile()

Previously the un-anchoring pattern was compiled resulting
in incorrect matches.

For example requests to:

    https://www.privoxy.org/user-manual/config.html

were redirected because of the default.action section:

    {+redirect{http://config.privoxy.org/}}
    # Sticky Actions = +redirect{http://config.privoxy.org/}
    # URL = http://www.privoxy.org/config
    # Redirected URL = http://www.privoxy.org/config
    # Redirect Destination = http://config.privoxy.org/
    .privoxy.org/config

As the path pattern is left-anchored it should not match.

configure: Fix --disable-pcre2

Previously it would result in neither pcre library being detected:

    checking for getnameinfo... (cached) yes
    configure: WARNING: Ignoring pcre2 even if it's available
    test: =: unexpected operator
    Enabling support for client-specific tags.
    checking for zlibVersion in -lz... (cached) yes
    Enabling compression support.
    test: =: unexpected operator
    test: =: unexpected operator
    configure: error: Detected neither pcre2 nor pcre library.

Disable fast-redirects for services.akteneinsichtsportal.de/

Add #205: Document how commit messages should look like

Add #204: Make the Privoxy website available over IPv6.

Add #203: Add HTTP/2 support

Remove TODO item #164

The PCRE2 code uses the native API as the POSIX API resulted in crashes.

Add regex_matches() to reduce HAVE_PCRE2 ifdefs

trust: Use the words 'allowlists' and 'blocklists'

... instead of "whitelists" and "blacklists" which some
people consider to be less inclusive.

Rebuild homepage without former sponsor Scrubtheweb.com

Remove sponsor Scrubtheweb.com as they canceled the subscription

ssl_send_certificate_error(): Remove a 'dead nested assignment'

pcrs_execute_single_command(): Remove a 'dead assignment'

fix gcc maybe-uninitialized compiler warnings in acl_addr

I can't tell if they're false positives or not, but compiling with
no warnings is better than seeing this:

filters.c: In function ‘acl_addr’:
filters.c:465:31: warning: ‘addr_len’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  465 |    for (i = 0; (i < addr_len) && masklength; i++)
      |                ~~~~~~~~~~~~~~~^~~~~~~~~~~~~
filters.c:454:18: warning: ‘mask_port’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  454 |       *mask_port = 1;
      |       ~~~~~~~~~~~^~~
filters.c:479:19: warning: ‘mask_data’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  479 |          mask_data[i] = (uint8_t)~((1 << (8 - masklength)) - 1);
      |                   ^

fix gcc maybe-uninitialized compiler warnings in match_sockaddr

I can't tell if they're false positives or not, but compiling with
no warnings is better than seeing this:

filters.c: In function ‘match_sockaddr’:
filters.c:205:42: warning: ‘address_port’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  205 |    if (*netmask_port && *network_port != *address_port)
      |                                          ^~~~~~~~~~~~~
filters.c:214:24: warning: ‘address_addr’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  214 |           (address_addr[i] & netmask_addr[i]))
      |            ~~~~~~~~~~~~^~~
filters.c:205:8: warning: ‘netmask_port’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  205 |    if (*netmask_port && *network_port != *address_port)
      |        ^~~~~~~~~~~~~
filters.c:200:20: warning: ‘netmask_addr’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  200 |       netmask_addr += 12;
      |                    ^~
filters.c:205:25: warning: ‘network_port’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  205 |    if (*netmask_port && *network_port != *address_port)
      |                         ^~~~~~~~~~~~~
filters.c:172:17: warning: ‘addr_len’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  172 |    unsigned int addr_len;
      |                 ^~~~~~~~

windows/MYconfigure: enable building Privoxy with OpenSSL on Windows.

way too many warnings in openssl.c about RSA deprecated-declarations
so add -Wno-deprecated-declarations to get rid of them.

make it easier to specify "--with-mbedtls" or "--with-openssl"

windows: Enable building Privoxy with OpenSSL

- openssl.c

I got random crashes when using openssl on windows; fixed with
  #include <openssl/applink.c>
I don't understand
  https://www.openssl.org/docs/faq.html
it seems to be applicable only to calling DLLs but even with
no DLLs involved that include also fixes the crashes that happen
when statically linking openssl

- project.h

pull in the required gunk for windows

windows/MYconfigure: Have gcc diagnostics in color

It's nice having gcc warnings in color even if I run a script that does
  'make 2>&1 | tee  log.make'

create_client_ssl_connection(): Don't keep the certificate lock longer than necessary

Enable use of the PCRE2 library on Windows

- windows/MYconfigure

Use the PCRE2 library for building Privoxy on Windows.

- configure.in

PCRE2_STATIC must be defined if you want to statically link the PCRE2 library.
If it isn't defined you'll get these kind of linker errors:
 undefined reference to `_imp__pcre2_compile_8'
 undefined reference to `_imp__pcre2_jit_compile_8'
 undefined reference to `_imp__pcre2_match_data_create_from_pattern_8'
 undefined reference to `_imp__pcre2_match_8'

windows/MYconfigure: Use Mbed TLS 2.28.4

LogCreatePatternMatchingBuffers(): Remove the #warning I added in 53748ca8c

According to Lee it doesn't seem to be warranted.

Fix a compiler warning emitted by clang 15.0.7

Silences:

    filters.c:2561:58: warning: passing arguments to a function without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
       content = (content_filter != NULL) ? (*content_filter)(csp) : NULL;
                                                             ^
    1 warning generated.

Bump copyright

Block requests to track.venatusmedia.com/

Block requests to i.clean.gg/

Block requests to s.cpx.to/

privoxy-regression-test.pl: Bump copyright

privoxy-regression-test.pl: Bump version

privoxy-regression-test.pl: Let the --min-level option increase the --max-level

.. if the latter is smaller than the former.

regression-tests.action: Bump copyright

regression-tests.action: Add a redirect{} test with a pcrs command

Add Gagan Sidhu as contributor

Add pcre2 support

This is currently expected to cause crashes on Windows
when compiled with GUI support.

Closes bug #935.
Initial patch submitted by: Gagan Sidhu

log_error() Win32: Only call LogShowActivity() for debug level LOG_LEVEL_REQUEST

As of b94bbe62a950, which was part of Privoxy 3.0.29,
LOG_LEVEL_REQUEST is used for all requests including
crunched ones.

Previously LogShowActivity() was called twice for crunched
requests, (presumably) resulting in an aborted animation.

Add #202:  Allow to use multiple log files with different debug settings.

Move the Scrubtheweb.com link to the Silver sponsor section where it belongs

Rebuild homepage

user-manual: Fix markup

Rebuild homepage without stray period

Ditch a stray period

Rebuild homepage with Gold sponsor Scrubtheweb.com

Add Gold sponsor Scrubtheweb.com

show-status template: declare https-inspection non-experimental.

Regenerate user-manual with HOWTOs.

Add HOWTOs for https inspection and client-tags to user-manual.

Update TODO item #150

blacklistd has been renamed to blocklistd. While at it,
add an URL to the man page.

Regenerate docs

Bump SMGL entities for 3.0.35 UNRELEASED

user-manual: Suggest to use the force-text-mode action when filtering binary content

... with external filters.

developer-manual: Bump copyright

parse_numeric_value(): Expect a base-ten number

configure: Bump version to 3.0.35 UNRELEASED

Rebuild developer manual

developer-manual: Document the regeneration of the RSS feed

Update RSS feed for the 3.0.34 release

Merge Debian 3.0.33-4 and 3.0.34-1 changes.

And prepare 3.0.35 git snapshot.

Rebuild docs with updated copyright

Bump copyright

Regenerate config file for the 3.0.34 release

Rebuild HTML man page

Regenerate privoxy.8 for the 3.0.34 release

Update license information in the annnouncement

Add a paragraph to the annoucement to explain why gmail users no longer get messages from Mailman

Regenerate the documentation

Prepare the announcement text for Privoxy 3.0.34 stable

Update SGML ChangeLog

Add two ChangeLog entries

user-manual: Note that FreeBSD and ElectroBSD users can try to install Privoxy

... as binary package using 'pkg'.

user-manual: Note that the FreeBSD installation instructions work for ElectroBSD as well

Rebuild user-manual with updated copyright

user-manual: Bump copyright

Rebuild user-manual with updated ChangeLog entries

Update SGML changelog

Improve a couple of ChangeLog entries

... by more consistently using periods at the end of the
entries and by slightly rephrasing some.