evhttp: Fix failure to send all output data for POST/PUT requests

Merge pull request #179 from pysiak/master

Provide support for SO_REUSEPORT through LEV_OPT_REUSABLE_PORT

Provide support for SO_REUSEPORT through LEV_OPT_REUSABLE_PORT

Merge pull request #174 from azat/dns-fail-disable-when-inactive-fix-v3

Dns fail disable when inactive fix v3

Merge pull request #171 from azat/bufferevent-openssl-fixes-v4

bufferevent_openssl: reset fd_is_set when setfd with -1 is called

Merge pull request #170 from azat/https-client-retries

https-client: add -retries argument, for connection retries

Merge pull request #108 from JoakimSoderberg/minor_issues

Fix compilation for older OpenSSL versions.

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

Fix a typo in a doxygen comment. Reported by 亦得.

evdns: fail ns after we are failing/retrasmitting request

In case we are failing request (evdns_request_timeout_callback()), we
delete timeout_event in request_finished(), while just before calling
request_finished() (for failing request) there was a call to
nameserver_failed(), that add event for timeout_event, IOW we must fail
ns after request because otherwise we will not have timeout_event
actived, and we will waiting forever.

Before this patch the dns/retry_disable_when_inactive will wait forever,
after - OK.

evdns: add retry/reissue tests for EVDNS_BASE_DISABLE_WHEN_INACTIVE

evdns: fix EVDNS_BASE_DISABLE_WHEN_INACTIVE in case retransmit/retry

Fix compilation for older OpenSSL versions.

For instance OpenSSL 0.9.7 does not have SSL_set_tlsext_host_name.

Also add the missing EVENT__DISABLE_SAMPLES CMake option.

bufferevent_openssl: reset fd_is_set when setfd with -1 is called

Otherwise we will use old fd after close(2) called on it:
(Traces trimmed, to minimize it)
$ strace -keclose,open,socket,epoll_ctl https-client -url https://libevent.org
socket(PF_INET, SOCK_STREAM|SOCK_NONBLOCK, IPPROTO_IP) = 5
 > /lib/x86_64-linux-gnu/libc-2.19.so(socket+0x7) [0xe6da7]
 > /usr/lib/libevent-2.1.so.4.0.0(evutil_socket_+0x25) [0x27835]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0xe9) [0x1b1a9]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0x21f) [0x1b2df]
 > /usr/lib/libevent-2.1.so.4.0.0(evutil_getaddrinfo_async_+0x65) [0x271a5]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect_hostname+0x106) [0x1b416]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0xe7) [0x39f07]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_make_request+0xb8) [0x3a218]
epoll_ctl(3, EPOLL_CTL_ADD, 5, {EPOLLIN, {u32=5, u64=5}}) = 0
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xfdd) [0x2ced]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_setfd+0x3b) [0x16d9b]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0x75) [0x1b135]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0x21f) [0x1b2df]
 > /usr/lib/libevent-2.1.so.4.0.0(evutil_getaddrinfo_async_+0x65) [0x271a5]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect_hostname+0x106) [0x1b416]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0xe7) [0x39f07]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_make_request+0xb8) [0x3a218]
epoll_ctl(3, EPOLL_CTL_MOD, 5, {EPOLLIN|EPOLLOUT, {u32=5, u64=5}}) = 0
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xfee) [0x2cfe]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_setfd+0x3b) [0x16d9b]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0x75) [0x1b135]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect+0x21f) [0x1b2df]
 > /usr/lib/libevent-2.1.so.4.0.0(evutil_getaddrinfo_async_+0x65) [0x271a5]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_socket_connect_hostname+0x106) [0x1b416]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0xe7) [0x39f07]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_make_request+0xb8) [0x3a218]
epoll_ctl(3, EPOLL_CTL_MOD, 5, {EPOLLOUT, {u32=5, u64=5}}) = 0
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xbbc) [0x28cc]
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xd96) [0x2aa6]
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xeb8) [0x2bc8]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x3fe) [0x1fd1e]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
epoll_ctl(3, EPOLL_CTL_DEL, 5, 7fffa1f841e0) = 0
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xbc4) [0x28d4]
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xd96) [0x2aa6]
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xeb8) [0x2bc8]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x3fe) [0x1fd1e]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
close(5)                                = 0
 > /lib/x86_64-linux-gnu/libpthread-2.19.so(__close_nocancel+0x7) [0xeb20]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_reset_+0x55) [0x373f5]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0x1d) [0x39e3d]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x3fe) [0x1fd1e]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
