Make event-config.h depend on Makefile.am

Check for NULL return on win32 mm_calloc, and set ENOMEM.

(It looks like windows doesn't set ENOMEM on overflow from calloc.)

Do not use system EAI/AI values if we are not using the system getaddrinfo.

Replace sole internal user of socklen_t with ev_socklen_t

This lets us remove the socklen_t definition from util-internal.h,
so that util-internal.h no longer confuses ws2tcpip.h

Fix evmap to work with HT-backed IO maps again.

evutil_accept4() should take ev_socklen_t

test-fdleak: Avoid perror, since it does not capture windows socket errors

remove debugging prints

test-fdleak: Call event callbacks event_cb.

Remove fixed port from test-fdleak, so two instances can run in parallel

Remove an unsafe programming practice from test-fdleak; add XX comments

It's a bad idea to have a read callback read a fixed amount; Instead,
we should drain it, or have some way to know that more data is
arrived.  Not unsafe in this case, but let's not encourage people to
learn bad habits.

This commit also notes some other stuff to fix.

Try and fix compilation and running of test-fdleak on Windows.

Allow compilation on machines that do not have setrlimit.

Simply test-fdleak by running server & client sockets in the same event loop.

Add a new test: test-fdleak which tests for fd leaks by creating many sockets.

This test opens a server socket, and forks a child which connects to that
server socket many times. It sets a low number for the max open file limit
to catch any file descriptor leaks.
It would not work on Windows since it uses fork() to be able to create both the
server and the clients.

If accept4 is absent, fall back to accept.

Avoid leaking fd in evutil_socket() by Linux.

Found by Amarin Phaosawasdi

Remove the last vestiges of _EVENT_USE_EVENTLIST

Merge branch '21_eventlist_v3_squashed'

Add doxygen for event_base_dump_events

Refactor the functions that run over every event.

Now there are appropriate "for each event", "for each fd", and "for
each signal" helpers that they can use; this makes the code a bit
simpler, and far less duplicated.

This lets me turn back on the functions I disabled when removing
eventlist.

Additionally, check more lists for circularity in
event_base_assert_ok().

Add typedefs for the callback types.

Name fewer things "ctx".

Adds an implementation of Floyd's tortoise-and-hare algorithm to check
for circularity in TAILQs and LISTs, to avoid the abuse of flags that
event_base_assert_ok() was doing before.

Suggested by Dave Hart.

Fix an "unused return value" warning on write() in signal handler

Start work on a 2.1 changelog and whats-new

Add another caveat to the TCP_DEFER_ACCEPT documentation

Support TCP_DEFER_ACCEPT sockopts for listeners

A listening socket can be enabled with the sockopt
TCP_DEFER_ACCEPT. This informs the kernel to not call the user-land
accept() until real data has been written to the socket.

A new flag LEV_OPT_DEFERRED_ACCEPT has been introduced to
automatically set this option. Optionally
evutil_make_tcp_listen_socket_deferred() can be called manually.

(Tweaked slightly by nickm.)

Merge remote-tracking branch 'github/21_fast_syscalls'

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

Stop crashing in evdns when nameserver probes give a weird error

When a nameserver is down, we periodically try sending a "probe"
message to that nameserver to see if it has come back up.  If a
nameserver comes up, we cancel any pending probe messages.

Cancelling a probe message while handling the probe's response would
result in a access-after-free or a double-free, so when we notice that
we're about to call a nameserver up because of having received a probe
from it, we need to check whether current response is the response
from the probe.

There was a case where we didn't to that, though: when the resolver
gave us an unusual error response to our request that it resolve
google.com.  This is pretty rare, but apparently it can happen with
some weird cacheing nameservers -- the one on the mikrotik router, for
example.  Without this patch, we would crash with a NULL pointer
derefernce.

Thanks to Hannes Sowa for finding this issue and helping me track it
down.

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

Conflicts:
configure.in

Changed OPENSSL_LDFLAGS to OPENSSL_LIBADD

Added OPENSSL_LDFLAGS env variable which is appended to SSL checks.

If openssl is not installed system-wide or not compiled as a shared library,
some systems require various link flags (e.g., -ld).

If open(O_CLOEXEC) fails, fall back to fcntl(CLOEXEC)

This is needed for folks who build with recent Linux kernel headers
but run with older kernels.

Fix up usage of reserved identifiers in ht-internal.h

Port changes from Tor for ht-internal.h

Invert the sense and the name of HT_CACHE_HASH_VALUES

This is meant to make it easier to merge ht-internal.h back upstream
into Tor.

Synchronize with upstream tinytest

Merge in a punctuation fix from upstream tinytest

Move libevent 1.x headers to include/, to put all public headers in one place.

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

Tweak the evutil_open_closeonexec patch to work on windows, old unixes.

Windows doesn't have a mode_t as far as I can tell.

Some unixes, iirc, don't like three-argument open without O_CREAT.

Make uses of open() close-on-exec safe by introducing evutil_open_closeonexec.

In a multi-process/threaded environment, opening fds internally
without the close-on-exec flag could leak fds to child processes.

Add a bufferevent_getcb() to find a bufferevent's current callbacks

Also make win32select.c conditional for IDE users

Merge branch 'ifdef' of git://github.com/rosslagerwall/libevent

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

Fix compilation on osx

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

Prefer epoll_create1 on Linuxen that have it

Save some syscalls when constructing a socket for a bufferevent

Save some syscalls when creating evdns sockets

Save syscalls when constructing listener sockets for evhttp

Use pipes for telling signals to main thread when possible

Use a wrapper function to create the notification pipe/socketpair/eventfd

The LEV_OPT_CLOSE_ON_EXEC flag now applies to accepted listener sockets too

Minimize syscalls during socket creation in listener.c

Infrastructure for using faster/fewer syscalls when creating sockets

Linux provides some features that allow avoiding extra calls to
fcntl when creating new nonblocking/close-on-exec sockets, so
we can add wrapper functions to emulate those when they are not
available.

Additionally, even when we are emulating those functions, we can
take a fast path that cuts our fcntl calls in half: we don't need to
look up the previous value of a file's flags when we have just
created it.

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

Put #ifdef around some files to support alternate build systems.

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

Remove the eventqueue list and the ev_next pointers.

Those pointers were once used to maintain a complete list of
inserted IO and signal events.  But such a list is now available by
walking over ev_io_map and ev_signal_map!  So all they did was
require extra pointer operations to maintain, and extra 8-16 bytes
of storage in each struct event.

To be cowardly and keep the option of going back to having this
redundancy, I'm wrapping the removed code in a set of ifdefs.

This is a first cut; it needs cleanups and stress-testing!!  In
particular, it just plain disables a couple of functions that could
probably be saved.

There seems to be a need for an evmap_{io,signal}_foreach() or something.

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