Fix a major parenthesis bug in EVUTIL_UPCAST.

Fortunately, this didn't hurt anything previously, since we had no actual users of the macro where the offset of the base type wasn't 0.

svn:r1488

Add more IOCP tests.  They might not pass yet.

svn:r1487

More refactoring for IOCP listener code

svn:r1486

Call the bufferevent_flush_mode variable "mode" more consistently in the documentation. Spotted by Alex.

svn:r1485

Fix an errant user of ssize_t to use ev_ssize_t.

My usual strategy of grep '[^_]ssize_t' had apparently failed me,
since this ssize_t was in the first column.

Resolves bug 2890434; spotted by Mihai Draghicioiu.

svn:r1484

Remove some duplicate code in kqueue.c and fix a small memory leak.

svn:r1483

Add a "many events" regression test.

This is a glass-box test to get more coverage on the event loop
backends.  We've run into bugs here before with fencepost errors, and
it turns out that none of our unit tests had enough events to
exercise the resize code.

Most of the backends have some kind of logic that resizes an array
when:
    - The highest fd is too high
    - The number of events added since the last iteration of the loop
      is too high
    - The number of active events is too high.

This test hits all 3 cases, and increases coverage in select.c by 7%,
in poll by 1%, and in kqueue by 9%.

svn:r1482

Keep openssl errors associated with the right bufferevent object.

OpenSSL has a per-thread error stack, and really doesn't like you
leaving errors on the stack.  Rather than discard the errors or force
the user to handle them, this patch pulls them off the openssl stack
and puts them on a stack associated with the bufferevent_openssl.  If
the user leaves them on the stack then, it won't affect any other
connections.

This bug was found by Roman Puls.  Thanks!

svn:r1481

Refactor evconnlistener to allow multiple implementations; add an (incomplete, not-yet-integrated) IOCP implementation.

svn:r1480

Extract XP-only functions when initializing the IOCP port

svn:r1479

reformat weird indentation in dns tests

svn:r1478

Unit test for reverse ipv6 lookup

svn:r1477

More documentation and unit tests for event_tagging.

svn:r1476

Actually use the logic in select.c designed to make the out_sets threadsafe.

svn:r1475

Unit test for strlcpy

svn:r1474

Fix from Chris Davis: get error-logging to be happy on win32.

svn:r1473

Avoid calling exit() during event_base_new*()

Previously, each of the three make-an-event-base functions would exit
under different, weird circumstances, but return NULL on others.
  - All three would exit on OOM sometimes.
  - event_base_new() and event_init() would die if all backends were
    disabled.
  - None of them would die if the socketpair() call failed.

Now, only event_init() exits on failure, and it exits on every kind of
failure.  event_base_new() and event_base_new_with_config() never do.

svn:r1472

Do not call the locking variant of event_add or event_active in some cases when we know we have the lock.

svn:r1471

Refactor event_assing even more to avoid unnecessary calls

svn:r1470

Give event_assign a return value, and make it less inclined to exit().

We also refactor event_assign so that it is the core function, and
event_set() is only the wrapper.

svn:r1469

Do not add a newline to the end of log statements.

svn:r1468

Refactor kq_init error handling.

svn:r1467

New test flag to suppress logging for one test.

svn:r1466

Note assert-related change in changelog

svn:r1465

Use EVUTIL_ASSERT() consistently instead of assert.

svn:r1464

Add an EVUTIL_ASSERT() to replace our calls to assert().

The big difference here is that EVUTIL_ASSERT() passes its message on
via event_errx() before aborting, so that the application has a prayer
of noticing and recording it.

svn:r1463

API to replace all calls to exit() with a user-supplied fatal-error handler.

Also, add unit tests for logging.

svn:r1462

Tweaks to IOCP interface.

svn:r1461

Fix my fix for the bufferevent_connect_fail() test.

svn:r1460

Expose a narrow window to the IOCP code.

svn:r1459

Improve the behavior of le-proxy in a few cases.

svn:r1458

Make the bufferevent_connect_fail test faster on OSX.

It seems that connecting to a listener that is bound but not accepting
or listening doesn't give a 'connection refused' error on OSX, but
rather makes the connect() time out after 75 seconds.  I couldn't find
any way to make the timout shorter.  Fortunately, closing the listener
after a second or so makes the desired error occur after another
second or so.

