test/http: turn off some tests that based on backlog filling (falky)

Since this technique does not work well everywhere, anyway it fails from time
to time.

See: https://travis-ci.org/libevent/libevent/jobs/186426446

Update README.md

The default option of  EVENT__DISABLE_OPENSSL:BOOL is `OFF`

test: fix util/date_rfc1123 under win32

Refs: #417
v2: check when gmtime() returns NULL
v3: fix overflow cases

Fix UB in evutil_date_rfc1123()

As pointed in https://github.com/libevent/libevent/pull/417#issuecomment-267860738
  "code is unsafe because in evutil_date_rfc1123() the pointer to the
  automatic variable struct tm cur is used outside the scope it defined."

Checked with `clang -fsanitize=address -fsanitize-address-use-after-scope`
and test that call evutil_date_rfc1123() with tm==NULL

Merge branch 'date-rfc1123'

Follow RFC2616 (Date header should be in RFC1123 format).

But to reproduce this bug, program that uses libevent http-server should
call `setlocale(LC_ALL, "")` to properly initliaze locale, so that
strftime() retruns locale-specific dates.

* date-rfc1123:
  add tests for evutil_date_rfc1123().
  http: do not use local settings for Date header

Closes: #417

add tests for evutil_date_rfc1123().

http: do not use local settings for Date header

Fix incorrect MIME type

be_openssl: make be_openssl_set_fd() static (no prototype required)

Merge branch 'be-openssl-fd-reset-fix-v2'

Two issues:
- dirty_shutdown for openssl 1.1
- BEV_CTRL_SET_FD for bufferevent_openssl didn't reset state

* be-openssl-fd-reset-fix-v2:
  Fix dirty_shutdown for openssl 1.1
  Fix reusing bufferevent_openssl after fd was reseted (i.e. on new connection)
  test/https: fix ssl dirty bypass for https_simple
  test/https: cover multiple request over the same connection
  test/http: sanity check for http_request_empty_done()

Reported-by: liutao74748@163.com
ML: http://archives.seul.org/libevent/users/Nov-2016/msg00041.html

Fix dirty_shutdown for openssl 1.1

SSL_read() returns -1, even when underlying read() return 0

Fix reusing bufferevent_openssl after fd was reseted (i.e. on new connection)

For example if you trying to issue multiple requests over the same
evhttp_conneciton, and if connection already closed (IOW it should be
re-connected), than you will get into trouble since it will got wrong
openssl state. This patch addresses this issue by restoring state to
initial if SETFD called with -1 fd.

test/https: fix ssl dirty bypass for https_simple

Tests:
- http/https_simple_dirty # not affected, since dirty is the default
- http/https_simple       # affected

v2: fix compilation with -DEVENT__DISABLE_OPENSSL=ON

test/https: cover multiple request over the same connection

Introduce two new tests:
- http/https_connection         # fail
- http/https_persist_connection # ok

Reported-by: liutao74748@163.com

test/http: sanity check for http_request_empty_done()

Merge branch 'openssl-1.1-init-fixes-v2'

* openssl-1.1-init-fixes-v2:
  sample/le-proxy: use TLS_method instead of SSLv23_method (latest deprecated)
  test: fix building under openssl 1.1 (init functions has been deprecated)
  le-proxy: fix building under openssl 1.1 (init functions has been deprecated)

Reported-by: varun.net@gmail.com
ML: http://archives.seul.org/libevent/users/Nov-2016/msg00002.html

sample/le-proxy: use TLS_method instead of SSLv23_method (latest deprecated)

test: fix building under openssl 1.1 (init functions has been deprecated)

Refs: #397

le-proxy: fix building under openssl 1.1 (init functions has been deprecated)

Refs: #397

https-cilent: fix compilation warnings about unused vars/functions on win32

Fix _FILE_OFFSET_BITS redinition (solaris/autotools)

So firstly include our header (config.h) -- <evconfig-private.h>, and
only after it <sys/types.h> since latest has #ifdef guard, while our
config.h is not inteded for this.

