X-Git-Url: http://www.privoxy.org/gitweb/?a=blobdiff_plain;f=doc%2Fsource%2Fchangelog.sgml;h=95f7cfaba880ba1516b1c83032f3ead97243b457;hb=4697cf05fcd27c0c559477f66d82784ebcb60b42;hp=047517c203159c34caeaa55c6c9704851222ae38;hpb=1ced0af6010e7bec3b2594046a6b9b27f7b0559e;p=privoxy.git diff --git a/doc/source/changelog.sgml b/doc/source/changelog.sgml index 047517c2..95f7cfab 100644 --- a/doc/source/changelog.sgml +++ b/doc/source/changelog.sgml @@ -1,11 +1,9 @@ - - Privoxy 3.0.23 stable is a bug-fix release, - some of the fixed bugs are security issues (CVE requests pending): - - + + Privoxy 3.0.33 fixes an XSS issue + and multiple DoS issues and a couple of other bugs. + The issues also affect earlier Privoxy releases. + Privoxy 3.0.33 also comes with + a couple of general improvements and new features. + + + Changes in Privoxy 3.0.33 stable: + + + + Security/Reliability: + + + + cgi_error_no_template(): Encode the template name to prevent + XSS (cross-site scripting) when Privoxy is configured to servce + the user-manual itself. + Commit 0e668e9409c. OVE-20211102-0001. CVE-2021-44543. + Reported by: Artem Ivanov + + + + + get_url_spec_param(): Free memory of compiled pattern spec + before bailing. + Reported by Joshua Rogers (Opera) who also provided the fix. + Commit 652b4b7cb0. OVE-20211201-0003. CVE-2021-44540. + + + + + process_encrypted_request_headers(): Free header memory when + failing to get the request destination. + Reported by Joshua Rogers (Opera) who also provided the fix. + Commit 0509c58045. OVE-20211201-0002. CVE-2021-44541. + + + + + send_http_request(): Prevent memory leaks when handling errors + Reported by Joshua Rogers (Opera) who also provided the fix. + Commit c48d1d6d08. OVE-20211201-0001. CVE-2021-44542. + + + + + Bug fixes: - Fixed a DoS issue in case of client requests with incorrect - chunk-encoded body. When compiled with assertions enabled - (the default) they could previously cause Privoxy to abort(). - Reported by Matthew Daley. + handle_established_connection(): Skip the poll()/select() calls + if TLS data is pending on the server socket. The TLS library may + have already consumed all the data from the server response in + which case poll() and select() will not detect that data is + available to be read. + Fixes SF bug #926 reported by Wen Yue. + + + + + continue_https_chat(): Update csp->server_connection.request_sent + after sending the request to make sure the latency is calculated + correctly. Previously https connections were not reused after + timeout seconds after the first request made on the connection. - Fixed multiple segmentation faults and memory leaks in the - pcrs code. This fix also increases the chances that an invalid - pcrs command is rejected as such. Previously some invalid commands - would be loaded without error. Note that Privoxy's pcrs sources - (action and filter files) are considered trustworthy input and - should not be writable by untrusted third-parties. + free_pattern_spec(): Don't try to free an invalid pointer + when unloading an action file with a TAG pattern while + Privoxy has been compiled without FEATURE_PCRE_HOST_PATTERNS. + Closes: SF patch request #147. Patch by Maxim Antonov. - Fixed an 'invalid read' bug which could at least theoretically - cause Privoxy to crash. So far, no crashes have been observed. + Adjust build_request_line() to create a CONNECT request line when + https-inspecting and forwarding to a HTTP proxy. + Fixes SF bug #925 reported by Wen Yue. - Compiles with --disable-force again. Reported by Kai Raven. + load_config(): Add a space that was missing in a log message. - Client requests with body that can't be delivered no longer - cause pipelined requests behind them to be rejected as invalid. - Reported by Basil Hussain. + read_http_request_body(): Fix two error messages that used an + incorrect variable. + + + + + If the the response is chunk-encoded, ignore the Content-Length + header sent by the server. + Allows to load https://redmine.lighttpd.net/ with filtering enabled. @@ -81,14 +138,175 @@ - If a pcrs command is rejected as invalid, Privoxy now logs - the cause of the problem as text. Previously the pcrs error - code was logged. + Allow to edit the add-header action through the CGI editor by + generalizing the code that got added with the suppress-tag action. + Closes SF patch request #146. Patch by Maxim Antonov. + + + + + Add a CGI handler for /wpad.dat that returns a + Proxy Auto-Configuration (PAC) file. + Among other things, it can be used to instruct clients + through DHCP to use Privoxy as proxy. + For example with the dnsmasq option: + dhcp-option=252,http://config.privoxy.org/wpad.dat + Initial patch by Richard Schneidt. + + + + + Don't log the applied actions in process_encrypted_request() + Log them in continue_https_chat() instead to mirror chat(). + Prevents the applied actions from getting logged twice + for the first request on an https-inspected connection. + + + + + OpenSSL generate_host_certificate(): Use config.privoxy.org as Common Name + Org and Org Unit if the real host name is too long to get accepted by OpenSSL. + Clients should only care about the Subject Alternative Name + anyway and we can continue to use the real host name for it. + Reported by Miles Wen on privoxy-users@. + + + + + Establish the TLS connection with the client earlier and decide + how to route the request afterwards. This allows to change the + forwarding settings based on information from the https-inspected + request, for example the path. + + + + + listen_loop(): When shutting down gracefully, close listening ports + before waiting for the threads to exit. Allows to start a second + Privoxy with the same config file while the first Privoxy is still + running. + + + + + serve(): Close the client socket as well if the server socket + for an inspected connection has been closed. Privoxy currently + can't establish a new server connection when the client socket + is reused and would drop the connection in continue_https_chat() + anyway. + + + + + Don't disable redirect checkers in redirect_url(). + Disable them in handle_established_connection() instead. + Doing it in redirect_url() prevented the +redirect{} and + +fast-redirects{} actions from being logged with LOG_LEVEL_ACTIONS. + + + + + handle_established_connection(): Slightly improve a comment. + + + + + handle_established_connection(): Fix a comment. + + + + + socks5_connect(): Fix indentation. + + + + + handle_established_connection(): Improve an error message. + + + + + create_pattern_spec(): Fix ifdef indentation. + + + + + Fix comment typos. + + + + + process_encrypted_request(): Improve a log message. + The function only processes request headers and there + may still be unread request body data left to process. + + + + + chat(): Log the applied actions before deciding how to forward the request. + + + + + parse_time_header(): Silence a coverity complaint when building without assertions. + + + + + receive_encrypted_request_headers(): Improve a log message. + + + + + mbedTLS get_ciphersuites_from_string(): Use strlcpy() instead of strncpy(). + Previously the terminating NUL wasn't copied which resulted + in a compiler warning. This didn't cause actual problems as + the target buffer was initialized by zalloc_or_die() so the + last byte of the target buffer was NUL already. + Actually copying the terminating NUL seems clearer, though. + + + + + Remove compiler warnings. "log_error(LOG_LEVEL_FATAL, ..." + doesn't return but apparently the compiler doesn't know that. + Get rid of several "this statement may fall through + [-Wimplicit-fallthrough=]" warnings. - The tests are less likely to cause false positives. + Store the PEM certificate in a dynamically allocated buffer + when https-inspecting. Should prevent errors like: + 2021-03-16 22:36:19.148 7f47bbfff700 Error: X509 PEM cert len 16694 is larger than buffer len 16383 + As a bonus it should slightly reduce the memory usage as most + certificates are smaller than the previously used fixed buffer. + Reported by: Wen Yue + + + + + OpenSSL generate_host_certificate(): Fix two error messsages. + + + + + Improve description of handle_established_connection() + + + + + OpenSSL ssl_store_cert(): Translate EVP_PKEY_EC to a string. + + + + + OpenSSL ssl_store_cert(): Remove pointless variable initialization. + + + + + OpenSSL ssl_store_cert(): Initialize pointer with NULL instead of 0. @@ -100,13 +318,244 @@ - '.sify.com/' is no longer blocked. Apparently it is not actually - a pure tracking site (anymore?). Reported by Andrew on ijbswa-users@. + Disable fast-redirects for .microsoftonline.com/. + + + + + Disable fast-redirects for idp.springer.com/. + + + + + Disable fast-redirects for .zeit.de/zustimmung. + + + + + Unblock adv-archiv.dfn-cert.de/. + + + + + Block requests to eu-tlp01.kameleoon.eu/. + + + + + Block requests to fpa-events.arstechnica.com/. + + + + + Unblock nlnet.nl/. + + + + + Unblock adguard.com/. + + + + + + + + Privoxy-Log-Parser: + + + + Highlight 'Socket timeout 3 reached: http://127.0.0.1:20000/no-filter/chunked-content/36'. + + + + + Improve documentation for inactivity-detection mode. + + + + + Detect date changes when looking for inactivity. + + + + + Add a --passed-request-statistics-threshold option + that can be set to get statistics for requests that + were passed. + + + + + Add a "inactivity detection" mode which can be useful + for debugging purposes. + + + + + Bump version to 0.9.4. + + + + + Only run print_intro() and print_outro() when syntax highlighting. + + + + + Rephrase a sentence in the documentation. + + + + + Highlight 'Client socket 7 is no longer usable. The server socket has been closed.'. + + + + + Clarify --statistics output by explicitly mentioning that + the status codes sent by the server may differ from the ones + in "debug 512" messages. + + + + + Fix typo in the --statistics output. + + + + + Remove an unused variable. + + + + + Highlight 'The peer notified us that the connection on socket 11 is going to be closed'. + + + + + + + + Privoxy-Regression-Test: + + + + Remove duplicated word in a comment. + + + + + + + + regression-tests.action: + + + + Add fetch test for http://p.p/wpad.dat. + + + + + Bump for-privoxy-version to 3.0.33 which introduced the wpad.dat support. + + + + + Add more tests for the '/send-banner' code. + + + + + Add test for OVE-20210203-0001. + + + + + Add a test for CVE-2021-20217. + + + + + + + + uagen: + + + + Bump generated Firefox version to 91 (ESR). + + + + + Bump version to 1.2.3. + + + + + Bump copyright. + + + + + + + + Build system: + + + + configure: Bump SOURCE_DATE_EPOCH. + + + + + GNUmakefile.in: Fix typo. + + + + + configure: Add another warning in case --disable-pthread + is used while POSIX threads are available. + Various features don't even compile when not using threads. + + + + + Add configure option to enable MemorySanitizer. - Unblock banners on .amnesty.de/ which aren't ads. + Add configure option to enable UndefinedBehaviorSanitizer. + + + + + Add configure option to enable AddressSanitizer. + + + + + Bump copyright. + + + + + Add a configure option to disable pcre JIT compilation. + While JIT compilation makes filtering faster it can + cause false-positive valgrind complaints. + As reported by Gwyn Ciesla in SF bug 924 it also can + cause problems when the SELinux policy does not grant + Privoxy "execmem" privileges. + + + + + configure: Remove obsolete RPM_BASE check. @@ -114,36 +563,112 @@ - Documentation improvements: + Windows build system: - The 'Would you like to donate?' section now also contains - a "Paypal" address. + Update the build script to use mbed tls version 2.6.11. + + + + + Update build script to use the final 8.45 pcre library. + + + + + Put all the '--enable-xxx' options in the configure call together. + + + + + + + + macOS build system: + + + + The OSXPackageBuilder repository has been updated and + can be used to create macOS packages again. + + + + + + + + Documentation: + + + + contacting: Remove obsolete reference to announce.sgml. + + + + + contacting: Request that the browser cache is cleared before + producing a log file for submission. + + + + + Sponsor FAQ: Note that Privoxy users may follow sponsor links + without Referer header set. + + + + + newfeatures: Clarify that https inspection also allows to + filter https responses. + + + + + developer-manual: Mention that announce.txt should be updated + when doing a release. + + + + + config: Explicitly mention that the CGI pages disclosing the + ca-password can be blocked and upgrade the disclosure paragraphs + to a warning. + + + + + Put all the requested debug options in the config file. + Section 11.1 of the Privoxy user manual lists all the debug + options that should be enabled when reporting problems or requesting support. + Make it easier for users to do the right thing by having all those + options present in the config. - The list of supported operating systems has been updated. + Update TODO list item #184 to note that WolfSSL support will + (hopefully) appear after the 3.0.34 release. - The existence of the SF support and feature trackers has been - deemphasized because they have been broken for months. - Most of the time the mailing lists still work. + Update max-client-connections's description. + On modern systems other than Windows Privoxy should + use poll() in which case the FD_SETSIZE value isn't + releveant. - The claim that default.action updates are sometimes released - on their own has been removed. It hasn't happened in years. + Add a warning that the socket-timeout does not apply + to operations done by TLS libraries. - Explicitly mention that Tor's port may deviate from the default - when using a bundle. Requested by Andrew on ijbswa-users@. + Make documentation slightly less "offensive" for some people + by avoiding the word "hell".