Refactor test_getaddrinfo_async_cancel_stress()

Turns out it didn't need an event base set up.

Spotted by clang's static anaylzer

add some (void) casts for unused variables

All these places already have an XXX marker, so we know we need to fix
them later. Let's not clutter the static analysis results with them.

Actually send NULL request in http_bad_request_test

Issue caught by clang's static analyzer

Add a forgotten return value check in the unit tests

Spotted by clang's static analyzer

Add an assert to appease clang's static analyzer

It got confused because in EVLOCK_ASSERT_LOCKED(lock) there is an if
(lock) check.

Fix possible NULL-deref in evdns_cancel_request

The clang static analyzer complained that base might be dereferenced
without being set, but this patch should fix another dereference issue.

Add a forgotten NULL check to evhttp_parse_headers

Issue detected by the clang static analyzer

Fix compilation under LLVM/clang with --enable-gcc-warnings

When compiling using clang (2.9 or lower) do not enable
-Wnormalized=id or -Woverride-init when --enable-gcc-warnings
or --enable-gcc-warnings-advisory is set as these options
are unsupported.

This commit is based on a patch for Tor
(git commit 56bdc844ba68ac0911efc7ad3398f1eafeaaac76 by Steven
Murdoch), Copyright (c) 2007-2011, The Tor Project, Inc.

Fix a memory leak on win32 socket->event map.

This would lose some memory every time an event_base was freed on win32.

Found by Dimitre Piskyulev.

Handle calloc failure in evdns. (Found by Dave Hart)

Merge branch '20_evdns_circular_list_patch' into patches-2.0

Fix a crash bug in evdns server circular list code

Really, this should use a circleq.  That's a change for 2.1, though.

Fix a couple of memory leaks in samples/http-server.c. Found by Dave Hart.

Avoid a free(NULL) if out-of-memory in evdns_getaddrinfo. Found by Dave Hart

Fix the check for multicast or broadcast addresses in evutil_check_interfaces

First of all, it is totally okay to have an address end with .255,
depending on what your netmask is, so we shouldn't reject a local
address if it ends with .255.

Second, our check for ending with .255 was broken.  So was our check
for class-d addresses.

Found by Dave Hart.

Merge branch '20_socktype_protocol_nomem' into patches-2.0

Check for allocation failures in apply_socktype_protocol_hack

Fix the case when failed evhttp_make_request() leaved request in the queue.

Fix subtle recursion in evhttp_connection_cb_cleanup().

Reset outgoing connection when read data in idle state.

Imagine server side is buggy and miscalculates Content-Length: in the
reply.  Data arriving in idle state shouldn't make us crash, instead we
can just reset the connection.

Add comment to configure.in to explain gc-sections test logic

(Basically, we need access to conftest.err for the test to work properly.)

Fix for --gc-sections on NetBSD

This patch fixes http://bugs.ntp.org/1844, works around
http://gnats.netbsd.org/cgi-bin/query-pr-single.pl?number=40401, by
improving the test for support of --gc-sections to run a program which
reads a file using stdio built with --gc-sections, instead of simply
link the binary.  This catches the buggy linker as the garbage
collection removes a tag NetBSD uses to distinguish its own elf
binaries from Linux ones, causing it to treat conftest as a Linux
binary and run it with the wrong syscall table.

Fix make distcheck & installation of libevent 1 headers

libevent/Makefile.am corrects a typo (thanks to Harlan for spotting it
once we realized make distcheck was broken when building the libevent
tearoff).  The result was the include/ev*.h were not distributed nor
installed whether or not --disable-libevent-install was used.  This
was introduced with the final round (3/3) of
--disable-libevent-install patch from me.

Fix incorrect assertions and possible use-after-free in evrpc_free()

Original description:

  The following patch fixes incorrect assertions in evrpc_free():
  evrpc_unregister_rpc() and evrpc_remove_hook() return 0 for success.

  Also, in evrpc_unregister_rpc(), it is better to free RPC structure
  at the end: evrpc_free() uses rpc->uri as "name" parameter when
  calling evrpc_unregister_rpc(), then rpc->uri is freed, but we have
  "registered_uri = evrpc_construct_uri(name)". So at this time "name"
  is invalid.

Disable lazy gettimeofday/clock_gettime comparison for now

Merge branch '20_timescale' into patches-2.0

Followup for Tomash Brechko's http patch

This patch makes bufferevent_disable_hard() non-public, and
adds a comment about what it's for and why it's used.

Workaround libevent bug
https://sourceforge.net/tracker/index.php?func=detail&aid=3078187&group_id=50884&atid=461324

The problem is that bufferevent_disable() doesn't disable EV_WRITE
when 'connecting' flag is set.  However from evhttp_connection_reset()
we want to disable EV_WRITE for sure (we are closing the socket next).
So we add bufferevent_disable_hard(), which acts like
bufferevent_disable(), but resets 'connecting' flag before the call to
the actual handler.

TODO: bufferevent_disable_hard() shouldn't be public, remove it from
event2/bufferevent.h.

Have event_base_gettimeofday_cached() always return wall-clock time

Based on code by Dave Hart

Make --no-libevent-install apply to event1 headers too

Make --no-libevent-install apply to headers too

Continue AM_CFLAGS to AM_CPPFLAGS correction

9c469db300e1b270a93c6b04c1709ac0f7751136 had started with the conversion
but left out another instance of incorrectly used AM_CFLAGS.

Dave Hart provided the suggestion for this fix.

Merge branch '20_uri_nonconformant' into patches-2.0

Make -ffunction-sections off-by-default in 2.0.x

