<!--
- File : $Source: /cvsroot/ijbswa/current/doc/source/changelog.sgml,v $
+ File : doc/source/changelog.sgml
Purpose : Entity included in other project documents.
- $Id: changelog.sgml,v 2.14 2016/01/17 14:31:33 fabiankeil Exp $
-
- Copyright (C) 2013 Privoxy Developers http://www.privoxy.org/
+ Copyright (C) 2013-2021 Privoxy Developers https://www.privoxy.org/
See LICENSE.
======================================================================
user-manual
-->
-<para>
- <application>Privoxy 3.0.24</application> stable contains a couple
- of new features but is mainly a bug-fix release. Two of the fixed bugs
- are security issues (CVE requests pending) and may be used to remotely
- trigger crashes on platforms that carefully check memory accesses (most don't).
-</para>
<!--
The SGML ChangeLog can be generated with: utils/changelog2doc.pl ChangeLog
-->
-<para>
<para>
+ <application>Privoxy 3.0.32</application> fixes multiple DoS issues
+ and a couple of other bugs. The issues also affect earlier Privoxy
+ releases.
+</para>
+ <para>
+ Changes in <application>Privoxy 3.0.32</application> stable:
+ </para>
+ <para>
<itemizedlist>
<listitem>
<para>
- Security fixes (denial of service):
+ Security/Reliability:
<itemizedlist>
<listitem>
<para>
- Prevent invalid reads in case of corrupt chunk-encoded content.
- Bug discovered with afl-fuzz and AddressSanitizer.
+ ssplit(): Remove an assertion that could be triggered with a
+ crafted CGI request.
+ Commit 2256d7b4d67. OVE-20210203-0001.
+ Reported by: Joshua Rogers (Opera)
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ cgi_send_banner(): Overrule invalid image types. Prevents a
+ crash with a crafted CGI request if Privoxy is toggled off.
+ Commit e711c505c48. OVE-20210206-0001.
+ Reported by: Joshua Rogers (Opera)
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ socks5_connect(): Don't try to send credentials when none are
+ configured. Fixes a crash due to a NULL-pointer dereference
+ when the socks server misbehaves.
+ Commit 85817cc55b9. OVE-20210207-0001.
+ Reported by: Joshua Rogers (Opera)
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ chunked_body_is_complete(): Prevent an invalid read of size two.
+ Commit a912ba7bc9c. OVE-20210205-0001.
+ Reported by: Joshua Rogers (Opera)
</para>
</listitem>
<listitem>
<para>
- Remove empty Host headers in client requests.
- Previously they would result in invalid reads.
- Bug discovered with afl-fuzz and AddressSanitizer.
+ Obsolete pcre: Prevent invalid memory accesses with an invalid
+ pattern passed to pcre_compile(). Note that the obsolete pcre code
+ is scheduled to be removed before the 3.0.33 release. There has been
+ a warning since 2008 already.
+ Commit 28512e5b624. OVE-20210222-0001.
+ Reported by: Joshua Rogers (Opera)
</para>
</listitem>
</itemizedlist>
<itemizedlist>
<listitem>
<para>
- When using socks5t, send the request body optimistically as well.
- Previously the request body wasn't guaranteed to be sent at all
- and the error message incorrectly blamed the server.
- Fixes #1686 reported by Peter Müller and G4JC.
+ Properly parse the client-tag-lifetime directive. Previously it was
+ not accepted as an obsolete hash value was being used.
+ Reported by: Joshua Rogers (Opera)
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ decompress_iob(): Prevent reading of uninitialized data.
+ Reported by: Joshua Rogers (Opera).
</para>
</listitem>
<listitem>
<para>
- Fixed buffer scaling in execute_external_filter() that could lead
- to crashes. Submitted by Yang Xia in #892.
+ decompress_iob(): Don't advance cur past eod when looking
+ for the end of the file name and comment.
</para>
</listitem>
<listitem>
<para>
- Fixed crashes when executing external filters on platforms like
- Mac OS X. Reported by Jonathan McKenzie on ijbswa-users@.
+ decompress_iob(): Cast value to unsigned char before shifting.
+ Prevents a left-shift of a negative value which is undefined behaviour.
+ Reported by: Joshua Rogers (Opera)
</para>
</listitem>
<listitem>
<para>
- Properly parse ACL directives with ports when compiled with HAVE_RFC2553.
- Previously the port wasn't removed from the host and in case of
- 'permit-access 127.0.0.1 example.org:80' Privoxy would try (and fail)
- to resolve "example.org:80" instead of example.org.
- Reported by Pak Chan on ijbswa-users@.
+ gif_deanimate(): Confirm that that we have enough data before doing
+ any work. Fixes a crash when fuzzing with an empty document.
+ Reported by: Joshua Rogers (Opera).
</para>
</listitem>
<listitem>
<para>
- Check requests more carefully before serving them forcefully
- when blocks aren't enforced. Privoxy always adds the force token
- at the beginning of the path, but would previously accept it anywhere
- in the request line. This could result in requests being served that
- should be blocked. For example in case of pages that were loaded with
- force and contained JavaScript to create additionally requests that
- embed the origin URL (thus inheriting the force prefix).
- The bug is not considered a security issue and the fix does not make
- it harder for remote sites to intentionally circumvent blocks if
- Privoxy isn't configured to enforce them.
- Fixes #1695 reported by Korda.
+ buf_copy(): Fail if there's no data to write or nothing to do.
+ Prevents undefined behaviour "applying zero offset to null pointer".
+ Reported by: Joshua Rogers (Opera)
</para>
</listitem>
<listitem>
<para>
- Normalize the request line in intercepted requests to make rewriting
- the destination more convenient. Previously rewrites for intercepted
- requests were expected to fail unless $hostport was being used, but
- they failed "the wrong way" and would result in an out-of-memory
- message (vanilla host patterns) or a crash (extended host patterns).
- Reported by "Guybrush Threepwood" in #1694.
+ log_error(): Treat LOG_LEVEL_FATAL as fatal even when --stfu is
+ being used while fuzzing.
+ Reported by: Joshua Rogers (Opera).
</para>
</listitem>
<listitem>
<para>
- Enable socket lingering for the correct socket.
- Previously it was repeatedly enabled for the listen socket
- instead of for the accepted socket. The bug was found by
- code inspection and did not cause any (reported) issues.
+ Respect DESTDIR when considering whether or not to install
+ config files with ".new" extension.
</para>
</listitem>
<listitem>
<para>
- Detect and reject parameters for parameter-less actions.
- Previously they were silently ignored.
+ OpenSSL ssl_store_cert(): Fix two error messages.
</para>
</listitem>
<listitem>
<para>
- Fixed invalid reads in internal and outdated pcre code.
- Found with afl-fuzz and AddressSanitizer.
+ Fix a couple of format specifiers.
</para>
</listitem>
<listitem>
<para>
- Prevent invalid read when loading invalid action files.
- Found with afl-fuzz and AddressSanitizer.
+ Silence compiler warnings when compiling with NDEBUG.
</para>
</listitem>
<listitem>
<para>
- Windows build: Use the correct function to close the event handle.
- It's unclear if this bug had a negative impact on Privoxy's behaviour.
- Reported by Jarry Xu in #891.
+ fuzz_server_header(): Fix compiler warning.
</para>
</listitem>
<listitem>
<para>
- In case of invalid forward-socks5(t) directives, use the
- correct directive name in the error messages. Previously they
- referred to forward-socks4t failures.
- Reported by Joel Verhagen in #889.
+ fuzz_client_header(): Fix compiler warning.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ cgi_send_user_manual(): Also reject requests if the user-manual
+ directive specifies a https:// URL. Previously Privoxy would try and
+ fail to open a local file.
</para>
</listitem>
</itemizedlist>
<itemizedlist>
<listitem>
<para>
- Set NO_DELAY flag for the accepting socket. This significantly reduces
- the latency if the operating system is not configured to set the flag
- by default. Reported by Johan Sintorn in #894.
+ Log the TLS version and the the cipher when debug 2 is enabled.
</para>
</listitem>
<listitem>
<para>
- Allow to build with mingw x86_64. Submitted by Rustam Abdullaev in #135.
+ ssl_send_certificate_error(): Respect HEAD requests by not sending a body.
</para>
</listitem>
<listitem>
<para>
- Introduce the new forwarding type 'forward-webserver'.
- Currently it is only supported by the forward-override{} action and
- there's no config directive with the same name. The forwarding type
- is similar to 'forward', but the request line only contains the path
- instead of the complete URL.
+ ssl_send_certificate_error(): End the body with a single new line.
</para>
</listitem>
<listitem>
<para>
- The CGI editor no longer treats 'standard.action' special.
- Nowadays the official "standards" are part of default.action
- and there's no obvious reason to disallow editing them through
- the cgi editor anyway (if the user decided that the lack of
- authentication isn't an issue in her environment).
+ serve(): Increase the chances that the host is logged when closing
+ a server socket.
</para>
</listitem>
<listitem>
<para>
- Improved error messages when rejecting intercepted requests
- with unknown destination.
+ handle_established_connection(): Add parentheses to clarify an expression
+ Suggested by: David Binderman
</para>
</listitem>
<listitem>
<para>
- A couple of log messages now include the number of active threads.
+ continue_https_chat(): Explicitly unset CSP_FLAG_CLIENT_CONNECTION_KEEP_ALIVE
+ if process_encrypted_request() fails. This makes it more obvious that the
+ connection will not be reused. Previously serve() relied on
+ CSP_FLAG_SERVER_CONTENT_LENGTH_SET and CSP_FLAG_CHUNKED being unset.
+ Inspired by a patch from Joshua Rogers (Opera).
</para>
</listitem>
<listitem>
<para>
- Removed non-standard Proxy-Agent headers in HTTP snipplets
- to make testing more convenient.
+ decompress_iob(): Add periods to a couple of log messages
</para>
</listitem>
<listitem>
<para>
- Include the error code for pcre errors Privoxy does not recognize.
+ Terminate the body of the HTTP snipplets with a single new line
+ instead of "\r\n".
</para>
</listitem>
<listitem>
<para>
- Config directives with numerical arguments are checked more carefully.
+ configure: Add --with-assertions option and only enable assertions
+ when it is used
</para>
</listitem>
<listitem>
<para>
- Privoxy's malloc() wrapper has been changed to prevent zero-size
- allocations which should only occur as the result of bugs.
+ windows build: Use --with-brotli and --with-mbedtls by default and
+ enable dynamic error checking.
</para>
</listitem>
<listitem>
<para>
- Various cosmetic changes.
+ gif_deanimate(): Confirm we've got an image before trying to write it
+ Saves a pointless buf_copy() call.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ OpenSSL ssl_store_cert(): Remove a superfluous space before the serial number.
</para>
</listitem>
</itemizedlist>
<itemizedlist>
<listitem>
<para>
- Unblock ".deutschlandradiokultur.de/".
- Reported by u302320 in #924.
+ Disable fast-redirects for .golem.de/
</para>
</listitem>
<listitem>
<para>
- Add two fast-redirect exceptions for "yandex.ru".
+ Unblock requests to adri*.
</para>
</listitem>
<listitem>
<para>
- Disable filter{banners-by-size} for ".plasmaservice.de/".
+ Block requests for trc*.taboola.com/
</para>
</listitem>
<listitem>
<para>
- Unblock "klikki.fi/adv/".
+ Disable fast-redirects for .linkedin.com/
</para>
- </listitem>
- <listitem>
- <para>
- Block requests for "resources.infolinks.com/".
- Reported by "Black Rider" on ijbswa-users@.
- </para>
- </listitem>
+ </listitem>
+ </itemizedlist>
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Filter file improvements:
+ <itemizedlist>
<listitem>
<para>
- Block a bunch of criteo domains.
- Reported by Black Rider.
+ Make the second pcrs job of the img-reorder filter greedy again.
+ The ungreedy version broke the img tags on:
+ https://bulk.fefe.de/scalability/.
</para>
</listitem>
+ </itemizedlist>
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Privoxy-Log-Parser:
+ <itemizedlist>
<listitem>
<para>
- Block "abs.proxistore.com/abe/".
- Reported by Black Rider.
+ Highlight a few more messages.
</para>
</listitem>
<listitem>
<para>
- Disable filter{banners-by-size} for ".black-mosquito.org/".
+ Clarify the --statistics output. The shown "Reused connections"
+ are server connections so name them appropriately.
</para>
</listitem>
<listitem>
<para>
- Disable fast-redirects for "disqus.com/".
+ Bump version to 0.9.3.
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- Documentation improvements:
+ Privoxy-Regression-Test:
<itemizedlist>
<listitem>
<para>
- FAQ: Explicitly point fingers at ASUS as an example of a
- company that has been reported to force malware based on
- Privoxy upon its customers.
+ Add the --check-bad-ssl option to the --help output.
</para>
</listitem>
<listitem>
<para>
- Correctly document the action type for a bunch of "multi-value"
- actions that were incorrectly documented to be "parameterized".
- Reported by Gregory Seidman on ijbswa-users@.
- </para>
- </listitem>
- <listitem>
- <para>
- Fixed the documented type of the forward-override{} action
- which is obviously 'parameterized'.
+ Bump version to 0.7.3.
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- Website improvements:
+ Documentation:
<itemizedlist>
<listitem>
<para>
- Users who don't trust binaries served by SourceForge
- can get them from a mirror. Migrating away from SourceForge
- is planned for 2016 (TODO list item #53).
+ Add pushing the created tag to the release steps in the developer manual.
</para>
</listitem>
<listitem>
<para>
- The website is now available as onion service
- (http://jvauzb4sb3bwlsnc.onion/).
+ Clarify that 'debug 32768' should be used in addition to the other debug
+ directives when reporting problems.
</para>
</listitem>
+ <listitem>
+ <para>
+ Add a 'Third-party licenses and copyrights' section to the user manual.
+ </para>
+ </listitem>
</itemizedlist>
</para>
</listitem>