And besides all this thing with LARGE_FILE is a abit awkward, since we
don't nefine _LP64/_LP32 anyway, and so we have next error actually (64bit VS
32bit):
==> solaris: In file included from ./util-internal.h:30:0,
==> solaris:                  from test/regress_ssl.c:49:
  ==> solaris: ./evconfig-private.h:29:0: warning: "_FILE_OFFSET_BITS" redefined
  ==> solaris:  #define _FILE_OFFSET_BITS 64
  ==> solaris:  ^
  ==> solaris: In file included from /usr/include/sys/types.h:17:0,
  ==> solaris:                  from test/regress_ssl.c:38:
  ==> solaris: /opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/sys/feature_tests.h:196:0: note: this is the location of the previous definition
  ==> solaris:  #define _FILE_OFFSET_BITS 32
  ==> solaris:  ^

For cmake it commented in: 8b228e27f57300be61b57a41a2ec8666b726dc34
("Lot's of cmake updates")

vagrant/osx: use make instead of gmake (there is no gmake)

$ make --version
GNU Make 3.81
Copyright (C) 2006  Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
PARTICULAR PURPOSE.

This program built for i386-apple-darwin11.3.0

util-internal: fix __func__ redefinition (netbsd)

==> netbsd: In file included from ../listener.c:57:0:
==> netbsd: ../util-internal.h:58:0: warning: "__func__" redefined [enabled by default]
==> netbsd:  #define __func__ EVENT____func__
==> netbsd:  ^
==> netbsd: In file included from /usr/include/amd64/types.h:39:0,
==> netbsd:                  from /usr/include/sys/types.h:45,
==> netbsd:                  from ../listener.c:30:
==> netbsd: /usr/include/sys/cdefs.h:394:0: note: this is the location of the previous definition
==> netbsd:  #define __func__ __PRETTY_FUNCTION__

Fix signedness differ for iov_base (solaris)

evutil_time: include <unistd.h> when there is only sleep()/usleep()

http: fix formatter for pritnf for req->ntoread (osx)

sample/http-connect: fix compilation with VS2010

Fixes: nmathewson/Libevent#137

C90 doesn't like declarations intermingled with statements

So move all of the declarations to the top of the offending function.

This patch includes both of issues (Fixes:), from @jeking3 and
@pprindeville

Fixes: #418
Fixes: nmathewson/Libevent#136

vagrant: add centos box

Packages list get from here:
  https://pkgs.org/search/

Refs: #424

Merge remote-tracking branch 'azat/release-2.1.7-rc-pull'

Finally 2.1 release candidate is out!

* azat/release-2.1.7-rc-pull:
  Bump version to 2.1.7-beta everywhere
  Update changelog for 2.1.7-rc
  README: update AUTHORS

Bump version to 2.1.7-beta everywhere

- cmake basic
- cmake detecting from git -- event_fuzzy_version_from_git()
- autotools
- win32
- appveyor

Update changelog for 2.1.7-rc

README: update AUTHORS

$ git log --format='%an' HEAD...refs/tags/release-2.1.6-beta | sort -u | xargs -n1 -i bash -c 'fgrep -q "{}" README.md || echo {}'

travis-ci/osx: install lcov

test/regress: cover a polling of invalid fd

Test that an event's callback is called if the fd is closed prior to being
polled for activity.

azat: make it run only for poll backend/method, and do not close fd
twice

poll: Prevent libevent from spinning if POLLNVAL occurs

This can happen, for example if libevent is being used to poll fds given
by another library where the other library closes the fds without
notifying the program using it that said fds were closed. In this case,
libevent will simply spin on poll() since there are active fds, but
won't call any event callback to handle the condition.

In epoll case after socket closed it automatically removed from epfd, so
IOW it will not spin in epoll* API, just a timeout.

Fixes: #379

Use coveralls.io via travis

Fix cmake -DEVENT__COVERAGE=ON

- do not use compiler check from the root cmake rules with syntax error,
  let CodeCoverage check it
- fix CodeCoverage to check C compiler not CXX
- case insensitive checking of CMAKE_BUILD_TYPE
- replace flags with --coverage, and fix flags with linking with
  --coverate, otherwise it will not compile during checking flags and
  fails.

