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

Add an empty section to the changelog for 2.0.18-stable

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

Correct a name in the credits. oops

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

Bump version to 2.0.17-stable-dev

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

Update the credits in the readme

Finalize the changelog

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

Conflicts:
Makefile.am
WIN32-Code/event2/event-config.h
configure.in

Update copyright notices to 2012

Bump version to 2.0.17-stable

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

In the kqueue backend, do not report EBADF as an EV_READ

We were doing this because of (correct) reports that NetBSD gives an
EBADF when you try to add the write side of a pipe for which the
read side has been closed.  But on most kqueue platforms, that
doesn't happen, and on *all* kqueue platforms, reporting a
nonexistent fd (which we usually have if we have seen EBADF) as
readable tends to give programs a case of the vapors.

Nicholas Marriott wrote the original patch here; I did the comment
fixes.

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

Avoid crash when freeing event_iocp and using event_set_mem_functions

There was a calloc that needed to be an mm_calloc.

Reported by "fffvvvzz" on sourceforge.  Ticket 3486114

Bring the changelog up to date

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

Loop on filtering SSL reads until we are blocked or exhausted.

This is not a perfect fix, but it's much much better than the
current buggy behavior, which could lead to filtering SSL
connections that just stopped reading.

Based on ideas by Maseeb Abdul Qadir and Mark Ellzey.

Replace more C99/C++ comments with oldschool /* */ comments

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

Use C-style comments in C source files (for compatibility with compilers such as xlc on AIX).

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

evdns: fix a bug in circular-queue implementation

found by Wang Qin

Fix a compile warning in event_reinit

Introduced in 272033efe575a9dc7ec6f123a96afba5c69ff1c6

Make test-changelist faster

Check changelist as part of checking representational integrity

Restore fast-path event_reinit() for slower backends

We used to use the needs_reinit flag in struct eventop to indicate
whether an event backend had shared state across a fork(), and
therefore would require us to construct a new event backend.  But
when we realized that the signal notification fds and the thread
notification fds would always be shared across forks, we stopped
looking at it.

This patch restores the old behavior so that poll, select, and
win32select don't need to do a linear scan over all pending
fds/signals when they do a reinit.  Their life is hard enough
already.

Make event_reinit() more robust and maintainable

Previously, event_reinit required a bunch of really dubious hacks,
and violated a lot of abstraction barriers to mess around with lists
of internal events and "pretend" to re-add them.

The new (and fairly well commented!) implementation tries to be much
smarter, by isolating the changes as much as possible to the backend
state, and minimizing the amount of abstraction violations.

Specifically, we now use event_del() to remove events we want to
remove, rather than futzing around with queues in event_reinit().
To avoid bogus calls to evsel->del(), we temporarily replace evsel
with a null-object stub.

Also, we now push the responsibility for calling evsel->add() down
into the evmap code, so that we don't actually need to unlink and
re-link all of our events.

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

Better workaround for Linux 3.2 edge-triggered epoll bug

On further investigation, it appears that this problem is limited to
AF_UNIX sockets, so let's just do the test on AF_INET sockets.

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

Do a memberwise comparison of threading function tables

Doing a memcmp risks comparing uninitialized padding bytes at the
end of the structure.

When including an -internal.h header outside the main tree, do so early

Some of our unit tests and sample code need functions and structures
defined in an -internal.h header.  But that can freak out OpenSolaris,
where stdio.h wants to define _FILE_OFFSET_BITS unless it's already
defined, and then evconfig-internal.h defines it.  Regular users
should never ever use our -internal.h headers, so the solution is
to make sure that if we're going to use them ourselves, we do so
before system headers.

Allow more slop in deferred_cb_skew test; freebsd needs it

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

Remove a couple of now-unused variables

Make event_base integrity check work on windows

Start writing a changelog for 2.0.17-stable

Use test_timeval_diff_eq more consistently

Make regression tests run over 3x faster.

This was mainly a matter of reducing timeouts and delays, paying
special attention to every test that took longer than a second to
finish.

We could do better here; IMO anything over .7 sec is probably too
long, but it's a big win as it is.

Remember, interactive computing is a big win over batch processing:
anything that makes you get up and walk away from the terminal might
as well be making you carry your punch cards over to the mainframe.

New evhttp function to adjust initial retry timeout

Reduce the timeout in the main/fork test.

There was no reason for it to be so long, except for the lack of a usleep

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.