Fix an unlikely but possible error case for http connections

Found by coverity

Move event_debug_note_teardown_ before mm_free.

This isn't a bug, since only the pointer value of ev was used, but
it's probably best not to tempt fate.  Found by coverity.

Add some assertions to please coverity.

In event_process_active_single_queue, EVLIST_INIT must be set on any
event that uses one of the event-only closures, and so "ev" will be
set in those cases.  But coverity's worried here (CIDs numerous).  So
instead, just add the assertions that should make it happy.

Fix compilation

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

Conflicts:
arc4random.c

Add evutil_secure_rng_set_urandom_device_file

This experimental function is needed for some seccomp2 hackery to
work, and should have no effect for systems that don't use it.

Remove an unreachable return statement in minheap-internal.h

Found by Brian Utterback; see http://bugs.ntp.org/show_bug.cgi?id=2446

Avoid leaking segment mappings when offset is not a page multiple

Found by Bob / Black Hole on the mailing list.

Add checks for evhttp_connection_get_server() in unit tests.

They validate that this function behave as expected; that is, it returns
NULL for outgoing connections, and returns the HTTP server object that
received the connection for incoming ones.

Add evhttp_connection_get_server().

Merge remote-tracking branch 'ellzey/bev_ratelimit_get_token_cfg'

Add Maxime Henrion to README

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

libevent/win32_dealloc() : fix sizeof(pointer) vs sizeof(*pointer)

Pass and return const for bufferevent_get_token_bucket_cfg

Add function to fetch underlying ratelimit cfg

bufferevent_get_token_bucket_cfg() will return the struct ev_token_bucket_cfg
for a bufferevent if available.

Merge pull request #69 from makotokato/clang

Don't use return since return type is void and build error occurs using ...

We should return after arc4random_buf()

Don't use return since return type is void and build error occurs using clang

Added event_base_get_num_events()

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

Merge remote-tracking branch 'public/pr/79' into patches-2.0

Merge branch '21_evdns_log_debug'

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

Document that arc4random is not a great cryptographic PRNG.

When we seed from /proc/sys/kernel/random/uuid, count it as success

Found by Joseph Spadavecchia

restore the comment

Fix SEGFAULT after evdns_base_resume if no nameservers installed.

If there is no nameservers installed, using
evdns_base_nameserver_ip_add(), than evdns_base_resume() will SEGFAULT,
because of NULL dereference in evdns_requests_pump_waiting_queue()

Conflicts:
evdns.c

Actually use the log facility for reporting evdns problems.

Fixes issue #63.  Also refactors the evdns logging and log backend a
bit, so that it wastes a little less code.

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

Use windows vsnprintf fixup logic on all windows environments

Previously I'd relied on mingw to provide a vsnprintf with a
conformant return value.  But it appears that some mingw
environments don't do that.

Merge remote-tracking branch 'origin/pr/62'

Remove a debugging assert that should not have been left in

Fix locking in bufferevent_get_options_().

Fix a double close() bug in evhttp when the underlying bufferevent uses BEV_OPT_CLOSE_ON_FREE.

Avoid redundant syscall if making a socket cloexec twice

I got the idea from Maxime's last patch.

Avoid redundant syscall to make a nonblocking socket nonblocking

If evsel->del() fails, don't leave the evmap in an inconsistent state.

 This fixes assertion failures in cases where epoll() fails with EBADF
 -- the root cause for which is as of yet unknown. It seems something
 (OpenSSL?) is closing the file descriptor under our feet.

Fix dns/leak_resume_send_err test.

Because we don't cancel request,
and want our callback to recieve DNS_ERR_SHUTDOWN,
we use deferred callback, and there was
- one extra malloc(),
  @see reply_schedule_callback()
- and one missing free
  @see request_finished() (req->handle->pending_cb = 1)
than we don't need to count in testleak_cleanup()

So just decrement allocated_chunks to 2,
like we already take care about it.

Add tests for evdns_base_resume().

- leak_resume
- leak_cancel_and_resume
- leak_resume_send_err
- leak_cancel_and_resume_send_err

Fix SEGFAULT after evdns_base_resume if no nameservers installed.

If there is no nameservers installed, using
evdns_base_nameserver_ip_add(), than evdns_base_resume() will SEGFAULT,
because of NULL dereference in evdns_requests_pump_waiting_queue()