epoll_ctl(3, EPOLL_CTL_ADD, 5, {EPOLLIN, {u32=5, u64=5}}) = -1 EBADF (Bad file descriptor)
) = -1 EBADF (Bad file descriptor)
/src/oss/strace-code/strace: Can't initiate libunwind: No such process
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xfdd) [0x2ced]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_setfd+0x3b) [0x16d9b]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0x14c) [0x39f6c]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x3fe) [0x1fd1e]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
/src/oss/strace-code/strace: Exit of unknown pid 28185 seen
[warn] Epoll ADD(1) on fd 5 failed.  Old events were 0; read change was 1 (add); write change was 0 (none); close change was 0 (none): Bad file descriptor
epoll_ctl(3, EPOLL_CTL_ADD, 5, {EPOLLOUT, {u32=5, u64=5}}) = -1 EBADF (Bad file descriptor)
) = -1 EBADF (Bad file descriptor)
/src/oss/strace-code/strace: Can't initiate libunwind: No such process
 > ...
 > /usr/lib/libevent_openssl-2.1.so.4.0.0(_init+0xfee) [0x2cfe]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_setfd+0x3b) [0x16d9b]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_connect_+0x14c) [0x39f6c]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x3fe) [0x1fd1e]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
/src/oss/strace-code/strace: Exit of unknown pid 28186 seen
[warn] Epoll ADD(4) on fd 5 failed.  Old events were 0; read change was 0 (none); write change was 1 (add); close change was 0 (none): Bad file descriptor
<getaddrinfo traces trimmed>
close(5)                                = -1 EBADF (Bad file descriptor)
)                                = -1 EBADF (Bad file descriptor)
/src/oss/strace-code/strace: Can't initiate libunwind: No such process
 > /lib/x86_64-linux-gnu/libpthread-2.19.so(__close_nocancel+0x7) [0xeb20]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_reset_+0x55) [0x373f5]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_fail_+0xc1) [0x3aed1]
 > /usr/lib/libevent-2.1.so.4.0.0(evhttp_connection_fail_+0x4a3) [0x3b2b3]
 > /usr/lib/libevent-2.1.so.4.0.0(bufferevent_enable+0x192) [0x16722]
 > /usr/lib/libevent-2.1.so.4.0.0(event_free+0x5df) [0x1feff]
 > /usr/lib/libevent-2.1.so.4.0.0(event_base_loop+0x407) [0x20677]
/src/oss/strace-code/strace: Exit of unknown pid 28187 seen
some request failed - no idea which one though!
socket error = Bad file descriptor (9)

Before this patch:
$ sample/https-client -retries 1 -url https://libevent.org
[warn] Epoll ADD(1) on fd 6 failed.  Old events were 0; read change was 1 (add); write change was 0 (none); close change was 0 (none): Bad file descriptor
[warn] Epoll ADD(4) on fd 6 failed.  Old events were 0; read change was 0 (none); write change was 1 (add); close change was 0 (none): Bad file descriptor
some request failed - no idea which one though!
socket error = Bad file descriptor (9)

After this patch:
$ sample/https-client -retries 1 -url https://libevent.org
Response line: 0 (null)

Reported-by: gerkenjohannes@web.de

https-client: add -retries argument, for connection retries

Using evhttp_connection_set_retries() API.

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

[Bugfix] fix bufferevent setwatermark suspend_read

Fix several memory leaks in the unit tests.

Also add a comment to buffer.c about why we call
evbuffer_file_segment_free on failure to add the segment.

Fix for a677b72bd6d9d378daf9775966aa8ca574e26e67

Make a buffer larger in the tests to avoid a scary evbuffer_copyout_from()

The call was safe, but coverity couldn't tell. CID 1239294.

Use a more precise calculation for max in time-ratelim.c