svn:r1457

Treat the bitwise OR of two enum values as an int.

This makes our interfaces usable from C++, which doesn't believe
you can say    "bufferevent_socket_nase(base, -1,
BEV_OPT_CLOSE_ON_FREE|BEV_OPT_DEFER_CALLBACKS)" but which instead
would demand "static_cast<bufferevent_options>(BEV_OPT_CLOSE_ON_FREE|
BEV_OPT_DEFER_CALLBACKS))" for the last argument.

Diagnosis and patch from Chris Davis.

svn:r1456

Test failing case of bufferevent_connect().

Code by Chris Davis.

svn:r1455

Fix win32 connect() event handling.

Christopher Davis reported:

    Connection failures aren't reported on Windows when
    using bufferevent_socket_connect, because Windows uses
    select's exceptfds to notify of failure, and libevent
    treats them like read events. Only the write event
    handler is currently used to handle connection events.

We should think hard about this one, since it changes
behavior from 1.4.x.  Anything that worked on Mac/Unix before
will work more consistently on Windows now... but this might
break stuff that worked only on Windows, but nowhere else.

Patch from Chris Davis.

svn:r1454

OSX compilation issues

svn:r1453

Remove an EVBASE_RELEASE_LOCK that I missed.

svn:r1452

Fix win32 compilation.

svn:r1451

Add locking to event_base_loop.

This is harder than it sounds, since we need to make sure to
release the lock around the key call to the kernel (e.g.,
select, epoll_wait, kevent), AND we need to make sure that
none of the fields that are used in that call are touched by
anything that might be running concurrently in another
thread.  I managed to do this pretty well for everything but
poll().  With poll, I needed to introduce a copy of the
event_set structure.

This patch also fixes a bug in win32.c where we called
realloc() instead of mm_realloc().

svn:r1450

Fix windows compilation warnings.

svn:r1449

Fix compilation for listener.h for C++ - missing extern "C".  Patch from Ferenc Szalai.

svn:r1448

Add a note that we should change previous_to_last to last_with_space

svn:r1447

Correct the signatures for the evmap_io_* functions to use evutil_socket_t.

svn:r1446

Spelling fixes in comments and strings.

svn:r1445

When a bufferevent_connect() call fails, give the client an error callback.

Patch from Christopher Davis.

svn:r1444

Fix some crash bugs when initializing evdns

svn:r1443

Declare struct timezone in util.h so that borken mingw versions do not complain

svn:r1441

Add changelog for last commit

svn:r1440

Do not notify the main thread more than needed.

Basically, we suppress the notification when an event is added or deleted
and:
  - The event has no fd, or there is no change in whether we are
    reading/writing on the event's fd.
  - The event has no timeout, or adding the event did not make the earliest
    timeout become earlier.

This should be a big efficiency win in applications with multiple threads and
lots of timeouts.

svn:r1439

Apply Ka-Hing Cheung's event_base_got_[break|exit] patch, with locking and whitespace fixes.

svn:r1438

Do not drop data from evbuffer when out of memory; reported by Jacek Masiulaniec

svn:r1436

Forward-port: fix android compilation

svn:r1435

Fix the assert I added to epoll.c: spotted by Dmitry Novikov

svn:r1432

Treat events with fd == -1 as addable.

This turns out to simplify a fair bit of logic, including the bufferevent
code, and should fix bug 2850656.

svn:r1431

Make epoll use less RAM.

We do this by not allocating the maximum epoll_event array for the epoll
backend at startup.  Instead, we start out accepting 32 events at a time, and
double the array's size when it seems that the OS is generating events faster
than we're requesting them.  This saves up to 374K per epoll-based
event_base.  Resolves bug 2839240.

svn:r1428

Add a trivial race-fix from Chromium: do not try to re-detect whether we have a monotonic clock every time we make a new event_base.

svn:r1427

Treat a negative number of bytes to read as the kernel saying "I don't know."

svn:r1426

On connect, call only one of BEV_EVENT_CONNECTED or writecb.

Previously, if we had a socket bufferevent in connect state, we'd send
both of these to indicate that the connection was done.  That was broken
since the point of adding BEV_EVENT_CONNECTED was so that we could
distinguish "we're connected" and "we wrote something".

