cygwin: make it possible to build DLLs

Patch from Brian Koehmstedt

Merge branch '20_select_nfds' into patches-2.0

Fix a fencepost bug in the select backend

This bug would sometimes lead us to looking one bit off the end of
the fdset arrays, and trying to activate a (nonexistent) event if
that bit was set.

Found by Harlann Stenn.  Fixes a test failure on OpenSolaris.

Make the new http_connect_fail_test use a multicast addr, not broadcast

Turns out that FreeBSD does _not_ give a ENETUNREACH error when
told to make a TCP socket to 255.255.255.255, but it is quite happy
to do so for 239.10.20.30.  So that's what we'll do.

Found by Robert Ransom and Dave Hart.

InitializeCriticalSectionAndSpinCount requires _WIN32_WINNT >= 0x0403.

Fix a couple of signed/unsigned warnings in http.c

Disable main/many_events_slow_add with evport backend

In 2.0 and earlier, evport only reports up to 8 events at a time, which
confuses this test badly.

Fix compilation.

Merge branch 'intcmp_overflow_fixes' into patches-2.0

updated EV_S(s)IZE definitions

Added overflow checks in evhttp_read_body and evhttp_get_body

Added several checks for under/overflow conditions in evhttp_handle_chunked_read

Always use evutil_snprintf, even if OS provides it

In test/tinytest_local.h we can't redefine snprintf if the OS has it
defined already.

Use evutil_gettimeofday instead of relying on the system gettimeofday.

Fix winsock2.h #include issues with MSVC

Define WIN32_LEAN_AND_MEAN, so that windows.h does not bring in
winsock.h which in turn makes it impossible to #include <winsock2.h>
(at least with MSVC)

Make sure TINYTEST_LOCAL is defined when building tinytest.c

unistd.h and sys/time.h might not exist.

MSVC does not provide S_ISDIR, so provide it manually.

Bring in the compile script from automake, if needed.

Make the tests build when OpenSSL is not available.

Don't #define HAVE_OPENSSL (to zero) when OpenSSL is not available.
Code written as #ifdef HAVE_OPENSSL do not expect that.

Link with -lshell32 and -ladvapi32 on Win32.

SHGetSpecialFolderPath is in Shell32.dll and the RegOpenKey (et al) and
CryptGenRandom (et al) functions are in -ladvapi32.dll. MinGW is "nice"
and brings those in automatically, but specify them explicitly for
other tool chains.

Fix new warnings from GCC 4.6

Prevent size_t overflow in evhttp_htmlescape.

Modified the `html_replace' function so that it returns the length of
the replacement string instead of the string itself. This is used to
easily check for overflows of the `new_size' variable in the first for
loop of the `evhttp_htmlescape' function, and thus potential out of
bounds writes in the second for loop (if an overflow occurs in
new_size, then new_size < old_size). Also check that new_size + 1
doesn't overflow in mm_malloc(new_size + 1).

Removed the `scratch_space' variable from the `evhttp_htmlescape'
function since it wasn't actually used; also removed the `buf'
variable from the `evhttp_htmlescape' function since it was only used
by `scratch_space'.

Fix a bug that prevented us from configuring IPv6 nameservers.

Test case for 0848814ac49616

 "I'm not sure if you'll like my use of the limited broadcast address
  for simulating an ENETUNREACH error with a TCP connection, but it's
  the best that I could think of.  Basically, we want to trigger a
  non-EINPROGRESS error in evutil_socket_connect() immediately at the
  connect() in order to bring about the assertion in the
  evhttp_connection_fail() error handling code."

Fix failing assertion introducd in commit 0d6622e

Patch in question:
 - Fix the case when failed evhttp_make_request() leaved request in the queue.
 - http://levent.git.sourceforge.net/git/gitweb.cgi?p=levent/libevent;a=commit;h=0d6622e

The above patch introduces a failing assertion in
evhttp_connection_fail().  This happens because the patch defers the
assignment of the outstanding request to the evcon->requests list,
while evhttp_connection_fail() assumes that the request lies in the
list.

One scenario in which this can happen is when the request list is
empty and a connection is made to an unreachable host.  The assertion
will then fail after bufferevent_socket_connect() errors out (with
ENETUNREACH in my case).

Fix compilation with GCC 2, which had no __builtin_expect

Fix a compile warning with zlib 1.2.4 and 1.2.5

I took this fix from Tor (commit 1a52e39c22d5, author Nick Mathewson,
Copyright (c) 2007-2011, The Tor Project, Inc.) and adapted it slightly
for libevent.

Correctly detect openssl on windows

