WinMain: Use the correct function to close the event handle

According to Microsoft's documentation DeleteObject() is
used to delete a "logical pen, brush, font, bitmap, region,
or palette" while CloseHandle() is supposed to be used
for the handle returned by CreateEvent():
https://msdn.microsoft.com/en-us/library/windows/desktop/dd183539%28v=vs.85%29.aspx
https://msdn.microsoft.com/en-us/library/windows/desktop/ms682396%28v=vs.85%29.aspx
https://msdn.microsoft.com/en-us/library/windows/desktop/ms724211%28v=vs.85%29.aspx

It's conceivable that this commit fixes a tiny memory leak,
but then again maybe not as DeleteObject() is apparently full
of magic:
http://blogs.msdn.com/b/oldnewthing/archive/2013/03/06/10399678.aspx

Reported by Jarry Xu in #891.

FAQ: Fix spelling of affect

#132: Note some work in progress

load_config(): In case of invalid forward-socks5(t) directives, use the correct directive name

Previously the error messages referred to forward-socks4t failures.

Reported by Joel Verhagen in #889.

Add #141: Port Privoxy to CloudABI

Unblock klikki.fi/adv/

translate_socks5_error(): Improve SOCKS5_REQUEST_HOST_UNREACHABLE translation

... to make it more obvious that it's the destination
host that is unreachable and not the host running the
socks server.

malloc_or_die(): Catch and prevent attempted zero-size allocations

execute_external_filter(): Don't rely on undefined malloc() behaviour

... and fix the read buffer scaling for initial sizes below READ_LENGTH.

Could fix the crash reported by Jonathan McKenzie on ijbswa-users@

docbook2man-spec.pl: Update the inserted man page banner

.. to make it more obvious that the script is part of Privoxy's source tarball.

Remove suggestion that bug reports should be reported to the original author
as the referenced address is no longer useful thanks to a domain squatter.

Disable filter{banners-by-size} for .plasmaservice.de/

get_actions(): Detect and reject parameters for parameter-less actions

Previously they were silently ignored.

Increase socks5_connect()'s optimism

... and let it send the request body optimistically as well.

It's not that complicated and, more importantly, previously
the request body wasn't guaranteed to be sent at all.

Should fix #1686 reported by Peter Müller and G4JC.

Regenerated docs with CVEs for 3.0.23 and bumped copyright

Bump copyright

Add CVEs for 3.0.23 stable

Bump copyright

Add CVEs for Privoxy 3.0.23

listen_loop(): Add number of active threads to a couple of log messages

decompress_iob(): Refine the log message emitted when the iob is too small

Unblock .deutschlandradiokultur.de/

Reported by u302320 in #924.

Add two fast-redirect exceptions for yandex.ru

Import last-minute changes

Fix contributor name

Regenerate docs for Privoxy 3.0.23 stable

Regenerate config file for Privoxy 3.0.23

Import changelog for Privoxy 3.0.23

Add CVEs for Privoxy 3.0.22 stable

Add changes for 3.0.23 stable

Declare 3.0.23 'stable'

Add Basil Hussain as contributor

Prevent parse errors after failing to deliver a client request with body

For now we err on the safe side and simply throw all the following
requests under the bus, even if no client body has been buffered.
A compliant client will repeat the dropped requests on an untainted
connection.

The proper fix is to discard the no longer needed client body
in the buffer (if there is one) and to continue parsing the
bytes that follow. This is less trivial and will have to wait
until the next release.

Reported by Basil Hussain.

Add Matthew Daley as contributor

Emphasize that Privoxy is serious business

Add parse_time_header(), a wrapper around parse_header_time()

... which skips the header name for the callers which means
they can't get it wrong. Previously two callers did. This could
result in 'invalid read of size X' issues in case of value-less
headers.

On the systems I tested, X was always 1 and nobody but valgrind
cared, but with different malloc() implementations Privoxy might
be less lucky and segfault.

Partially discovered with afl-fuzz.

chunked_body_is_complete(): Check input more carefully

Previously a nul-chunk without mandatory trailing "\r\n" would
not be rejected as invalid. When compiled with assertions enabled,
this would cause Privoxy to abort().

Reported by Matthew Daley.

pcrs_compile_replacement(): Fix multiple segfaults and memory leaks

... and report errors more reliably. Previously some invalid
pcrs commands were silently accepted but didn't work as expected.

