https-client: correctly handle URLs with no path (like "https://host:port")

path == NULL check removed

Fixes: #233
Fixes: #234

sample/http-connect: don't use assert() to make it work with NDEBUG

Otherwise it will not work for cmake release target type.

test/regress_http: initialize "dns_base" to avoid reading trash

Even though it is very unlikely, but it will better to fix this.

sample/http-connect: made it compatible with C90

Merge branch 'reuse-conn_address-on-retry-v11'

There is regression tests and also this code worked/tested during crawling a
huge number of pages (billions).

* reuse-conn_address-on-retry-v11:
  be_sock: bufferevent_socket_set_conn_address(): assert instead of silent no-op
  http: reuse connected address only with EVHTTP_CON_REUSE_CONNECTED_ADDR
  be_sock: sanity check in bufferevent_socket_set_conn_address()
  be: replace sockaddr_storage with sockaddr_in6 for conn_address
  be: we don't need to use getpeername() we we have conn_address
  be: replace conn_address by full struct instead of pointer
  test/http: cover retrying with saved conn_address by shutting down dns server
  http: use IP address that we got before (if any) during retrying
  bufferevent: move conn_address out from http into bufferevent
  be: make @sa const for bufferevent_socket_connect()
  util: make @sa const for evutil_socket_connect_()

be_sock: bufferevent_socket_set_conn_address(): assert instead of silent no-op

http: reuse connected address only with EVHTTP_CON_REUSE_CONNECTED_ADDR

be_sock: sanity check in bufferevent_socket_set_conn_address()

be: replace sockaddr_storage with sockaddr_in6 for conn_address

We need only ipv6/ipv4 for connect in bufferevent, and since
sockaddr_storage is pretty big (128 bytes) it will be better to use
sockaddr_in6 here (it will fit ipv4 too).

be: we don't need to use getpeername() we we have conn_address

Since getpeername() could fail.

be: replace conn_address by full struct instead of pointer

test/http: cover retrying with saved conn_address by shutting down dns server

http: use IP address that we got before (if any) during retrying

Before this patch every time we are retrying our request we resolve
domain, but we could optimize this (since UDP is slow) by using cached
conn_address value, so do this.

bufferevent: move conn_address out from http into bufferevent

In http the only case when when we could store it is when we already
connected, *but* if we are doing request using domain name, then we need
to do request to nameserver to get IP address, and this is handled by
bufferevent.
So when we have IP address (from nameserver) and don't have connection
to this IP address, we could already cache it to avoid extra DNS
requests (since UDP is slow), and we can't do this from http layer, only
from bufferevent.

be: make @sa const for bufferevent_socket_connect()

util: make @sa const for evutil_socket_connect_()

sample: add HTTP CONNECT tunnelling example using libevent http layer

http: introduce evhttp_request_free_() helper

http: introduce evhttp_is_request_connection_close() helper

Updated gitignore

Merge pull request #190 from JohnOhl/evhttp-post-fix

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

Update dns-example.

There's somebody out there who is going to have a compiler from 1986 who will complain. Better to fix c99 now.

Merge pull request #183 from azat/dns-example

Some improvements for dns-example

Merge pull request #254 from zeliard/master

Fix bench_httpclient to support win32

Update bench_httpclient.c

*fixed: socket_geterror instead of WSAGetLastError

Merge pull request #253 from ellzey/master

Set correct socklen for PF_INET6 sockaddr len

*fix: bench_httpclient to support win32

Set correct socklen for PF_INET6 sockaddr len

Reported and patched by Pankaj Sharma

Merge pull request #251 from ellzey/21_err_on_thread_init_ordering

Debug mode option to error on evthread init AFTER other event calls.

Debug mode option to error on evthread init AFTER other event calls.

