Update Doxyfile to produce more useful output

Revise the event/evbuffer/bufferevent doxygen for clarity and accuracy

Merge remote-tracking branch 'github/20_global_locks_init' into patches-2.0

Speed up invoke_callbacks on evbuffers when there are no callbacks

This fixes a performance regression against 1.4

Fix bug in SSL bufferevents backed by a bev with a write high-watermarks

Original mail:

   the logic that handles write watermarks in "bio_bufferevent_write"
   is not working. It currently doesn't write any data if the high
   watermark is *above* the amount of data to write (i.e. when there
   is actually enough room available).

Fix a few warnings on OpenBSD

- redeclaration of dst_size

- arpa/inet.h requires netinet/in.h first

- don't use a local with the same name as a global - it isn't needed so
  remove it

Fix AIX build issue with TAILQ_FOREACH definition

Reported by Lawnstein Chan.

Don't install event_rpcgen.py when --disable-libevent-install is used

Merge branch '20_kqueue_badf' into patches-2.0

Don't break when building tests from git without python installed

Simplify windows commandname logic in tinytest

Instead of using a dup'd pointer, let's use a static array, so we
don't need to free it.  This patch also makes tinytest build on
non-windows again.

Fix tinytest invocation from windows shell

Original post:

  This post is in response to a posting last December on a Windows
  regression fork failure ([Libevent-users] Re: Libevent 2.0.10-stable
  is released by Dongsheng Song).  I noticed the question was not
  answered and I recently experienced the same error myself when
  trying to run the Windows regression tests myself.

  I checked the return status from the CreateProcess call and found it
  was "file not found".  This led me to look at the command-line I was
  using which was .\regress in a Visual Studio 2008 command prompt
  window.  Windows could not find the file because it did not have the
  .exe extension on the end.  The code that builds the command should
  be modified to ensure the extension is present.

Check if the `evhttp_new_object' function in `http.c' returns NULL.

Report kqueue ebadf, epipe, and eperm as EV_READ events

When asked to add one side of a pipe, and the other side has been
closed, kqueue on NetBSD will say EBADF; kqueue on FreeBSD will say
EPIPE, and kqueue on OpenBSD will say EPERM.  So treat all of these
as EV_READ events, to give the user an opportunity to notice that
the pipe is closed.

Diagnosed by Nicholas Marriott and Dale Rahn; based on a patch by
Nicholas Marriott.

Replace an assertion for event_base_free(NULL) with a check-and-warn

event_base_free(NULL) means "free the current event base".
Previously, it would assert if there was no 'current' base.  Now it
just warns and returns.

Reported by Gilad Benjamini

Add some missing checks for mm_calloc failures

Found by Gilad Benjamini

Fix incorrect results from evbuffer_search_eol(EOL_LF)

Our evbuffer_strchr() function [which was only used for
search_eol(EOL_LF) could give incorrect results if it found its answer
in the first chunk but didn't start searching from the front of the
chunk.

Also, this patch adds unit tests for evbuffer_search_eol, particularly
in those cases that evbuffer_readln() tests didn't exercise.

Bump version to 2.0.12-stable-dev

Merge branch 'patches-2.0' of ssh://levent.git.sourceforge.net/gitroot/levent/libevent into patches-2.0

tweak date on changelog

Merge branch 'patches-2.0' of ssh://levent.git.sourceforge.net/gitroot/levent/libevent into patches-2.0

Changelog and new credits for 2.0.12-stable

Bump version to 2.0.12-stable.  Not release just yet

Fix regress_ssl.c build on openbsd

Avoid a segfault when all methods are disabled or broken

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.

Complain if the caller tries to change threading cbs after setting them

We never supported this; it was always fraught with errors; and I
don't believe there is a good reason to _want_ it to work.

Avoid race-condition when initializing global locks

Previously, we did stuff like
   if (!lock)
      EVTHREAD_ALLOC_LOCK(lock,0);
for the evsig base global lock, the arc4random lock, and the debug_map
lock.  But that's potentially racy!  Instead, we move the
responisiblity for global lock initialization to the functions where
we set up the lock callbacks.

(Rationale: We already require that you set up the locking callbacks
before you create any event_base, and that you do so exatly once.)

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.