Add an (internal) usleep function for use by unit tests

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

Oops:remove an accidentally committed "sleep(1)" in a unit test

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

Workaround in the unit tests for an apparent epoll bug in Linux 3.2

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

Conflicts:
event.c

Edits required in:
        evmap.c

Fix a fd leak in event_reinit()

We were supposed to be closing the ev_signal_pair sockets.

Fix a list corruption bug when using event_reinit() with signals present

While re-adding all the events, event_reinit() could add a signal
event, which could then cause evsig_add() to add the
base->sig.ev_signal event.  Later on its merry path through
base->eventqueue, event_reinit() would find that same event and give
it to event_io_add a second time.  This would make the ev_io_next
list for that fd become circular.  Ouch!

Check event_base correctness at end of each unit test

Fix compilation of windows evutil_check_ifaddrs

Add function to check referential integrity of an event_base

Merge remote-tracking branch 'github/linked_list'

Conflicts:
include/event2/event_struct.h

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

Force strict validation of HTTP version in response.

This sometimes accepted invalid versions like 'ICY' (n = 0, major = undefined, sometimes > 1).

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

Suppress a gcc warning from ignoring fwrite return in http-sample.c

Found by Steve Snyder

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

Fix a race condition in the dns/bufferevent_connect_hostname test.

As originally written, the test would only pass if the accept()
callbacks for the evconnlistener were all invoked before the last of
the CONNECTED/ERROR callbacks for the connecting/resolving bufferevent
had its call to event_base_loopexit() complete.  But this was only
accidentally true in 2.0, and might not be true at all in 2.1 where
we schedule event_base_once() callbacks more aggressively.

Found by Sebastian Hahn.

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

Make evconnlistener work around bug in older Linux when getting nmapped

Older Linuxes sometimes respond to some nmap probes by having accept()
return a success but with socklen 0.  That can lead to confusing behavior
when you go to process the sockaddr.

Remove bogus casts of socket to int before calling ev_callback

This should make 64-bit windows act better.

Found by Mark Heily.

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

Backport evhttp_connection_get_bufferevent to Libevent 2.0

Backport by Arno Bakker; original implementation in 8d3a8500f4

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

Slightly clarify evbuffer_peek documentation

Fix behavior of evbuffer_peek(buf,-1,NULL,NULL,0)

(Patch altered by nickm to not affect the behavior of
evbuffer_peek(buf,-1,NULL,vec,n_vec).)

Remove some accidentally-committed debugging code

whitespace fixes

Merge remote-tracking branch 'fancycode/buffer_references'

Conflicts:
buffer.c

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

Conflicts:
configure.in

check for sysctl before we use it

Not all C libraries under Linux support the sysctl() func.

Split up extra-long AC_CHECK_FUNCS/HEADERS lines in configure.in

Merge branch '21_ifaddr'

Use getifaddrs to detect our interfaces if possible

The old scheme involved connected UDP sockets and getsockname(), and is
generally best avoied.

Changed use of refcounts to make sure referenced chains are freed in all cases.

Add evbuffer_copyout_from to copy data from the middle of a buffer

You could previously do this with evbuffer_peek() and some memcpys,
but it was a bit more work than most folks wanted to get into.

Closes sourceforge ticket 3108072

Bypass event_add when using event_base_once() for a 0-sec timeout

Some people use event_base_once(EV_TIMEOUT) to make a callback get
called "immediately".  But this is pretty roundabout: it uses the
timeout heap to immediately put the event onto the active queue, when
it could just use event_active.  Additionally, it can lead to
surprising re-ordering behavior.

This patch changes event_base_once so it bypasses event_add() and
called event_active() directly on a pure-timeout event with an empty
timeout.

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

Fix a silly compilation error with the sun compiler

Apparently, other compilers were allowing "return free(x)" in a
function returning void.

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

Be absolutely sure to clear pncalls before leaving event_signal_closure

I thought we'd fixed the cases where this could come up, but
apparently having an event_base_break() happen while processing
signal events could get us in trouble.

Found by Remi Gacogne.  Sourceforge issue 3451433 .

fix windows compilation issues with new file segment code

Found by Linus Nordberg

Fix two issues in the allow_dirty_shutdown code.

First, it shouldn't crash when it's passed a non-ssl bufferevent.

Second, it should behave correctly when it gets a true argument
other than 1.

Tweak allow_dirty_shutdown documentation

Allow users to set allow_dirty_shutdown

bev_ssl: Be more specific in event callbacks. evhttp in particular gets confused without at least one of BEV_EVENT_{READING|WRITING}.

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

Increment version to 2.0.16-stable-dev

Increment version to 2.0.16-stable

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

Revise 2.0.16-stable changelog

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

add comment to new consider_reading code

Avoid spinning on OpenSSL reads