Whoops. It is gdi.lib, not gdi32.lib. (github issue #61)

Missed lock acquire/release in event_base_cancel_single_callback_()

Call backtrace:
...
event_queue_remove_active()
event_callback_cancel_nolock_()
event_base_cancel_single_callback_()
event_base_free_()
event_base_free()
...

Fix for e9ebef83

Fix test compilation with nmake: add the gdi.lib dependency

ACK: efekty

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

Checking request nameserver for NULL, before using it.

Bump version to 2.1.3-alpha-dev

Add regress_finalize to makefile.nmake

Set a release date

Make http/connection_retry off-by-default

Fix a windows compilation regression

This is github issue #57; reported by "efekty". I assume the compiler
is MSVC.

Fix a bug in fixing a bug in out-of-tree test-dumpevents

Make regress_finalize work with malloc replacement disabled

Fix some out-of-tree build bugs

Incremement version to 2.1.3-alpha

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

Add acknowledgements to readme

Start editing the changelog for 2.1.3-alpha

Mark the finalize stuff as experiemental in case it needs to change

Start a changelog for Libevent 2.0.22-stable

Clarify an important point about event_base_foreach_event()

Fix a bug in the new main/event_foreach test

It wasn't making sure that the events weren't internal.

More unit tests for initializing common timeouts.

Try with: misformed usecs in inputs, and with initializing common timeouts
from other common timeouts.

A test for event_get_assignment()

Unit tests for event_base_gettimeofday_cached() and event_base_update_cache_time()

New tests for event_base_foreach_event()

Our dump-events code had exercised this a bit, but only via the
nonlocking backend. Also, nothing was checking the "Search for a
specific event" code.

New test for active_later->active transition on event_active

When event_active is called, an active_later event is supposed to become
active now.  We had lacked a test for that.

Merge branch '21_deadlock_fix_v2'

Remove a no-longer-true XXXX comment.

Make the event_finalize* functions return an error code

More documentation for finalization feature

Remove bufferevent_del_generic_timeout_cbs as now unused

Always run pending finalizers when event_base_free() is called

There was actually a bug in the original version of this: it tried to
run the finalizers after (potentially) setting current_base to NULL;
but those finalizers could themselves (potentially) be invoking stuff
that needed to know about the current event_base.  So the right time to
do it is _before_ clearing current_base.

Use finalization feature so bufferevents can avoid deadlocks

Since the bufferevents' events are now EV_FINALIZE (name pending),
they won't deadlock.  To clean up properly, though, we must use the
finalization feature.

This patch also split bufferevent deallocation into an "unlink" step
that happens fast, and a "destruct" step that happens after
finalization.

More work is needed: there needs to be a way to specify a finalizer
for the bufferevent's argument itself.  Also, this finalizer business
makes lots of the reference counting we were doing unnecessary.

Also, more testing is needed.

Add some verbose notes to bufferevent unit tests

Implement event_finalize() and related functions to avoid certain deadlocks

Make all tests pass under EVENT_DEBUG_MODE=1

The fix was easy: the tests that were failing were those tests that
also enabled debug mode themselves. So, let them know when
EVENT_DEBUG_MODE is on, so they won't do that.

Merge branch '21_http_error_cb_squashed'

Drop extra header http_struct.h from regress_http.c

Add test for EVREQ_HTTP_REQUEST_CANCEL into http_cancel_test()

Add new error_cb for actual reporting of HTTP request errors.

It is useful to know why you callback called with NULL (i.e. it failed),
for example if you set max_body with evhttp_connection_set_max_body_size()
you must know that it failed because of body was longer than this size.

 (Commit message tweaked by Nick)

Fix another warning introduced in 0c6ec5d8

Fix a warning introduced in 0c6ec5d8

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

Conflicts:
configure.ac

Merge remote-tracking branch 'origin/pr/36' into patches-2.0

Merge branch '21_evdns_disable_when_inactive'

evdns: New flag to make evdns not prevent the event loop from exiting

Here is the brief description of problem:
When you are use evdns to resolve domains to IP adresses (see
./sample/dns-example) you loop never returns from event_base_dispatch(),
and because of this the program will never terminated.

Because existing programs may be depending on the old behavior, we
only apply the fix when evdns_base_new() is created with a new flag -
EVDNS_BASE_DISABLE_WHEN_INACTIVE.

 (Commit message edited by Nick while squashing the branch.)

event_base_update_cache_time should be a no-op if the loop isn't running

Add a test with an active_later event at event_base_free time.

Another tweak to https-client.c

Remove http_struct.h usage in sample/https-client.c

Add an environment variable (EVENT_DEBUG_MODE) to run unit tests in debug mode

Not all tests currently pass with debug mode on.

Merge branch '21_empty_strlcpy'

Do not build strlcpy.c when it will have no code.

Merge pull request #75 from altf4/master

Header update to specify evbuffer_pullup() behavior

Specify return behavior in header for evbuffer_pullup() in corner case

Function returns NULL when told to pullup more data than exists

Merge pull request #47 from ppelleti/https

HTTPS example adapted from Catalin

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

Fix a typo in a comment in buffer.h. Spotted by Alt_F4

build test/test-script.sh on systems with a less-featureful $<