Rebuild docs with updated ChangeLog
[privoxy.git] / doc / webserver / user-manual / whatsnew.html
index 269b847..57c6090 100644 (file)
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
 "http://www.w3.org/TR/html4/loose.dtd">
-
 <html>
 <head>
   <title>What's New in this Release</title>
-  <meta name="GENERATOR" content=
-  "Modular DocBook HTML Stylesheet Version 1.79">
-  <link rel="HOME" title="Privoxy 3.0.23 User Manual" href="index.html">
+  <meta name="GENERATOR" content="Modular DocBook HTML Stylesheet Version 1.79">
+  <link rel="HOME" title="Privoxy 3.0.33 User Manual" href="index.html">
   <link rel="PREVIOUS" title="Installation" href="installation.html">
-  <link rel="NEXT" title="Quickstart to Using Privoxy" href=
-  "quickstart.html">
+  <link rel="NEXT" title="Quickstart to Using Privoxy" href="quickstart.html">
   <link rel="STYLESHEET" type="text/css" href="../p_doc.css">
-  <meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
+  <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
   <link rel="STYLESHEET" type="text/css" href="p_doc.css">
 </head>
-
-<body class="SECT1" bgcolor="#EEEEEE" text="#000000" link="#0000FF" vlink=
-"#840084" alink="#0000FF">
+<body class="SECT1" bgcolor="#EEEEEE" text="#000000" link="#0000FF" vlink="#840084" alink="#0000FF">
   <div class="NAVHEADER">
-    <table summary="Header navigation table" width="100%" border="0"
-    cellpadding="0" cellspacing="0">
+    <table summary="Header navigation table" width="100%" border="0" cellpadding="0" cellspacing="0">
       <tr>
-        <th colspan="3" align="center">Privoxy 3.0.23 User Manual</th>
+        <th colspan="3" align="center">Privoxy 3.0.33 User Manual</th>
       </tr>
-
       <tr>
-        <td width="10%" align="left" valign="bottom"><a href=
-        "installation.html" accesskey="P">Prev</a></td>
-
+        <td width="10%" align="left" valign="bottom"><a href="installation.html" accesskey="P">Prev</a></td>
         <td width="80%" align="center" valign="bottom"></td>
-
-        <td width="10%" align="right" valign="bottom"><a href=
-        "quickstart.html" accesskey="N">Next</a></td>
+        <td width="10%" align="right" valign="bottom"><a href="quickstart.html" accesskey="N">Next</a></td>
       </tr>
     </table>
     <hr align="left" width="100%">
   </div>
-
   <div class="SECT1">
-    <h1 class="SECT1"><a name="WHATSNEW" id="WHATSNEW">3. What's New in this
-    Release</a></h1>
-
-    <p><span class="APPLICATION">Privoxy 3.0.23</span> stable is a bug-fix
-    release, some of the fixed bugs are security issues (CVE requests
-    pending):</p>
-
+    <h1 class="SECT1"><a name="WHATSNEW" id="WHATSNEW">3. What's New in this Release</a></h1>
+    <p><span class="APPLICATION">Privoxy 3.0.33</span> fixes an XSS issue and multiple DoS issues and a couple of other
+    bugs. The issues also affect earlier Privoxy releases. <span class="APPLICATION">Privoxy 3.0.33</span> also comes
+    with a couple of general improvements an new features.</p>
+    <p>Changes in <span class="APPLICATION">Privoxy 3.0.33</span> stable:</p>
     <ul>
+      <li>
+        <p>Security/Reliability:</p>
+        <ul>
+          <li>
+            <p>cgi_error_no_template(): Encode the template name to prevent XSS (cross-side scripting) when Privoxy is
+            configured to servce the user-manual itself. Commit 0e668e9409c. OVE-20211102-0001. CVE-2021-44543.
+            Reported by: Artem Ivanov</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+        </ul>
+      </li>
       <li>
         <p>Bug fixes:</p>
-
         <ul>
           <li>
-            <p>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.</p>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>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.</p>
+            <p>continue_https_chat(): Update csp-&#62;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.</p>
           </li>
-
           <li>
-            <p>Fixed an 'invalid read' bug which could at least theoretically
-            cause Privoxy to crash. So far, no crashes have been
-            observed.</p>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>Compiles with --disable-force again. Reported by Kay
-            Raven.</p>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>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.</p>
+            <p>load_config(): Add a space that was missing in a log message.</p>
+          </li>
+          <li>
+            <p>read_http_request_body(): Fix two error messages that used an incorrect variable.</p>
+          </li>
+          <li>
+            <p>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.</p>
           </li>
         </ul>
       </li>