Partially discovered with afl-fuzz.

Add ElectroBSD to the list of BSDs Privoxy is known to run on

Fix comment typo

Properly explain why the pcrs job couldn't be compiled

Grammar fixes for the dok-tidy description

Add a web-user-manual target that uses rsync

Remove #84 as it's a duplicate of #51

Rephrase #51 to make it more obvious.

Fix spelling in #126

Regenerate documentation

Add the Paypal address

Remove CVS $Log

Unblock .amnesty.de/

Ditch previously missed 'Bookmarklets' section in the templates

Deemphasize the existence of the SF support trackers which have been broken for months

Stop pretending that default.action updates are made available on their own

Remove the 'Request New Features' section

The tracker has been broken for months and the TODO list is long
enough anyway.

Add +limit-cookie-lifetime{} to the list of avilable actions

Fix comment typos

Explicitly mention that Tor's port may deviate from the default

... when using a bundle.

Reported by Andrew on ijbswa-users@.

Stop blocking .sify.com/

It's not actually a pure tracking site (anymore?).

Reported by Andrew on ijbswa-users@.

Bump version to 3.0.23 UNRELEASED

Add Kai Raven to the list of contributors

Update the announcement for 3.0.22

Add a bunch of missing test levels

Prevents test failures when the action file editor has
been disabled or isn't available.

Add FEATURE_CGI_EDIT_ACTIONS dependency for test level 12

Rebuild docs

Note that --disable-force is known to be broken

Skip test level 13 unless the keep-alive-timeout is above 20

The tests are expected to fail with lower values, for example
the 5 seconds specified in the default config file.

Reported by Lee.

Make the FORCE_PREFIX define always visible

Fixes a compile error when configured with --disable-force in
which case the define is still required for the show-status page.

Reported by Kai Raven.

Fix a ChangeLog entry

Revert r1.54 "Add range-related tests for external-filter{}"

When configured with the defaults, Privoxy does not accept
the external-filter{} action. Privoxy should probably be
changed to silently ignore the action, but this will have
to wait until the next release.

Rebuild HTML docs

Rebuild text docs

Rebuild config file for 3.0.22 stable

Update SGML ChangeLog

Prevent leading and trailing whitespace in quotes

ChangeLog cosmetic

Declare 3.0.22 'stable'

ChangeLog cosmetic

Update ChangeLog to reflect recent changes

Fix memory leak in rfc2553_connect_to()

CID 66382.

pcrs_execute(): Consistently set *result to NULL in case of errors

Should make use-after-free in the caller less likely.

CID 66391, CID 66376.

unmap(): Prevent use-after-free if the map only consists of one item

CID 66394.

Make sure csp flags have unique values

Previously CSP_FLAG_CRUNCHED and CSP_FLAG_CLIENT_CONNECTION_HEADER_SET
shared the same bit which obviously wasn't intentional.

The only negative effect was that Privoxy potentially reused less
connections than it could have, which explains why the problem hasn't
been noticed for seven years and was only found by manual code
inspection.

Link to the 'Would you like to donate?' FAQ

... instead of repeating parts of it while leaving out the rest.

Let server_keep_alive() always remove the Keep-Alive header

Not forwarding the header to the client is a MUST and
non-compliance has no advantage here.

Fixes Co-Advisor test case rfc2616/hopHdr-Keep-Alive-toClt.

Update URL of Polipo's website

Disable banners-by-size for kggp.de/

Preserve negative HTTP status numbers

While they are technically invalid, "supporting" them doesn't
hurt and this gets the recently added curl test 1430 "working".

Add two consenting donors to the list of contributors

Add changes for 3.0.22 stable

Add a couple of contributors

Regenerate docs

Add #138: Bring back the scripts to provide actions file feedback

Bump copyright

Fix web-actions target

Garbage-collect bookmarklets section

Most of the bookmarklets can't possible work due to not using a
trusted referer header and the rest of them can be replaced with
common bookmarks.

Formally declare the scripts in doc/webserver/actions/ out of action

They haven't worked in years and the docs no longer reference them.

Add a web-faq target that only updates the FAQ on the webserver

Remove already-commented-out non-portable DOSFILTER alternatives

Remove the obsolete targets dok-put and dok-get

Add a sf-shell target

Regenerate FAQ

Mention more stuff donations may be used for