Now, writecb is called only when
   A) the connection finished but the user never put the socket into a
      "connecting" state, or
   B) data was actually written.

svn:r1425

Fix build warnings and add changelog entry for evhttp patches.

svn:r1424

Define evhttp_del_accept_socket

[Patch from David Reiss]

svn:r1423

Define evhttp_{bind,accept}_socket_with_handle

[Patch from David Reiss]

svn:r1422

Make evhttp_bound_socket visible, and provide an accessor to its fd

Declare the previously private struct evhttp_bound_socket in
event2/http.h as an opaque struct.

Implement evhttp_bound_socket_get_fd, which returns the file descriptor
of an evhttp_bound_socket.

[Patch from David Reiss]

svn:r1421

Minor documentation fixes

[Patch from David Reiss]

svn:r1420

Support sendfile on solaris: patch from Caitlin Mercer.

svn:r1419

New function to put an SSL bufferevent into a renegotiating state.

svn:r1418

Disable whichever struct event we don't want during ssl handshaking.

svn:r1417

Only send a connected event _after_ we've adjusted the SSL state.

This is important if the callback adjusts it to something else.

svn:r1416

Remove an extraneous puts().

svn:r1415

tab/whitespace fixes in bufferevent_sock.c

Also note that write() doesn't usually say 0.

svn:r1414

When running with deferred callbacks, always send the "connected" event
before any read/write events, and send timeout/error/eof events after.

svn:r1413

Add a few missing changelog entries

svn:r1412

When bufferevent_socket_connect is called with no address, assume that our existing fd is connecting and put the connection into "connecting" mode.

svn:r1411

Add an evbuffer_search_range() to search a bounded range of a buffer

This can be handy when you have one search to find the end of a header
section, and then you want to find a substring within the header
section without looking at the body.

svn:r1410

Add a couple more evdns tests.  Libevent is now, for me, at 80.02% coverage.

svn:r1409

Another DNS unit tests, to handle reissues.

The evdns module is now up to ~72% coverage; Libevent is up to nearly 80%.

svn:r1408

why say fprintf(stdout, X) when you can say printf?

svn:r1407

Fix an annoying evdns crash bug, and add more unit tests for evdns.

svn:r1406

Unit tests for DNS search.

svn:r1405

Refactor evbuffer_readln() into a search-for-eol function and an extract-line function.

svn:r1404

Refactor evbuffer_readln to use evbuffer_ptr; remove old evbuffer_iterator.

svn:r1403

Improved coverage for evbuffer_readln()

svn:r1402

New function to expose bufferevent.enabled

svn:r1401

Export sockaddr comparison functionality.

svn:r1400

More unit tests for Openssl, including initializing with no socket. Up to 75% coverage.

svn:r1399

Fix some bugs in bufferevent_socket_connect

svn:r1398

Add unit tests for SSL session renegotiation.

This tickles the write-blocked-on-read code, and in this case turned
up a bug in it.

svn:r1397

Always retry SSL_write() with the same number you told it last time.

svn:r1396

Add a function to extract the SSL object from a bufferevent_openssl.

svn:r1395

Set the SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER flag, and explain why.

svn:r1394

Use SSL_do_handshake in place of SSL_connect/SSL_accept

svn:r1393

More evconnlistener unit tests: bump its coverage from 71% to 83%.

svn:r1392

Export an ev_socklen_t.

svn:r1391

Accessor function to get a listener's associated fd

svn:r1390

Add unit test for parsing addresses with bad ports.

svn:r1389

Changelog entry for msvc fixes.

svn:r1388

Export evutil_str[n]casecmp as evutil_ascii_str[n]casecmp.

svn:r1387

Some tweaks to Brodie Thesfield's MSVC patch.

svn:r1386

Various MSVC cleanups from Brodie Thiesfield.

svn:r1385

Fix segfault during failed allocatino of locked evdns base.

We need to comb the rest of the code to make sure that we don't blindly wrap
functions in LOCK(x), UNLOCK(x) when those functions might contain a FREE(x)
in the middle.

Rocco Carbone found and reported this bug.

svn:r1384

Fix a dumb bug where we would allocate too little memory in event_get_supported_methods().

svn:r1383