Previously, if some sender were generating data to read on an
OpenSSL connection as fast as we could process it, we could easily
wind up looping on an openssl do_read operation without ever
considering other sockets.

The difference between this and the original method in
consider_reading() is that it only loops for a single completed
*frame* instead of looping until fd is drained or an error condition
was triggered.

{Patch split out by nickm}

Move SSL rate-limit enforcement into bytes_to_read()

Refactor amount-to-read calculations in buffervent_ssl consider_reading()

Split up consider_reading()'s conditional checks into another function
can_read() for simplicity sake.

{Split into a separate patch by nickm}

Revert "Avoid potential SSL read spinlocks"

This reverts commit fc52dbac87f4937f8306759506d6a2ad15ca244c.

Revert "openssl bufferevent has the same issue with writing as prior commit."

This reverts commit 7353663eb7c0b2a1caaaa5acd818515f156cf2ca.

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

Add new entries to changelog, new credits to README

Fix DNS memleak checks when running with malloc-replacement/debugging disabled

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

Don't try to make notifiable event_base when no threading fns are configured

Warn when unable to construct base because of failing make_base_notifiable

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

openssl bufferevent has the same issue with writing as prior commit.

Avoid potential SSL read spinlocks

OpenSSL bufferevents with deferred callbacks enabled under high load will
spinlock in the function consider_reading(). This loop continues until all
data has been read.

Because of this condition; openssl bufferevents will never return back into
event_base_loop() until SSL_read has determined data is no longer ready.

As of yet I have not found a reason why this while loop exists, so this patch
just swaps out while for if.

If needed I can write same code which would trigger this effect; optionally
libevhtp has a test.c program which can be run with the following flags:

./test -s <keyfile.pem>

curl -vvvv -k -d@<HUGE_ASS_FILE> https://127.0.0.1:8081/

The return data will include the number of times the readcb got data and the
length of that read.

Without this patch, you are likely to see a small amount of "bytes read....",
otherwise the "bytes read..." return data should show much more reasonable
numbers.

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

Conflicts:
sample/dns-example.c

unit tests for EVBUFFER_EOL_NUL

New EVBUFFER_EOL_NUL to read NUL-terminated strings from an evbuffer

Tweaks, fixups, and comments on evbuffer_add_iovec

Added evbuffer_add_iovec and unit tests.

Improve win32 behavior of dns-sample.c codex

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

Fix typo in whatsnew-2.0.txt

Merge remote-tracking branch 'github/20_evbuffer_remove_bug'

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

Use the free-trailing-chains function in evbuffer_insert_chain too

Fix an evbuffer crash in evbuffer_remove_buffer()

Found by Greg Hazel.

improve test to remove at least one buffer from src

unit test for remove_buffer bug

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

Fix compile warning from saying event2/*.h inside a comment

Based on a patch by Adrian Chadd

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

epoll: close fd on alloc fail at initialization

If the memory allocations fail then we free any other allocated
structures but don't close the file descriptor resulting in an leak of
fd's.

evhttp: Add evhttp_foreach_bound_socket.

Applies the function specified in the first argument to all
evhttp_bound_sockets associated with a struct evhttp. The user
must not attempt to free or remove any connections, sockets or
listeners in the callback function.

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

Update copyright dates to 2011.

New event_base_update_cache_time() to set cached_tv to current time

This function is particularly useful for selectively increasing
the accuracy of the cached time value in 'base' during callbacks
that take a long time to execute.

This function has no effect if the base is currently not in its
event loop or if timeval caching is disabled via EVENT_BASE_FLAG_NO_CACHE_TIME.

Add argument checks to some memory functions in `event.c'.

Add a zero check to the function `event_mm_malloc_',
i.e. simply return NULL if the sz argument is zero.
On failure, set errno to ENOMEM and return NULL.

Add a zero check to the function `event_mm_calloc_',
i.e. simply return NULL if either argument is zero.
Also add an unsigned integer multiplication check, and if an integer
overflow would occur, set errno to ENOMEM and return NULL.
On failure, set errno to ENOMEM and return NULL.

Add a NULL check to the function `event_mm_strdup_',
i.e. set errno to EINVAL and return NULL.
Also add an unsigned integer addition check, and if an integer
overflow would occur, set errno to ENOMEM and return NULL.
If a memory allocation error occurs, again set errno to ENOMEM
and return NULL.

Add unit tests to `test/regress_util.c'.

Add note about evhttp_send_reply_end to its doxygen

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

More detailed message in case of libevent self-debugging failure.

Style and comment tweaks for dns/leak* tests

Test for commit aff6ba1

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

Empty DNS reply with OK status is another way to say NODATA.

Sometimes DNS reply has nothing but query section. It does not look like
error, so it should be treated as NODATA with TTL=0 as soon as there is
no SOA record to deduce negative TTL from.

Tests for 94fba5b and f72e8f6

fixed typo

added comments to describe refcounting of multicase chains