- A handy event_enable_debug_mode() feature which will error and abort the
  application if any thread-aware libevent functions are called BEFORE the
  evthread API has been initialized (manually, or through
  evthread_use_windows_threads() / evthread_use_pthreads()

- This is done by setting the global debug variable
  'event_debug_created_threadable_ctx_' whenever the following functions
  are called:

     evthreadimpl_lock_alloc_()
     evthreadimpl_cond_alloc_()
     event_base_new_with_config() <- this checks to see if the thread
                                     callbacks are enabled first, so we
                                     have to manually set the variable.

- Example:

int main(int argc, char ** argv) {
    struct event_base * base;

    event_enable_debug_mode();

    base = event_base_new();

    evthread_use_pthreads();

    return 0;
}

When executed, the program will throw an error and exit:

[err] evthread initialization must be called BEFORE anything else!

Merge pull request #245 from ellzey/21_size_and_misc_fixes

Fix garbage value in socketpair util function, stdint?

Merge pull request #239 from ellzey/issue237

Call underlying bev ctrl SET_FD on filtered bufferevents

Update README.md

Update README.md

Update README.md

Update travis config for status updates

Update README.md

Use autotools for appveyor until cmake is fixed.

Commented out a WIN32 threading / timing test for now

- seems as if windows has some time scale issues which I am looking
  into. For now I am commenting out the regression test until it is
  fixed.

Merge branch 'master' into 21_size_and_misc_fixes

Merge pull request #241 from ellzey/21_fix_appveyor_ssl_link

Fix download link for appveyor SSL

Fix the link for appveyor OpenSSL installer (WIN32)

* change and move the openssl self installer off to a site we own
   - the old link was 404, probably due to being replaced with a newer
     non-vuln version. But since we are only using this installer to
     auto-build with on appveyor (not as a release), then having a file
     we don't change and own seems to be a better solution.
* reduce verbosity

Merge branch 'master' into issue237

Fix garbage value in socketpair util function, stdint?

* Fixed an issue with evutil_ersatz_socketpair_, listen_addr could all
  be compared against with agarbage values. So just memset it before
  using it anywhere.

* Nick might punch me in the face, but if we have stdint.h; (as in
  EVENT__HAVE_STDINT_H is defined), might as well use those instead of
  the manual [U]INT[X}_MAX/MIN muck in there now.

Merge pull request #242 from ellzey/pullRequest118

expose bufferevent_incref/decref (with fewer modifications)

expose bufferevent_incref/decref (with fewer modifications)

Call underlying bev ctrl SET_FD on filtered bufferevents

If a bufferevent_filter is set on an underlying bufferevent which has
ctrl functions, bufferevent_filter needs to handle this.

For now I have added just BEV_CTRL_SET_FD, since this is needed for
bufferevent_sock to assign file descriptors to the proper
bufferevent_read/write callbacks.

A good example of the problem can be found in issue #237
https://github.com/libevent/libevent/issues/237

Fix potential fd leak in listener_read_cb()

As pointed out by harlan_ in #libevent after running a coverity sweep.
If the listener is free'd, 'new_fd' is never closed.

tab

fix the return value of event_deferred_cb_schedule_

Merge pull request #220 from miniupnp/fix_warnings

Fix warnings

Fix mixed declarations and code (forbidden by ISO C90)

Fix "function declaration isn’t a prototype"

add "void" to argument list

Merge pull request #214 from JoakimSoderberg/appveyor

Forgot to install OpenSSL for appveyor

Forgot to install OpenSSL for appveyor

Merge pull request #117 from JoakimSoderberg/appveyor

Add support for appveyor.com windows CI

Add support for appveyor.com windows CI

This does the same thing as Travis-CI but for windows.

@nmathewson
Go to: https://ci.appveyor.com/login -> Login using Github
Click **+New Project** -> Choose **Github** to the left -> Find **Libevent** in the list and click **Add**

Merge pull request #116 from JoakimSoderberg/fix_build_tree_cmake_config

This fixes a bug introduced in 27bd9faf498b91923296cc91643e03ec4055c230

This fixes a bug introduced in 27bd9faf498b91923296cc91643e03ec4055c230

CMake configuration files are intended to be used by other projects to find the library. Specifically the CMake find_package command can use it to find all files related to the project.

The idea is to support 2 different CMake configuration files for Libevent. One if you simply build libevent that is generated for the build tree.
And a second one that is generated for an install target that will be installed on the system and point to where on the system the lib files and such can be find.

So for instance, in the build tree the config would set the cmake variable `LIBEVENT_INCLUDE_DIRS` to `/path/to/libevent/build/include`.
And for the system config it would be set to `/usr/local/include` (or whatever target the user chose when running cmake).

27bd9faf498b91923296cc91643e03ec4055c230 changed this behavior so that both configs would point to the system wide path `/usr/local/include`

This meant that projects just wanting to import directly for the build tree would fail.

Merge remote-tracking branch 'azat/be-pair-fix-freeing-shared-lock-v5'

Merge pull request #207 from azat/avoid-leaking-of-event_debug_map_HT_GROW

Avoid leaking of event_debug_map_HT_GROW

Merge pull request #212 from xbao/master

changed strtotimeval signature to avoid conflict with Android NDK sys/time.h header (#211)

Merge remote-tracking branch 'public/master'

New function to get address for nameserver.

changed strtotimeval signature as per #211

Merge pull request #210 from mc-server/master

Added cmake-generated files to ignore list.

Added cmake-generated files to ignore list.

These are generated by cmake on Windows with MSVC.

be_pair/regress: cover use of shared lock (lock/unlock/free)

For more info look at 92a359ee3adf4636db508e6c6d7179d4d59eaafc
("be_pair: release shared lock with the latest of bufferevent_pair")

event_free_debug_globals_locks(): disable lock debugging

This will allow to use library event after
event_free_debug_globals_locks()/libevent_global_shutdown() without
invalid read/write's.

evthread: evthreadimpl_disable_lock_debugging_() for libevent_global_shutdown()

evthread: add evthread_get_{lock,condition}_callbacks() helpers

be_pair: release shared lock with the latest of bufferevent_pair

Then next code sample will use free'd lock:
  evthread_use_pthreads();
  ...
  assert(!bufferevent_pair_new(base, BEV_OPT_THREADSAFE, pair));
  ...
  bufferevent_free(pair[0]); # refcnt == 0 -> unlink
  bufferevent_free(pair[1]); # refcnt == 0 -> unlink
  ...
  event_base_free() -> finalizers -> EVTHREAD_FREE_LOCK(bev1->lock)
                                  -> BEV_LOCK(bev2->lock) <-- *already freed*

While if you will reverse the order:
  bufferevent_free(pair[1]); # refcnt == 0 -> unlink
  bufferevent_free(pair[0]); # refcnt == 0 -> unlink
  ...
  event_base_free() -> finalizers -> BEV_LOCK(bev2->lock)/!own_lock/BEV_UNLOCK(bev2->lock)
                                  -> EVTHREAD_FREE_LOCK(bev1->lock) (own_lock)

It is ok now, but I guess that it will be better to relax order of
freeing pairs.

Merge pull request #115 from jer-gentoo/master

EVBUFFER_PTR_SET -> EVBUFFER_PTR_ADD

EVBUFFER_PTR_SET -> EVBUFFER_PTR_ADD

Looks like EVBUFFER_PTR_ADD should have been used instead of EVBUFFER_PTR_SET.

regress_dns: drop hack for event_debug_map_HT_GROW in leak tests

event: call event_disable_debug_mode() in libevent_global_shutdown()

This will avoid leaking of event_debug_map_HT_GROW

I buildin it into libevent_glboal_shutdown() because
event_disable_debug_mode() -> event_free_debug_globals() ->
event_free_debug_globals_locks() will clean event_debug_map_lock_ that
used in event_disable_debug_mode().

ht-internal: don't reset hth_table_length explicitly in name_##HT_CLEAR

name_##HT_CLEAR calls name_##HT_INIT that reset hth_table_length to 0.

Update changelog

Merge branch '21_cve_2014_6272_v2'

Fix CVE-2014-6272 in Libevent 2.1

For this fix, we need to make sure that passing too-large inputs to
the evbuffer functions can't make us do bad things with the heap.

Also, lower the maximum chunk size to the lower of off_t, size_t maximum.

This is necessary since otherwise we could get into an infinite loop
if we make a chunk that 'misalign' cannot index into.

Increment version to 2.1.5-beta

work on changelog for 2.1.5-alpah

Merge pull request #201 from nsuke/cmake-shared-build

Fix CMake shared library build

Fix CMake shared library build

This fixes following problems in shared library build:
* visibility=hidden was not enabled for gcc because of incorrect variable name
* test programs that need internal APIs caused link errors

Merge pull request #197 from JohnOhl/fix-clang-compile-warnings

Fix warnings when compiling with clang 3.5

Fix warnings when compiling with clang 3.5

Fix annoying heisenbug in test-time.c

Merge pull request #192 from miniupnp/master

Fix mixed declarations and code (forbidden by ISO C90)

Fix mixed declarations and code (forbidden by ISO C90)

Implement new/free for struct evutil_monotonic_timer and export monotonic time functions

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

Fix evbuffer_peek() with len==-1 and start_at non-NULL.

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

Conflicts:
ChangeLog
event.c

Merge branch 'patches-2.0' of github.com:libevent/libevent into patches-2.0

Start on 2.1.5-alpha/beta changelog

Work on the changelog for 2.0.22

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

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

Fix race caused by event_active

There is a race between manual event_active and natural event activation. If both happen at the same time on the same FD, they would both be protected by the same event base lock except for 1 LoC where the fields of struct event are read without any kind of lock. This commit does those reads into local variables inside the lock and then invokes the callback with those local arguments outside the lock. In 2.0-stable, none of this is inside the lock; in HEAD, only the callback is read inside the lock. This gets the callback and all 3 arguments inside the lock before calling it outside the lock.

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