Move the win32 detection in configure.in

Use OPENSSL_LIBS in Makefile.am

Libtool brings in the dependencies of libevent_openssl.la automatically

libssl depends on libcrypto, not the other way around.

Oops; that new unit test breaks for epoll.  Fix it.

Fix a warn-and-fail bug in kqueue by providing kevent() room to report errors

Apparently, kevent fails gracefully if there is not enough space in its
output events array to report every _event_... but it just dies and returns
-1 if there is not enough space to report every _error_.

There are a couple of possible fixes here.  One would to handle -1
returns from kevent better by re-growing the array and retrying... but
that seems a little error prone.  Instead, I'm just going to say that
the events array must be large enough to handle all the errors.

This patch also adds a unit test designed to make sure that our
many-events-out code works even if not all the events are added at
once.

Add missing words to EVLOOP_NONBLOCK documentation

Bump version to 2.0.11-stable-dev

Increment version number to 2.0.11-stable

Add more people to the acknowledgments

Changelog for 2.0.11-stable

Fix clang warning when resetting connection

This was a regression on 2.0.10-stable: clang was warning about
values that were unused (because event_debug wasn't using them unless
USE_DEBUG was defined).  Found by Sebastian Hahn.

Use AM_CONDITIONAL, not AC_SUBST, to tell automake to build regress

For some reason, this makes mingw much happier about building
regress.exe by default.  No, I'm afraid I don't know why.

Fix msvc build: it needed to know about _EVENT_HAVE_STRUCT_SOCKADDR_STORAGE

Try to fix up CHECK_REGRESS hack to not break on win32

Add missing header for regress_http compilation on win32

Merge branch '20_iocp_conn_sockopt' into patches-2.0

call setsockopt correctly on connecting() IOCP-bufferevents so shutdown() will work

Zero a struct sockaddr_in before using it

Found by Dave Heart

Remote some dead code from dns-example.c

Found by Dave Heart

Check return value of evbuffer_add_cb in tests

Caught by clang's static analyzer

Be nice and "handle" error return values in sample code

Caught by clang's static analyzer

Refactor test_getaddrinfo_async_cancel_stress()

Turns out it didn't need an event base set up.

Spotted by clang's static anaylzer

add some (void) casts for unused variables

All these places already have an XXX marker, so we know we need to fix
them later. Let's not clutter the static analysis results with them.

Actually send NULL request in http_bad_request_test

Issue caught by clang's static analyzer

Add a forgotten return value check in the unit tests

Spotted by clang's static analyzer

Add an assert to appease clang's static analyzer

It got confused because in EVLOCK_ASSERT_LOCKED(lock) there is an if
(lock) check.

Fix possible NULL-deref in evdns_cancel_request

The clang static analyzer complained that base might be dereferenced
without being set, but this patch should fix another dereference issue.

Add a forgotten NULL check to evhttp_parse_headers

Issue detected by the clang static analyzer

Fix compilation under LLVM/clang with --enable-gcc-warnings

When compiling using clang (2.9 or lower) do not enable
-Wnormalized=id or -Woverride-init when --enable-gcc-warnings
or --enable-gcc-warnings-advisory is set as these options
are unsupported.

This commit is based on a patch for Tor
(git commit 56bdc844ba68ac0911efc7ad3398f1eafeaaac76 by Steven
Murdoch), Copyright (c) 2007-2011, The Tor Project, Inc.

Fix a memory leak on win32 socket->event map.

This would lose some memory every time an event_base was freed on win32.

Found by Dimitre Piskyulev.

Handle calloc failure in evdns. (Found by Dave Hart)

Merge branch '20_evdns_circular_list_patch' into patches-2.0

Fix a crash bug in evdns server circular list code

Really, this should use a circleq.  That's a change for 2.1, though.

Fix a couple of memory leaks in samples/http-server.c. Found by Dave Hart.

Avoid a free(NULL) if out-of-memory in evdns_getaddrinfo. Found by Dave Hart

Fix the check for multicast or broadcast addresses in evutil_check_interfaces

First of all, it is totally okay to have an address end with .255,
depending on what your netmask is, so we shouldn't reject a local
address if it ends with .255.

Second, our check for ending with .255 was broken.  So was our check
for class-d addresses.

Found by Dave Hart.

Merge branch '20_socktype_protocol_nomem' into patches-2.0

Check for allocation failures in apply_socktype_protocol_hack

Fix the case when failed evhttp_make_request() leaved request in the queue.

Fix subtle recursion in evhttp_connection_cb_cleanup().

Reset outgoing connection when read data in idle state.