-
       <li>
         <p>General improvements:</p>
-
         <ul>
           <li>
-            <p>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.</p>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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@.</p>
+          </li>
+          <li>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>The tests are less likely to cause false positives.</p>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>handle_established_connection(): Slightly improve a comment.</p>
+          </li>
+          <li>
+            <p>handle_established_connection(): Fix a comment.</p>
+          </li>
+          <li>
+            <p>socks5_connect(): Fix indentation.</p>
+          </li>
+          <li>
+            <p>handle_established_connection(): Improve an error message.</p>
+          </li>
+          <li>
+            <p>create_pattern_spec(): Fix ifdef indentation.</p>
+          </li>
+          <li>
+            <p>Fix comment typos.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>chat(): Log the applied actions before deciding how to forward the request.</p>
+          </li>
+          <li>
+            <p>parse_time_header(): Silence a coverity complaint when building without assertions.</p>
+          </li>
+          <li>
+            <p>receive_encrypted_request_headers(): Improve a log message.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>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</p>
+          </li>
+          <li>
+            <p>OpenSSL generate_host_certificate(): Fix two error messsages.</p>
+          </li>
+          <li>
+            <p>Improve description of handle_established_connection()</p>
+          </li>
+          <li>
+            <p>OpenSSL ssl_store_cert(): Translate EVP_PKEY_EC to a string.</p>
+          </li>
+          <li>
+            <p>OpenSSL ssl_store_cert(): Remove pointless variable initialization.</p>
+          </li>
+          <li>
+            <p>OpenSSL ssl_store_cert(): Initialize pointer with NULL instead of 0.</p>
           </li>
         </ul>
       </li>
-
       <li>
         <p>Action file improvements:</p>
-
         <ul>
           <li>
-            <p>'.sify.com/' is no longer blocked. Apparently it is not
-            actually a pure tracking site (anymore?). Reported by Andrew on
-            ijbswa-users@.</p>
+            <p>Disable fast-redirects for .microsoftonline.com/.</p>
+          </li>
+          <li>
+            <p>Disable fast-redirects for idp.springer.com/.</p>
+          </li>
+          <li>
+            <p>Disable fast-redirects for .zeit.de/zustimmung.</p>
+          </li>
+          <li>
+            <p>Unblock adv-archiv.dfn-cert.de/.</p>
+          </li>
+          <li>
+            <p>Block requests to eu-tlp01.kameleoon.eu/.</p>
+          </li>
+          <li>
+            <p>Block requests to fpa-events.arstechnica.com/.</p>
+          </li>
+          <li>
+            <p>Unblock nlnet.nl/.</p>
+          </li>
+          <li>
+            <p>Unblock adguard.com/.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>Privoxy-Log-Parser:</p>
+        <ul>
+          <li>
+            <p>Highlight 'Socket timeout 3 reached: http://127.0.0.1:20000/no-filter/chunked-content/36'.</p>
+          </li>
+          <li>
+            <p>Improve documentation for inactivity-detection mode.</p>
+          </li>
+          <li>
+            <p>Detect date changes when looking for inactivity.</p>
+          </li>
+          <li>
+            <p>Add a --passed-request-statistics-threshold option that can be set to get statistics for requests that
+            were passed.</p>
+          </li>
+          <li>
+            <p>Add a "inactivity detection" mode which can be useful for debugging purposes.</p>
+          </li>
+          <li>
+            <p>Bump version to 0.9.4.</p>
+          </li>
+          <li>
+            <p>Only run print_intro() and print_outro() when syntax highlighting.</p>
+          </li>
+          <li>
+            <p>Rephrase a sentence in the documentation.</p>
+          </li>
+          <li>
+            <p>Highlight 'Client socket 7 is no longer usable. The server socket has been closed.'.</p>
+          </li>
+          <li>
+            <p>Clarify --statistics output by explicitly mentioning that the status codes sent by the server may differ
+            from the ones in "debug 512" messages.</p>
+          </li>
+          <li>
+            <p>Fix typo in the --statistics output.</p>
+          </li>
+          <li>
+            <p>Remove an unused variable.</p>
+          </li>
+          <li>
+            <p>Highlight 'The peer notified us that the connection on socket 11 is going to be closed'.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>Privoxy-Regression-Test:</p>
+        <ul>
+          <li>
+            <p>Remove duplicated word in a comment.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>regression-tests.action:</p>
+        <ul>
+          <li>
+            <p>Add fetch test for http://p.p/wpad.dat.</p>
+          </li>
+          <li>
+            <p>Bump for-privoxy-version to 3.0.33 which introduced the wpad.dat support.</p>
+          </li>
+          <li>
+            <p>Add more tests for the '/send-banner' code.</p>
+          </li>
+          <li>
+            <p>Add test for OVE-20210203-0001.</p>
           </li>