CID 1239297

Use evutil_weakrand() in unit tests.

(Coverity doesn't like random() or rand().  We don't care; this is
for unit tests.)

Fixes CID 1239298, 1239296, 1239295, 1239293.

Fix a c90 warning

Fix a dead-code warning in unit tests. CID 1193548

Fix a use-after-free in unit tests. CID 752027

Merge pull request #168 from ufo2243/master

make bufferevent_getwatermark api more robust

Merge pull request #119 from ldx/winhosts

Fix hosts file use on Windows.

Fix use-after-free error in EV_CLOSURE_EVENT callback

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

regress_buffer: fix 'memcmp' compare size

Merge pull request #118 from azat/http-forward-family-to-bufferevent

Add evhttp_connection_set_family() to set addrinfo->family for DNS requests

use correct tt macro for pointer compare

Merge pull request #128 from azat/tests-disable-when-inactive-v5

Some tests for EVDNS_BASE_DISABLE_WHEN_INACTIVE flag

Merge pull request #129 from azat/disable-probing-with-disable-when-inactive-v2

evdns: disable probing with EVDNS_BASE_DISABLE_WHEN_INACTIVE

make bufferevent_getwatermark api more robust

Merge pull request #167 from snikulov/pr_libevent_cmake_upd

updates in cmake build

Merge pull request #103 from ufo2243/master

[Bugfix] fix grammer error

[Bugfix] fix grammer error

updates in cmake build

Correctly handle allocation failures in evdns_getaddrinfo

Kuldeep Gupta appears to have used some kind of automated tool to
find this and generate a patch for half the issue as #149.

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

Consistently check for failure from evbuffer_pullup()

Closes issue #148.

Merge pull request #153 from azat/tests-simplestsignal

test/regress: add simplestsignal: to track reorder bugs separately

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

Conflicts:
sample/http-server.c

Use FindClose for handle from FindFirstFile in http-server.c

Noted by Miles Chan as issue #145

Change return type of evutil_load_windows_system_library_ to HMODULE

Noted by Miles Chan as issue #146

Fix issue #127, double free for filterevents that use BEV_OPT_CLOSE_ON_FREE

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

Add a few files created by "make verify" to .gitignore.

test/regress: add simplestsignal: to track reorder bugs separately

Merge pull request #140 from vapier/ignore

ignore config.cache/test-driver files

Merge pull request #143 from vapier/disable-sample

add a --disable-samples configure flag

add a --disable-samples configure flag

For people building & installing libevent in a distro, building all these
sample apps are not useful.  Add a flag to turn them off.

ignore config.cache/test-driver files

config.cache is generated when you run `./configure -C`.

test-driver comes from newer autotools.

Handle ENOTCAPABLE from FreeBSD - this is returned if an event in the
changelist is for an FD that has been closed.

Make http-server.c output into good html5

Update to a more recent tinytest_macros.

This one has less of a tendency to suppress important type warnings.

Merge pull request #132 from credil/master

ignore one more test binary

ignore one more test binary