Imagine server side is buggy and miscalculates Content-Length: in the
reply.  Data arriving in idle state shouldn't make us crash, instead we
can just reset the connection.

Add comment to configure.in to explain gc-sections test logic

(Basically, we need access to conftest.err for the test to work properly.)

Fix for --gc-sections on NetBSD

This patch fixes http://bugs.ntp.org/1844, works around
http://gnats.netbsd.org/cgi-bin/query-pr-single.pl?number=40401, by
improving the test for support of --gc-sections to run a program which
reads a file using stdio built with --gc-sections, instead of simply
link the binary.  This catches the buggy linker as the garbage
collection removes a tag NetBSD uses to distinguish its own elf
binaries from Linux ones, causing it to treat conftest as a Linux
binary and run it with the wrong syscall table.

Fix make distcheck & installation of libevent 1 headers

libevent/Makefile.am corrects a typo (thanks to Harlan for spotting it
once we realized make distcheck was broken when building the libevent
tearoff).  The result was the include/ev*.h were not distributed nor
installed whether or not --disable-libevent-install was used.  This
was introduced with the final round (3/3) of
--disable-libevent-install patch from me.

Fix incorrect assertions and possible use-after-free in evrpc_free()

Original description:

  The following patch fixes incorrect assertions in evrpc_free():
  evrpc_unregister_rpc() and evrpc_remove_hook() return 0 for success.

  Also, in evrpc_unregister_rpc(), it is better to free RPC structure
  at the end: evrpc_free() uses rpc->uri as "name" parameter when
  calling evrpc_unregister_rpc(), then rpc->uri is freed, but we have
  "registered_uri = evrpc_construct_uri(name)". So at this time "name"
  is invalid.

Disable lazy gettimeofday/clock_gettime comparison for now

Merge branch '20_timescale' into patches-2.0

Followup for Tomash Brechko's http patch

This patch makes bufferevent_disable_hard() non-public, and
adds a comment about what it's for and why it's used.

Workaround libevent bug
https://sourceforge.net/tracker/index.php?func=detail&aid=3078187&group_id=50884&atid=461324

The problem is that bufferevent_disable() doesn't disable EV_WRITE
when 'connecting' flag is set.  However from evhttp_connection_reset()
we want to disable EV_WRITE for sure (we are closing the socket next).
So we add bufferevent_disable_hard(), which acts like
bufferevent_disable(), but resets 'connecting' flag before the call to
the actual handler.

TODO: bufferevent_disable_hard() shouldn't be public, remove it from
event2/bufferevent.h.

Have event_base_gettimeofday_cached() always return wall-clock time

Based on code by Dave Hart

Make --no-libevent-install apply to event1 headers too

Make --no-libevent-install apply to headers too

Continue AM_CFLAGS to AM_CPPFLAGS correction

9c469db300e1b270a93c6b04c1709ac0f7751136 had started with the conversion
but left out another instance of incorrectly used AM_CFLAGS.

Dave Hart provided the suggestion for this fix.

Merge branch '20_uri_nonconformant' into patches-2.0

Make -ffunction-sections off-by-default in 2.0.x

I don't see how this can burn anybody, but I don't want to take
chances: new build options are something that should be done in an
alpha.  To turn -ffunction-sections on, pass
--enable-function-sections to configure.

Remove nonfunctional enable_libevent_(install|regress)_def variables

Use $(), not @@, for LIBEVENT_GC_SECTIONS

Be explicit about how long event loops run in event.h documentation

Use %s with printf in test.sh

Correct an AM_CFLAGS to an AM_CPPFLAGS in test/Makefile.am

Add configure options to disable installation, regression tests

The main reason for disabling installation is if you're building
libevent as a subpackage for embedding: you want to have your main
package's "make all" build libevent, but you don't want your main
package's "make install" to install libevent.

Add -D_OSF_SOURCE to fix hpux builds

Use the gcc -ffunction-segments feature to allow gc when linking with static libevent

Check for getservbyname even if not on win32.

Fix autoconf bracket issues; make check for getaddrinfo include netdb.h

Skip check for zlib if we have no zlib.h

Make test-eof fail with a timeout if we never get an eof.

Previously, it would just wait forever.

FIX: comment internal headers used in sample code.

NM: Fix sample/http-server ipv6 fixes

Fix 1: Use #include "../xyz" rather than #include <../xyz>

Fix 2: util-internal.h includes ipv6-internal.h, which in turn defines
       a replacement AF_INET6 on platforms that don't have it.  As such,
       the AF_INET6 checks shouldn't be needed.  (right?)

Include util-internal.h as needed to build on platforms with no sockaddr_storage