-
           <li>
-            <p>Unblock banners on .amnesty.de/ which aren't ads.</p>
+            <p>Add a test for CVE-2021-20217.</p>
           </li>
         </ul>
       </li>
-
       <li>
-        <p>Documentation improvements:</p>
-
+        <p>uagen:</p>
         <ul>
           <li>
-            <p>The 'Would you like to donate?' section now also contains a
-            "Paypal" address.</p>
+            <p>Bump generated Firefox version to 91 (ESR).</p>
+          </li>
+          <li>
+            <p>Bump version to 1.2.3.</p>
+          </li>
+          <li>
+            <p>Bump copyright.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>Build system:</p>
+        <ul>
+          <li>
+            <p>configure: Bump SOURCE_DATE_EPOCH.</p>
+          </li>
+          <li>
+            <p>GNUmakefile.in: Fix typo.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>Add configure option to enable MemorySanitizer.</p>
+          </li>
+          <li>
+            <p>Add configure option to enable UndefinedBehaviorSanitizer.</p>
+          </li>
+          <li>
+            <p>Add configure option to enable AddressSanitizer.</p>
+          </li>
+          <li>
+            <p>Bump copyright.</p>
+          </li>
+          <li>
+            <p>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.</p>
+          </li>
+          <li>
+            <p>configure: Remove obsolete RPM_BASE check.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>Windows build system:</p>
+        <ul>
+          <li>
+            <p>Update the build script to use mbed tls version 2.6.11.</p>
+          </li>
+          <li>
+            <p>Update build script to use the final 8.45 pcre library.</p>
+          </li>
+          <li>
+            <p>Put all the '--enable-xxx' options in the configure call together.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>macOS build system:</p>
+        <ul>
+          <li>
+            <p>The OSXPackageBuilder repository has been updated and can be used to create macOS packages again.</p>
+          </li>
+        </ul>
+      </li>
+      <li>
+        <p>Documentation:</p>
+        <ul>
+          <li>
+            <p>contacting: Remove obsolete reference to announce.sgml.</p>
+          </li>
+          <li>
+            <p>contacting: Request that the browser cache is cleared before producing a log file for submission.</p>
+          </li>
+          <li>
+            <p>Sponsor FAQ: Note that Privoxy users may follow sponsor links without Referer header set.</p>
+          </li>
+          <li>
+            <p>newfeatures: Clarify that https inspection also allows to filter https responses.</p>
+          </li>
+          <li>
+            <p>developer-manual: Mention that announce.txt should be updated when doing a release.</p>
+          </li>
+          <li>
+            <p>config: Explicitly mention that the CGI pages disclosing the ca-password can be blocked and upgrade the
+            disclosure paragraphs to a warning.</p>
+          </li>
+          <li>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>The list of supported operating systems has been updated.</p>
+            <p>Update TODO list item #184 to note that WolfSSL support will (hopefully) appear after the 3.0.34
+            release.</p>
           </li>
-
           <li>
-            <p>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.</p>
+            <p>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.</p>
           </li>
-
           <li>
-            <p>The claim that default.action updates are sometimes released
-            on their own has been removed. It hasn't happened in years.</p>
+            <p>Add a warning that the socket-timeout does not apply to operations done by TLS libraries.</p>
           </li>
-
           <li>
-            <p>Explicitly mention that Tor's port may deviate from the
-            default when using a bundle. Requested by Andrew on
-            ijbswa-users@.</p>
+            <p>Make documentation slightly less "offensive" for some people by avoiding the word "hell".</p>
           </li>
         </ul>
       </li>
     </ul>
-
     <div class="SECT2">
-      <h2 class="SECT2"><a name="UPGRADERSNOTE" id="UPGRADERSNOTE">3.1. Note
-      to Upgraders</a></h2>
-
-      <p>A quick list of things to be aware of before upgrading from earlier
-      versions of <span class="APPLICATION">Privoxy</span>:</p>
-
+      <h2 class="SECT2"><a name="UPGRADERSNOTE" id="UPGRADERSNOTE">3.1. Note to Upgraders</a></h2>
+      <p>A quick list of things to be aware of before upgrading from earlier versions of <span class=
+      "APPLICATION">Privoxy</span>:</p>
       <ul>
         <li>
