Support negative arguments to _bufferevent_decrement_(read/write)_buckets()

Cleanup on 7c11e51e1ab: fix strtol usage

Cleanup on 7c11e51e1ab: restore c90 declaration compliance

Clean up some problems identified by Coverity.

Make rate limiting work with common_timeout logic

Merge branch '20_iocp_fixes' into patches-2.0

Make overlapped reads result in evbuffer callbacks getting invoked

IOCP: don't launch reads or writes on an unconnected socket

Try to fix 'make distcheck' errors when building out-of-tree

Merge branch '20_low_ratelim' into patches-2.0

Solaris sendfile: correctly detect amount of data sent

Original message:

   Solaris sendfile seems to fail when sending moderately large (<1GB)
   files. Not a 32/64 problem, but a buffer problem.

   Anyone else ever try this? It is definitely broken in http-server.c.

   It seems to be broken in the following way:

   When sendfile sends partial data (EAGAIN, would block), "res" is
   always -1, rather than the amount sent.

   Here's a patch that reads from the "offset" pointer instead to
   discover what was sent. This seems to work:

Fix request_finished memory leak with debugging turned on.

Fix evsig_dealloc memory leak with debugging turned on.

Another docstring fix.

Fix handling of group rate limits under 64 bytes of burst

The "min_share" logic, which was designed to prevent piles of
extremely small writes when running up against a group rate limit,
could lead to confusing behavior if you ever set a min_share less
than your burst rate.  If that happened, then as soon as your group
rate limit was exhausted, you'd stop reading/writing, and never
start again, since the amount readable/writeable would never
actually hit min_share.

We now cap min_share at the rate per tick.

Found by George Kadianakis

Ignore deprecation warnings on OS X

Starting with Lion, Apple decided to deprecate the system openssl. We
can start requiring users to install their own openssl once OS X doesn't
ship with it anymore.

Allow OS-neutral builds for platforms where some versions have arc4random_buf

Propagate errors on the underlying bufferevent to the user.

Fix typo in event_compat.h comments.

Bump version to 2.0.13-stable-dev

Bump version to 2.0.13-dev

acks and changelog for 2.0.13-stable

Give Makefile.am echo a non-null argument. msys likes this

Fix a warning in evutil_rand when building with threads disabled

Use AM_CPPFLAGS in sample/Makefile.am, not AM_CFLAGS

Reported by Dagobert Michelsen.

Add doxygen to .gitignore

Fix up test_evutil_snprintf

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

Merge branch 'doxygen' into patches-2.0

Update Doxyfile to produce more useful output

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

Fix select.c compilation on systems with no NFDBITS

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.

Use the correct printf args when formatting size_t

Based on a patch from Mansour Moufid

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