I don't see how this can burn anybody, but I don't want to take
chances: new build options are something that should be done in an
alpha.  To turn -ffunction-sections on, pass
--enable-function-sections to configure.

Remove nonfunctional enable_libevent_(install|regress)_def variables

Use $(), not @@, for LIBEVENT_GC_SECTIONS

Be explicit about how long event loops run in event.h documentation

Use %s with printf in test.sh

Correct an AM_CFLAGS to an AM_CPPFLAGS in test/Makefile.am

Add configure options to disable installation, regression tests

The main reason for disabling installation is if you're building
libevent as a subpackage for embedding: you want to have your main
package's "make all" build libevent, but you don't want your main
package's "make install" to install libevent.

Add -D_OSF_SOURCE to fix hpux builds

Use the gcc -ffunction-segments feature to allow gc when linking with static libevent

Check for getservbyname even if not on win32.

Fix autoconf bracket issues; make check for getaddrinfo include netdb.h

Skip check for zlib if we have no zlib.h

Make test-eof fail with a timeout if we never get an eof.

Previously, it would just wait forever.

FIX: comment internal headers used in sample code.

NM: Fix sample/http-server ipv6 fixes

Fix 1: Use #include "../xyz" rather than #include <../xyz>

Fix 2: util-internal.h includes ipv6-internal.h, which in turn defines
       a replacement AF_INET6 on platforms that don't have it.  As such,
       the AF_INET6 checks shouldn't be needed.  (right?)

Include util-internal.h as needed to build on platforms with no sockaddr_storage

Include arpa/inet.h as needed on HPUX

Correctly detect and stop non-chunked http requests when the body is too long

Based on analysis and code from Bas Verhoeven and from
Constantine Verutin.

fix spelling mistake in whatsnew-2.0.txt

Correct evhttp_del_accept_socket documentation on whether socket is closed

Thanks to Constantine Verutin for pointing this out.

Merge branch '20_evport_pollhup' into patches-2.0

Clarify event_set_mem_functions doc

Make URI parser able to tolerate nonconformant URIs.

If the EVHTTP_URI_NONCONFORMANT flag is passed in (which it is when
parsing URIs we get over the wire), then we relax our checks a lot.
Specifically, we do nothing to check for correct characters in the
path, query, and fragment parts of such a URI.

We could do much more here: we could relax our hostname requirements,
deal with spaces differently/better, trap some errors but not others,
etc.  But this should solve the worst user-agent compatibility issues
for now; the other issues can wait for a later release.

Fix a type error in our (unused) arc4random_stir()

Fix warnings about AC_LANG_PROGRAM usage

Autoconf 2.68 introduced some new warnings that were triggered during
autogen.sh. Fix those.

Fix test.sh output on solaris

Solaris echo -n doesn't omit newlines, but printf omits newlines in
more places.

Do not check for gethostbyname_r versions if we have getaddrinfo

Add compile-time check for AF_UNSPEC==PF_UNSPEC

Fix http unit test on non-windows platforms without getaddrinfo

Build on systems without AI_PASSIVE

Try to build correctly on platforms with no IPv6 support

Build correctly on platforms without sockaddr_storage

Fall back to sscanf if we have no other way to implement strtoll

Make --enable-gcc-warnings a no-op if not using gcc

Correctly free selectop fields when select_resize fails in select_init

Handle resize failures in the select backend better.

Use event_err() only if the failure is truly unrecoverable.

Detect and handle more allocation failures.

Check for POLLERR, POLLHUP and POLLNVAL for Solaris event ports

Fix compilation on Windows with NDEBUG

Dongsheng Song reports that when building on windows with NDEBUG, you
run into an attempt to do EVUTIL_ASSERT(x) where x is a bitfield,
which turns into _EVUTIL_NIL_CONDITION(x), which takes sizeof(x),
which is illegal.  This patch fixes _EVUTIL_NIL_CONDITION to work on
bitfields too.

Fix evport handling of POLLHUP and POLLERR

In other backends, they make _all_ events trigger; with evport they
previously triggered nothing.  Found by Phua Keat Yee.

Bump version to 2.0.10-stable-dev

Add changelog for 2.0.10-stable

Add more acknowledgements to the readme for 2.0.10-stable

Set the version to 2.0.10-stable

EVUTIL_ASSERT: Use sizeof() to avoid "unused variable" warnings.

Add stuff to whatsnew based on reading include changes since August.

Document that the cpu_hint is only used on Windows with IOCP for now

Consistentize tabs

Remove end-of-line whitespace

tweak readme, whatsnew

Fix a signed/unsigned comparison in the last commit

Reject overlong http requests early when Expect:100-continue is set

tests: Use relative includes ("") instead of system includes (<>)

tests: Use new event2 headers instead of old compatibility headers.

Use relative includes instead of system includes consistently.

bufferevent-internal.h: Use the new event2/util.h header, not evutil.h

Correctly notify the main thread when activating an event from a subthread

Minor fix for IOCP shutdown handling fix

Call setsockopt() on the actual new socket, not on as->s, which was
set to INVALID_SOCKET.

Increment version to 2.0.9-rc-dev

Give the correct date for 2.0.9-rc in the changelog

Increment version to 2.0.9-rc

Bump VERSION_INFO to 5:0:0

Changelog and acknowledgments for 2.0.9-rc

Fix win32 build of http-server.c with unicode enabled

Correctly detect timeouts during http connects

Try to fix an assertion failure related to close detection

f700566c removed a line from evhttp_connection_stop_detectclose that
cleared the EVHTTP_CON_CLOSEDETECT flag.  I think this was an
accident, and suspect that it may be the cause of bug 3069555.