Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

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

Merge remote-tracking branch 'origin/patches-2.0'

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.

Do not define WIN32 in Makefile.nmake

In configure, test for _WIN32 not WIN32.

Use "_WIN32", not WIN32: it's standard and we don't need to fake it

This patch was automatically generated with perl.

Based on a patch by Peter Rosin.

Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

Fix new warnings from GCC 4.6

Merge remote-tracking branch 'origin/patches-2.0'

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'.

Merge remote-tracking branch 'origin/patches-2.0'

Fix a bug that prevented us from configuring IPv6 nameservers.

Merge remote-tracking branch 'origin/patches-2.0'

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."

Merge remote-tracking branch 'origin/patches-2.0'

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).

Merge remote-tracking branch 'ellzey/feature/evhttp_parse_speedup'

Merge remote-tracking branch 'origin/patches-2.0'

Fix compilation with GCC 2, which had no __builtin_expect

Merge remote-tracking branch 'origin/patches-2.0'

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

Performance tweak to evhttp_parse_request_line.

Method parsing has been tweaked out to use a lookup table instead of multiple
branching. In our testing it has increased performance by a fair bit.

Move the win32 detection in configure.in

Use OPENSSL_LIBS in Makefile.am

Merge remote-tracking branch 'origin/patches-2.0'

Libtool brings in the dependencies of libevent_openssl.la automatically

libssl depends on libcrypto, not the other way around.

Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

Add missing words to EVLOOP_NONBLOCK documentation

Merge remote-tracking branch 'origin/patches-2.0'

Bump version to 2.0.11-stable-dev

Merge remote-tracking branch 'origin/patches-2.0'

Increment version number to 2.0.11-stable

Merge remote-tracking branch 'origin/patches-2.0'

Add more people to the acknowledgments

Merge remote-tracking branch 'origin/patches-2.0'

Changelog for 2.0.11-stable

Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

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

Merge remote-tracking branch 'origin/patches-2.0'

Conflicts:
evthread.c

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.

Merge remote-tracking branch 'origin/patches-2.0'

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 remote-tracking branch 'origin/patches-2.0'

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.

Merge remote-tracking branch 'origin/patches-2.0'

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

Merge remote-tracking branch 'origin/patches-2.0'

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 remote-tracking branch 'origin/patches-2.0'

Merge branch '20_socktype_protocol_nomem' into patches-2.0

Check for allocation failures in apply_socktype_protocol_hack

Merge remote-tracking branch 'origin/patches-2.0'

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.

Merge remote branch 'origin/patches-2.0'