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 $<

Make --disable-libevent-regress work again

Double-check next timeout when adding events

When resuming the system from a suspended state, the ev_timeout field
of a scheduled timer event may be in the past. This leads to
unexpected behavior when scheduling a short-duration timer event
immediately after returning from suspension, because the new event
does not land on top of the timeout minheap and so the event loop
(blocked on a possibly long-duration timeout) is not notified.

This patch checks for this condition and, if it obtains, notifies the
event loop.

Test: decoding just part of string with evhttp_decode_uri_internal()

Move prototype of evhttp_decode_uri_internal() to http-internal.h

Make it non static, that can be called from tests

uri decode: changed the test for the existence of the next character

Fix for 64b6eceaba1a4

More info here
https://github.com/azat/libevent/commit/64b6eceaba1a40ab0b175fa9fd9329d3e978ce6e#commitcomment-2714685

uri decode: fix for warning "use of uninitialised value"

This patch add check in evhttp_decode_uri_internal() that next 2 symbols
are exists in array of chars for decoding, if don't have two next 2
symbols don't try to decode '%FF'

pull in wildcard matching code from cURL

Now, https-client accepts both:

https://ip.appspot.com/ (matching wildcard certificate)
https://github.com/     (matching non-wildcard certificate)

but still rejects

https://www.kegel.com/  (non-matching wildcard certificate)

which should match the behavior of these sites in a web browser.

avoid sign mismatch warning in openssl_hostname_validation.c

sample/openssl_hostname_validation.c: In function 'matches_common_name':
sample/openssl_hostname_validation.c:80: warning: comparison between signed and unsigned integer expressions
sample/openssl_hostname_validation.c: In function 'matches_subject_alternative_name':
sample/openssl_hostname_validation.c:124: warning: comparison between signed and unsigned integer expressions

use iSECPartners code to validate hostname in certificate

The problem is that if you go to a website whose certificate does not
match its hostname, it should fail.  Try this in a web browser for
https://www.kegel.com/ for example.  Your web browser will say the
certificate is for *.pair.com, not for www.kegel.com, and won't let
you visit it without clicking through a bunch of scary warnings.

However, prior to this commit, https-client was happy to fetch
https://www.kegel.com/ without complaining.  That is bad.  Now, with
this commit, it will properly complain, which is good:

pelletier@chives:~/src/libevent/sample$ ./https-client https://www.kegel.com/
Got 'MatchNotFound' for hostname 'www.kegel.com' and certificate:
/C=US/postalCode=15203/ST=Pennsylvania/L=Pittsburgh/street=Suite 210/street=2403 Sidney Street/O=pair Networks, Inc./OU=Provided by pair Networks, Inc./OU=PairWildcardSSL $250,000/CN=*.pair.com
some request failed - no idea which one though!
error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
ppelletier@chives:~/src/libevent/sample$

It will still succeed for sites with an exactly-matching certificate,
such as https://github.com/ and that is also good!

However, the problem is that the iSECPartners code doesn't handle
wildcards, which means we reject https://ip.appspot.com/ even though
it is perfectly legitimate, because we don't understand the wildcard:

ppelletier@chives:~/src/libevent/sample$ ./https-client https://ip.appspot.com/
Got 'MatchNotFound' for hostname 'ip.appspot.com' and certificate:
/C=US/ST=California/L=Mountain View/O=Google Inc/CN=*.appspot.com
some request failed - no idea which one though!
error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
ppelletier@chives:~/src/libevent/sample$

So, we need to fix this.  In other words, "to be continued..."

use Debian's default root certificate location

as suggested here:
http://archives.seul.org/libevent/users/Feb-2013/msg00034.html

although curl's acinclude.m4 reveals many other possibilities:

dnl /etc/ssl/certs/ca-certificates.crt Debian systems
dnl /etc/pki/tls/certs/ca-bundle.crt Redhat and Mandriva
dnl /usr/share/ssl/certs/ca-bundle.crt old(er) Redhat
dnl /usr/local/share/certs/ca-root.crt FreeBSD
dnl /etc/ssl/cert.pem OpenBSD
dnl /etc/ssl/certs/ (ca path) SUSE

And none of these cover Windows :(

better handling of OpenSSL errors

https-client was putting newlines at 256-byte boundaries

presumably this was meant to put a ">" before every line, but that
isn't what it does, since evbuffer_remove is simply returning
fixed-size chunks.  So, when retrieving a document of more than 256
bytes (e. g. any nontrivial document), we got "> " and newlines thrown
in at very arbitrary places.

use ${OPENSSL_LIBS} instead of -lssl -lcrypto

This made the difference between segfaulting and not segfaulting for
me when I run https-client, when I've built libevent using an OpenSSL
in a non-standard location.

In the same spirit as 1d9d5110a4aebf5833f6fd78bd0252affde0f4d0 and
d70af27d0152d0a87a25127faf215604beb8ffe0.

Add sample/https-client.c, an example of stacking evhttp as a client on top of bufferevent_ssl.

Signed-off-by: Catalin Patulea <catalinp@google.com>

Merge pull request #41 from ppelleti/winsock-errs

FormatMessage for winsock errors

test filling up the hash table a bit

use hashtable instead of linked list to cache winsock errors

as discussed here:
https://github.com/libevent/libevent/pull/41#issuecomment-13611817

make sure caching works, and we don't leak memory

use FormatMessage for winsock errors

as discussed here:
http://archives.seul.org/libevent/users/Feb-2013/msg00004.html

a program to print out the error strings for winsock errors

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

fix #73 and fix http_connection_fail_test to catch it

Unit test for event_remove_timer with EV_PERSIST.

Patch from dcicppin on sourceforge.

Make event_remove_timer behave correctly with persistent timers

Merge remote-tracking branch 'ppelleti/nmake-clean-exes'

Avoid using $(top_srcdir) in TESTS.

Newer automakes don't like this.

Use AC_CONFIG_HEADERS in place of AM_CONFIG_HEADERS for autmake 1.13 compat

Patch from cazfi.

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

This is an "ours" commit: I'm not taking the recent autotools changes
from patches-2.0, since they'd conflict with master.

Avoid using top_srcdir in TESTS-new automakes do not like this

Rename configure.in to configure.ac to appease newer autoconfs

Use AC_CONFIG_HEADERS in place of AM_CONFIG_HEADERS for autmake 1.13 compat

Patch from cazfi.