Testing with vagrant for 6 OS and cmake+autoconf

OS:
- linux
- freebsd
- osx
- netbsd
- solaris
- win

And mostly they all are more or less fine, except for timing issues

Also it supports next environment variables:
- NO_PKG
- NO_CMAKE
- NO_AUTOTOOLS

For instance if you run ubuntu box seocnd time, then you can run with:
$ NO_PKG=true vagrant provision ubuntu

autogen.sh: learn about gmake

Fixes: 69cce257eb6183dda114d8f5b8ca0325cfabe2e1
("69cce257eb6183dda114d8f5b8ca0325cfabe2e1")

autogen.sh: remove all autoconf/automake caches, if any

Otherwise if you for example share root of libevent repository with some
vm (for example windows+cygwin) then if you try to build with configure
on host, and then on guest it will fail, so let's just ignore this since
autogen.sh is just for this -- cleaning all stuff.

cmake: fix finding python2, and check that it is really 2

python version check removed in a4d044c0cd97f68b4b3dde414b83d609719d33d7
("cmake: use PYTHON_EXECUTABLE to find python2"), but in
bcb990ab7cf4921c2c30a1aa4fcb4c0131fffe94 ("cmake/win32: fix running
regress, but fixing finding python2 interpreter") we set
PYTHON_EXECUTABLE only if it exists.

Ignore temporary configure files

So if you kill configure, git still should ignore them.

cmake: fix CheckFunctionExistsEx/CheckPrototypeDefinition (CMP0054)

cmake: cleanup

- Remove a redundant check on netdb.h (EVENT__HAVE_NETDB_H).
- Properly setup EVENT__SIZEOF_SSIZE_T for the case where "ssize_t" is
  not "int".
- Remove unused EVENT__HAVE_PTHREAD.
- Set EVENT__HAVE_LIBZ instead of EVENT__HAVE_ZLIB (and remove the
  latter).

Closes: #391 (cherry-picked from PR)

use ev_uint16_t instead of unsigned short for port

Like in `sockaddr_in` structure in /usr/include/netinet/in.h

@azat: convert all other users (bench, compat, ..) and tweak message
Fixes: #178
Fixes: #196
Refs: 6bf1ca78
Link: https://codereview.appspot.com/156040043/#msg4

cmake/win32: fix running regress, but fixing finding python2 interpreter

  Could not find executable C:/projects/libevent/build/bin/regress
  Looked in the following places:
  C:/projects/libevent/build/bin/regress
  C:/projects/libevent/build/bin/regress.exe
  ...
   9/13 Test  #9: regress__WIN32_debug ................***Not Run   0.00 sec

Link: https://ci.appveyor.com/project/nmathewson/libevent/build/2.1.6.271/job/wvy2jp7uojcut5cx#L1256
Fixes: #411

README.md: fix typo: ar -> are

travis-ci: use container-based infrastructure

- by disabling sudo
- and since we use addons.apt.packages we don't need apt-get update, that requires sudo

Link: https://docs.travis-ci.com/user/migrating-from-legacy/

travis-ci/osx: fix compiling/linking openssl libraries

And we need that $(echo /path/*) otherwise bash will expand it only during
assignment that variable to another one, and this doesn't work correctly.

travis-ci: use gcc-5 (fixes osx|gcc failures)

Refs: travis-ci/travis-ci#4587

Make it build using OpenSSL 1.1.0

Rebased (azat):
- tabs instead of whitespaces
- make openssl-compat.h safe for complex expressions
- do not call sk_SSL_COMP_free() in 1.1 (fixes double free)

TODO:
- clean methods_bufferevent

Closes: #397 (cherry-picked)

Don't call BIO_number_{read|written} on NULL BIOs.

OpenSSL doesn't document the behaviour of these functions when given a
NULL BIO, and it happens to return zero at the moment. But don't depend
on that.

Closes: #406 (cherry-picked)

Switch from a 512 to 2048-bit RSA key.

The 512 bit key is too small to sign larger hashes. This can cause the
regression tests to fail depending on the defaults in libssl.

Merge branch 'contrib-guide-v2'

Something that should help with reviewing patches.

* contrib-guide-v2:
  Add CONTRIBUTING.md (with checkpatch.sh examples)
  Add checkpatch script
  Add config for clang-format
  Add config for uncrustify

Refs: #301

Add CONTRIBUTING.md (with checkpatch.sh examples)

Add checkpatch script

Support:
- files
- files-diffs
- patches
- git-refs

Add config for clang-format

Add config for uncrustify

Original: https://strcpy.net/mark/libevent-uncrustify.cfg (@ellzey)

be: just a simple mistake, reinclude the <errno.h>

include the <errno.h> twice.

cmake: use PYTHON_EXECUTABLE to find python2

Since most of the distros have symlinks for this.

Merge branch 'force-disable-clockgettime'

* force-disable-clockgettime:
  Add -DEVENT__DISABLE_CLOCK_GETTIME switch for cmake
  Adding option to ignore clock_gettime: --disable-clock-gettime

Add -DEVENT__DISABLE_CLOCK_GETTIME switch for cmake

See-also: adc402ba5f0e15f4c77505852507f33b50f37ab6 ("Adding option to
ignore clock_gettime: --disable-clock-gettime")

Adding option to ignore clock_gettime: --disable-clock-gettime

macOS 10.12 introduced `clock_gettime` to libsystem. This means, built
on OS X 10.12 application would crash on earlier versions of OS X
because it will try to call clock_gettime. This options is useful to
make backwards compatible macOS apps.

Fixes: #398
Fixes: #399
Fixes: #400

Merge branch 'release-2.1.6-beta-pull'

* release-2.1.6-beta-pull:
  Bump version to 2.1.6-beta everywhere
  ChangeLog: Add brief notes
  Update changelog for 2.1.6-beta (ugh, pretty huge)
  Update AUTHORS section in README

Bump version to 2.1.6-beta everywhere

- cmake basic
- cmake detecting from git -- event_fuzzy_version_from_git()
- autotools
- win32
- appveyor

ChangeLog: Add brief notes

Update changelog for 2.1.6-beta (ugh, pretty huge)

Changes for:
- 2.1.5-beta: 118
- 2.1.6-beta: 350

We should really hurry with the release!

Update AUTHORS section in README

P.S. "Sebastian Hahn" was declared two times.

Fix evhttp_uriencode() regression.

http_uriencode_test() (in test/regress_http.c) has been failed after
72afe4c as "hello\0world" is encoded to "hello" instead of
"hello%00world". This is because of a misplaced overflow check which
causes the non-negative "size" specified in parameter being ignored in
within-bound URI.

Fixes: #392

removed unused vars

pointer overflow checks for evhttp_uriencode

Check to make sure pointer math is all OK.

tests: use waitpid(..., WNOWAIT) to fix failing of main/fork under solaris

According to solaris docs:
  "One instance of a SIGCHLD signal is queued for each child process whose
  status has changed. If waitpid() returns because the status of a child
  process is available, and WNOWAIT was not specified in options, any pending
  SIGCHLD signal associated with the process ID of that child process is
  discarded. Any other pending SIGCHLD signals remain pending."

And interesting thing that it works if you add sleep(1) before waitpid(), and
also if you run with --verbose (some race or what).

But linux doesn't support WNOWAIT in waitpid() so add detection into
cmake/autotools.

Fixes: #387
Link: https://bugzilla.redhat.com/show_bug.cgi?id=840782

test: replace sleeping with syncing pair in main/fork

About this syncing pair:
- read endpoint, must be blocked, to make it a checkpoint or smth like this
- write endpoint, must be nonblocking, to avoid readcb hung

Refs: #387

travis: split long lines, and make it cleaner

travis: fix autotools on osx by reinstalling libtool

Link: mkrufky/libdvbtee#22

be_sock: handle readv() returns ECONNREFUSED (freebsd 9.2)

During testing sometimes bufferevent/bufferevent_connect_fail_eventcb fails,
and after some digging I found that this was the case when connect() doesn't
return ECONNREFUSED immediately, and instead next operation will return it (in
our case evbuffer_read()->readv(), needless to say that after this
bufferevent_writecb() called and it checks error with getsockopt() but of
course it doesn't return any error), so this patch checks "errno" after
"readv()" and installs "bufev::connection_refused" flag, to handle this from
writecb and only once.

Fixes: bufferevent/bufferevent_connect_fail_eventcb
Fixes: bufferevent/bufferevent_connect_fail_eventcb_defer
Refs: #388

test/http: do not run tests that based on backlog filling (freebsd)

I cannot find any other solution for now, so simply ignore them for now, we
should think about normal fix for this.

Refs: #388

evutil: mark ai_find_protocol() static (prototype-less)

test/bufferevent/iocp: fix test name for "bufferevent_connect_fail_eventcb"

Fix getaddrinfo under solaris (for multiprotocol case)

During testing on solaris 11.3, util/getaddrinfo failed at:
  memset(&hints, 0, sizeof(hints));
  hints.ai_flags = EVUTIL_AI_NUMERICHOST;
  r = evutil_getaddrinfo("1.2.3.4", NULL, &hints, &ai);
  tt_assert(ai_find_by_protocol(ai, IPPROTO_TCP));

And this is because solaris's getaddrinfo() returns:
$6 = {
  ai_flags = 32,
  ai_family = 2,
  ai_socktype = 0,
  ai_protocol = 0, <-- no proto
  ai_addrlen = 16,
  ai_canonname = 0x0,
  ai_addr = 0x815d658,
  ai_next = 0x0 <-- nothing else
}

So we should emulate this too.

Plus introduce helper that will search through all results, not only first one.

Fixes: util/getaddrinfo
Fixes: #354

cmake/solaris: set CMAKE_REQUIRED_LIBRARIES to fix functions detections

Otherwise we will not detect next functions for instance:
- gethostbyname_r # and related

And now both autotools/cmake builds passes all regress tests in basic env (on
solaris of course).

Fixes: dns/client_fail_requests_getaddrinfo

cmake/solaris: fix building (link with socket,nsl)

Merge branch 'appveyor-cmake-v6'

This patch set introduces matrix for appveyor and fixes bugs related to win32,
during debugging this matrix. This should greatly reduce number of work for
making a release ('check under windows' part only).

Right now tests on appveyor still failes in summary, but we should dig into
problems, not hide them.

Test matrix looks like this:
- autotools
- autotools --disable-openssl
- autotools --disable-thread-support
- autotools --disable-debug-mode
- autotools --disable-malloc-replacement
- cmake
- cmake -DEVENT__DISABLE_OPENSSL=ON
- cmake -DEVENT__DISABLE_THREAD_SUPPORT=ON
- cmake -DEVENT__DISABLE_DEBUG_MODE=ON
- cmake -DEVENT__DISABLE_MM_REPLACEMENT=ON
- cmake -DEVENT__ENABLE_VERBOSE_DEBUG=ON
- cmake -DCMAKE_C_FLAGS='-DUNICODE -D_UNICODE'
And takes 90min (was 7min, and this is the sad part).

* appveyor-cmake-v6:
  test/ssl: use send()/recv()/EVUTIL_ERR_RW_RETRIABLE()/EVUTIL_SOCKET_ERROR() to fix win32
  test/https_basic: increase timeout for complete write (fixes win32)
  cmake: check for ZLIB_INCLUDE_DIR, since we can have only library without headers
  autotools/win32: fix searching ssl library
  appveyor/autotools: link with openssl by passing LDFLAGS/CFLAGS
  appveyor: image already had openssl installed
  cmake/win32: do not compile regress_thread on -DEVENT__DISABLE_THREAD_SUPPORT=ON
  cmake/win32: do not compile evthread_win32 on -DEVENT__DISABLE_THREAD_SUPPORT=ON
  appveyor: check -DUNICODE -D_UNICODE according to ReleaseChecklist (cmake only)
  cmake: fix -DEVENT__ENABLE_VERBOSE_DEBUG (typo on -DUSE_DEBUG)
  test: fix building with --disable-thread-support under win32
  cmake: do not use stderr for notifications/version-info
  autoconf: fix --disable-thread-support build under win32
  appveyor: ignore failure of mingw-get
  appveyor: drop shallow_clone, since we use tags for detecting version in cmake
  appveyor: support cmake & autotools using build matrix (like travis-ci has)

Fixes: #364

test/ssl: use send()/recv()/EVUTIL_ERR_RW_RETRIABLE()/EVUTIL_SOCKET_ERROR() to fix win32

Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.5.107/job/k70our1xdp0ym4dm#L1906
Fixes: ssl/bufferevent_connect_sleep

test/https_basic: increase timeout for complete write (fixes win32)

Otherwise on win32 we got 2, but test is ok, some timings issue.

Introduced-in: c968eb3
Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.5.107/job/k70our1xdp0ym4dm#L1906

cmake: check for ZLIB_INCLUDE_DIR, since we can have only library without headers

autotools/win32: fix searching ssl library

appveyor/autotools: link with openssl by passing LDFLAGS/CFLAGS

This never works, hence green tests in production. and now it works:
https://ci.appveyor.com/project/azat/libevent/build/2.1.5.100/job/eib4xqfb7d3ruo16
(look by /"evthread_win32.c")

appveyor: image already had openssl installed

Fixes messages like "Version mismatch for openssl: compiled with 1000201f but
running with 1000208f" [1].

According to next link:
  http://help.appveyor.com/discussions/questions/1132-openssl-installation-issues
Refs: appveyor/ci#576
[1]: https://ci.appveyor.com/project/azat/libevent/build/2.1.5.95/job/dcy9cfoj3rfo4gdh

cmake/win32: do not compile regress_thread on -DEVENT__DISABLE_THREAD_SUPPORT=ON

cmake/win32: do not compile evthread_win32 on -DEVENT__DISABLE_THREAD_SUPPORT=ON

There is duplicated "evthread_win32.c" appending to ${SRC_CORE} list, leave
only one that under if EVENT__DISABLE_THREAD_SUPPORT

Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.5.96/job/44q3rgifasny7gek

appveyor: check -DUNICODE -D_UNICODE according to ReleaseChecklist (cmake only)

Link: https://github.com/libevent/libevent/wiki/ReleaseChecklist

cmake: fix -DEVENT__ENABLE_VERBOSE_DEBUG (typo on -DUSE_DEBUG)

Fixes: 8b228e27f57300be61b57a41a2ec8666b726dc34 ("Lot's of cmake updates")

test: fix building with --disable-thread-support under win32

cmake: do not use stderr for notifications/version-info

For all except "STATUS" message() will write to stderr, and this also includes
case when you does not have <mode> at all. Plus for every message in stderr
powershell generates exception and make appveyor fails.

Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.5.85/job/i10k7m0t80330mtr

autoconf: fix --disable-thread-support build under win32

Fixes: https://ci.appveyor.com/project/azat/libevent/build/job/gvud4tcqsd5bnarl
Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.6.80/job/5frnb1c3n4quxxqy
Fixes: https://ci.appveyor.com/project/azat/libevent/build/2.1.6.80/job/3wdahbrew7setmoa

appveyor: ignore failure of mingw-get

Since according to doc [1], appveyour will fail the build on every powershell's
exception:
  "PowerShell script is considered successful if it finishes without exception."

[1]: https://www.appveyor.com/docs/build-configuration#powershell

appveyor: drop shallow_clone, since we use tags for detecting version in cmake

appveyor: support cmake & autotools using build matrix (like travis-ci has)

This will allow us to avoid possible compilation regressions, and also some
failures in different environments (for example because of different
sizeof(size_t)).

For example we can avoid next issues:
- #361
- #314
- #311
(And I'm pretty sure that this is not complete list)

And of course it reduce job for maintainers.
But it also have one downside, appveyor doesn't have such concurrency like
travis-ci, so it can take a while to finish the whole build, but we can resolve
this later, or reduce build matrix to only one for autotools and one for cmake.

Fixes: #364