-          <p>The recommended way to upgrade <span class=
-          "APPLICATION">Privoxy</span> is to backup your old configuration
-          files, install the new ones, verify that <span class=
-          "APPLICATION">Privoxy</span> is working correctly and finally merge
-          back your changes using <span class="APPLICATION">diff</span> and
-          maybe <span class="APPLICATION">patch</span>.</p>
-
-          <p>There are a number of new features in each <span class=
-          "APPLICATION">Privoxy</span> release and most of them have to be
-          explicitly enabled in the configuration files. Old configuration
-          files obviously don't do that and due to syntax changes using old
-          configuration files with a new <span class=
+          <p>The recommended way to upgrade <span class="APPLICATION">Privoxy</span> is to backup your old
+          configuration files, install the new ones, verify that <span class="APPLICATION">Privoxy</span> is working
+          correctly and finally merge back your changes using <span class="APPLICATION">diff</span> and maybe
+          <span class="APPLICATION">patch</span>.</p>
+          <p>There are a number of new features in each <span class="APPLICATION">Privoxy</span> release and most of
+          them have to be explicitly enabled in the configuration files. Old configuration files obviously don't do
+          that and due to syntax changes using old configuration files with a new <span class=
           "APPLICATION">Privoxy</span> isn't always possible anyway.</p>
         </li>
-
         <li>
-          <p>Note that some installers remove earlier versions completely,
-          including configuration files, therefore you should really save any
-          important configuration files!</p>
+          <p>Note that some installers remove earlier versions completely, including configuration files, therefore you
+          should really save any important configuration files!</p>
         </li>
-
         <li>
-          <p>On the other hand, other installers don't overwrite existing
-          configuration files, thinking you will want to do that
-          yourself.</p>
+          <p>On the other hand, other installers don't overwrite existing configuration files, thinking you will want
+          to do that yourself.</p>
         </li>
-
         <li>
-          <p>In the default configuration only fatal errors are logged now.
-          You can change that in the <a href="config.html#DEBUG">debug
-          section</a> of the configuration file. You may also want to enable
-          more verbose logging until you verified that the new <span class=
-          "APPLICATION">Privoxy</span> version is working as expected.</p>
+          <p>In the default configuration only fatal errors are logged now. You can change that in the <a href=
+          "config.html#DEBUG">debug section</a> of the configuration file. You may also want to enable more verbose
+          logging until you verified that the new <span class="APPLICATION">Privoxy</span> version is working as
+          expected.</p>
         </li>
-
         <li>
-          <p>Three other config file settings are now off by default:
-          <a href="config.html#ENABLE-REMOTE-TOGGLE">enable-remote-toggle</a>,
-          <a href=
-          "config.html#ENABLE-REMOTE-HTTP-TOGGLE">enable-remote-http-toggle</a>,
-          and <a href=
-          "config.html#ENABLE-EDIT-ACTIONS">enable-edit-actions</a>. If you
-          use or want these, you will need to explicitly enable them, and be
-          aware of the security issues involved.</p>
+          <p>Three other config file settings are now off by default: <a href=
+          "config.html#ENABLE-REMOTE-TOGGLE">enable-remote-toggle</a>, <a href=
+          "config.html#ENABLE-REMOTE-HTTP-TOGGLE">enable-remote-http-toggle</a>, and <a href=
+          "config.html#ENABLE-EDIT-ACTIONS">enable-edit-actions</a>. If you use or want these, you will need to
+          explicitly enable them, and be aware of the security issues involved.</p>
         </li>
       </ul>
     </div>
   </div>
-
   <div class="NAVFOOTER">
     <hr align="left" width="100%">
-
-    <table summary="Footer navigation table" width="100%" border="0"
-    cellpadding="0" cellspacing="0">
+    <table summary="Footer navigation table" width="100%" border="0" cellpadding="0" cellspacing="0">
       <tr>
-        <td width="33%" align="left" valign="top"><a href="installation.html"
-        accesskey="P">Prev</a></td>
-
-        <td width="34%" align="center" valign="top"><a href="index.html"
-        accesskey="H">Home</a></td>
-
-        <td width="33%" align="right" valign="top"><a href="quickstart.html"
-        accesskey="N">Next</a></td>
+        <td width="33%" align="left" valign="top"><a href="installation.html" accesskey="P">Prev</a></td>
+        <td width="34%" align="center" valign="top"><a href="index.html" accesskey="H">Home</a></td>
+        <td width="33%" align="right" valign="top"><a href="quickstart.html" accesskey="N">Next</a></td>
       </tr>
-
       <tr>
         <td width="33%" align="left" valign="top">Installation</td>
-
         <td width="34%" align="center" valign="top">&nbsp;</td>
-
-        <td width="33%" align="right" valign="top">Quickstart to Using
-        Privoxy</td>
+        <td width="33%" align="right" valign="top">Quickstart to Using Privoxy</td>
       </tr>
     </table>
   </div>