regress dns: drop alarm() (since windows don't have it)

evdns: we don't need to add retry timer in transmit_to it will be done on fail

evdns: add regress test for flag EVDNS_BASE_DISABLE_WHEN_INACTIVE without ns

evdns: add regress for EVDNS_BASE_DISABLE_WHEN_INACTIVE base flag

evdns: disable probing with EVDNS_BASE_DISABLE_WHEN_INACTIVE

When user install EVDNS_BASE_DISABLE_WHEN_INACTIVE flag for evdns base,
we must remove the timer that is used for probing, if current dns server
failed, otherwise it won't break the loop.

Merge pull request #125 from azat/tree.h-drop-copy

tree.h: drop duplicated content of tree.h

tree.h: drop duplicated content of tree.h

Introduced in 9859bc783adcd51729b3d7563902ac8caca67b42
("r15764@tombo:  nickm | 2007-12-31 15:46:16 -0500")

test/http: add regress test for set family to AF_INET6

test: add regress for evhttp_connection_set_family() with AF_INET and AF_UNSPEC

test: add family argument for http_connection_test_()

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

Fix a crash in evdns related to shutting down evdns

(Improved version to deal correctly with probe requests)

Patch from YASUOKA Masahiko; fix for libevent github issue #113.

Merge pull request #121 from azat/dns-regress-leaks

regress_dns: fix leaks in getaddrinfo_async{,_cancel_stress} tests

Merge pull request #122 from trondn/missing-file

Add missing headerfile for cmake

Add missing headerfile for cmake

regress_dns: fix leaks in getaddrinfo_async{,_cancel_stress} tests

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

Fix a crash in evdns related to shutting down evdns

Patch from YASUOKA Masahiko; fix for libevent github issue #113.

Increment version to 2.1.4-alpha-dev

Update release date in changelog

Merge pull request #116 from trondn/cmake-build-fixes

Cmake build fixes

Don't truncate hosts file path on Windows.

Since evutil_snprintf() (actually evutil_vsnprintf() called by it) will
make sure the buffer is null-terminated by placing a null byte at
len_out - 1, we need to pass the full length of the buffer; otherwise
the path will end in "\\host" instead of "\\hosts".

Load hosts file on Windows.

Hosts files are not loaded in evdns_base_config_windows_nameservers() if
load_nameservers_with_getnetworkparams() succeeds on Windows. Parse and
load it first before setting up nameservers.

Add evhttp_connection_set_family() to set addrinfo->family for DNS requests

This is useful if you want to avoid extra dns requests.

Fixup make install for cmake projects

Install all of the public headers and put the dll's in the
same directory as the binaries.

Renamed sin to saddr due to name conflict

During building on MSVC 2013 I got a compiler error by a
type conflict for sin:

test-fdleak.c(60) : error C2365: 'sin' : redefinition; previous definition was 'function'
test-fdleak.c(134) : error C2070: 'double (__cdecl *)()': illegal sizeof operand
test-fdleak.c(134) : error C2198: 'evconnlistener_new_bind' : too few arguments for call
test-fdleak.c(148) : error C2070: 'double (__cdecl *)()': illegal sizeof operand
test-fdleak.c(148) : error C2168: 'memcpy' : too few actual parameters for intrinsic function
test-fdleak.c(149) : error C2224: left of '.sin_family' must have struct/union type
test-fdleak.c(212) : error C2070: 'double (__cdecl *)()': illegal sizeof operand
test-fdleak.c(212) : error C2198: 'bufferevent_socket_connect' : too few arguments for call
test-fdleak.c(239) : error C2070: 'double (__cdecl *)()': illegal sizeof operand
test-fdleak.c(239) : error C2168: 'memset' : too few actual parameters for intrinsic function
test-fdleak.c(240) : error C2224: left of '.sin_family' must have struct/union type
test-fdleak.c(241) : error C2224: left of '.sin_addr' must have struct/union type
test-fdleak.c(242) : error C2224: left of '.sin_port' must have struct/union type

The simplest solution to this problem would be to rename
the variable.

Correctly skip ipv6 http test on systems without ipv6

Remove integer-overflow unit tests

There's not much point checking undefined behavior.

evtag: detect tags over 32-bits earlier

Catch over-large port numbers early in http

Otherwise integer overflow potentially turns the port number into garbage.

Fix ubsan warnings when parsing ipv4/ipv6 addrs

left-shifting a one-byte integer by 24 invokes undefined behavior.
Let's not do that.

Fix a use-after-free error on EV_CLOSURE_EVENT_FINALIZE callbacks

After running the callback, we were checking evcb->evcb_closure to
decide whether to call mm_free(ev).  But the callback itself might
have freed ev, so we need to grab that field first

Found with AddressSanitizer

Fix an illegal read error in the evbuffer_add_reference tests

Found with AddressSanitizer

Add new APIs to whatsnew-2.1

Increment version to 2.1.4-alpha

Bring changelog up to date for 2.1.4-alpha

Avoid double-close paths in http tests

Coverity spotted that there were some paths through the test
functions that would double-close some sockets.

Add missing include to regress_finalize.h

Skip http/ipv6_for_domain test when we have no ipv6 support

Add an include to evrpc-internal to fix openbsd compilation warning

Heap-allocate zlib data structure in regress_zlib tests