Nick Mathewson [Fri, 20 Nov 2009 17:17:14 +0000 (12:17 -0500)]
Add a .gitignore file.
.gitignore plays the same role in Git as the svn:ignore property does in
subversion.
Niels Provos [Fri, 20 Nov 2009 00:18:35 +0000 (00:18 +0000)]
call it 2.0.3-alpha
svn:r1556
Niels Provos [Thu, 19 Nov 2009 23:08:50 +0000 (23:08 +0000)]
Remove most calls to event_err() in http and deal with memory errors instead
svn:r1555
Niels Provos [Thu, 19 Nov 2009 22:02:33 +0000 (22:02 +0000)]
nick found a race condition in the pthreads test case
svn:r1554
Niels Provos [Thu, 19 Nov 2009 21:14:31 +0000 (21:14 +0000)]
On FreeBSD and other OSes, connect can return ECONREFUSED immediately; instead of failing the function call, pretend with faileld in the callback.
svn:r1553
Nick Mathewson [Thu, 19 Nov 2009 00:21:48 +0000 (00:21 +0000)]
Documentation adjustments
svn:r1552
Nick Mathewson [Thu, 19 Nov 2009 00:21:38 +0000 (00:21 +0000)]
Fix a spelling error and remove some dead code
svn:r1551
Nick Mathewson [Wed, 18 Nov 2009 23:18:55 +0000 (23:18 +0000)]
Do the proper hack for the (Open)BSD getaddrinfo quirk.
From evutil.c:
Some older BSDs (like OpenBSD up to 4.6) used to believe that
giving a numeric port without giving an ai_socktype was verboten.
We test for this so we can apply an appropriate workaround. If it
turns out that the bug is present, then:
- If nodename==NULL and servname is numeric, we build an answer
ourselves using evutil_getaddrinfo_common().
- If nodename!=NULL and servname is numeric, then we set
servname=NULL when calling getaddrinfo, and post-process the
result to set the ports on it.
We test for this bug at runtime, since otherwise we can't have the
same binary run on multiple BSD versions.
svn:r1550
Nick Mathewson [Wed, 18 Nov 2009 21:17:00 +0000 (21:17 +0000)]
Make our failing-connection bufferevent test more tolerant.
I thought we had a way to do connect() that would never fail
immediately, but always wait for a moment before failing. It
turns out that on FreeBSD it can fail immediately. This is not
FreeBSD's fault, or even a real bug anywhere but in the unit test.
svn:r1549
Nick Mathewson [Wed, 18 Nov 2009 21:16:53 +0000 (21:16 +0000)]
Add a temporary workaround for an ssl bug found on FreeBSD.
Basically, we only want to report the 'connected' event because of
the socket connect() finishing when we have an actual socket
bufferevent; on an SSL bufferevent, 'connected' means 'SSL
connection finished.'
This isn't FreeBSD's fault: it just has a connect() that tends to
succeed pretty early.
svn:r1548
Nick Mathewson [Wed, 18 Nov 2009 21:16:47 +0000 (21:16 +0000)]
Build correctly with mm replacement turned off.
svn:r1547
Nick Mathewson [Wed, 18 Nov 2009 21:16:33 +0000 (21:16 +0000)]
Fix compilation with threading disabled.
svn:r1546
Nick Mathewson [Tue, 17 Nov 2009 20:31:09 +0000 (20:31 +0000)]
Move responsibility for IOCP callback into bufferevent_async.
This patch from Chris Davis saves some callback depth, and adds proper
ref-counting to bufferevents when there's a deferred evbuffer callback
inflight. It could use a couple more comments to really nail down what
its invariants are.
svn:r1543
Nick Mathewson [Tue, 17 Nov 2009 18:29:44 +0000 (18:29 +0000)]
Clarify even more about various system-specific problems with getaddrinfo
svn:r1542
Nick Mathewson [Tue, 17 Nov 2009 03:36:43 +0000 (03:36 +0000)]
Use the common-case code from getaddrinfo_common *always*; OS differences are just too huge.
svn:r1541
Nick Mathewson [Tue, 17 Nov 2009 02:57:32 +0000 (02:57 +0000)]
Never pass our weird flags to the system getaddrinfo. Make sure there is no overlap between flag values.
svn:r1540
Nick Mathewson [Tue, 17 Nov 2009 02:40:14 +0000 (02:40 +0000)]
OpenBSD demands that sys/types.h be included before sys/socket.h
svn:r1539
Nick Mathewson [Tue, 17 Nov 2009 02:38:19 +0000 (02:38 +0000)]
Windows *does* have getservbyname, no matter what autoconf says.
TODO: figure out why autoconf is confused about this.
svn:r1538
Nick Mathewson [Mon, 16 Nov 2009 22:25:46 +0000 (22:25 +0000)]
Add two implementations of getaddrinfo: one blocking and one nonblocking.
The entry points are evutil_getaddrinfo and evdns_getaddrinfo respectively.
There are fairly extensive unit tests.
I believe this code conforms to RFC3493 pretty closely, but there are
probably more issues. It should get tested on more platforms.
This code means we can dump the well-intentioned but weirdly-implemented
bufferevent_evdns and evutil_resolve code.
svn:r1537
Nick Mathewson [Mon, 16 Nov 2009 22:23:55 +0000 (22:23 +0000)]
Remove the stupid brokenness where DNS option names needed to end with a
colon.
svn:r1536
Nick Mathewson [Mon, 16 Nov 2009 22:23:06 +0000 (22:23 +0000)]
Fix a declaration of __func__ in rpcgen.
svn:r1535
Nick Mathewson [Sun, 15 Nov 2009 19:00:12 +0000 (19:00 +0000)]
Prefer calloc(a,b) to malloc(a*b). via openbsd.
svn:r1531
Nick Mathewson [Sun, 15 Nov 2009 18:59:59 +0000 (18:59 +0000)]
When running set[ug]id, don't check the environment.
Idea from OpenBSD, but made a bit more generic to handle uncivilized lands
that do not define issetugid.
svn:r1530
Nick Mathewson [Sun, 15 Nov 2009 18:59:48 +0000 (18:59 +0000)]
Use arc4random() for dns transaction ids where available. Patch taken from OpenBSD
svn:r1528
Nick Mathewson [Sat, 14 Nov 2009 21:54:30 +0000 (21:54 +0000)]
Fix a couple of event_debug calls.
svn:r1527
Nick Mathewson [Mon, 9 Nov 2009 19:37:27 +0000 (19:37 +0000)]
Change event_base.activequeues to "array of eventlist".
Previously, event_base.activequeues was of type "array of pointers to
eventlist." This was pointless: none of the eventlists were allowed
to be NULL. Worse, it was inefficient:
- It made looking up an active event queue take two pointer
deferences instead of one, thus risking extra cache misses.
- It used more RAM than it needed to, because of the extra pointer
and the malloc overhead.
Also, this patch fixes a bug where we were saying
calloc(N,N*sizeof(X)) instead of calloc(N,sizeof(X)) when allocating
activequeues. That part, I'll backport.
Also, we warn and return -1 on failure to allocate activequeues,
rather than calling event_err.
svn:r1525
Nick Mathewson [Mon, 9 Nov 2009 19:37:21 +0000 (19:37 +0000)]
Change an OOM err to a warn in event_tagging.c
svn:r1524
Nick Mathewson [Mon, 9 Nov 2009 19:37:15 +0000 (19:37 +0000)]
Change an err to a warn in bufferevent_openssl
svn:r1523
Nick Mathewson [Mon, 9 Nov 2009 18:50:20 +0000 (18:50 +0000)]
Patch from Ryan Phillips: accept ipv6 addresses returned by getaddrinfo in http.c
svn:r1522
Nick Mathewson [Mon, 9 Nov 2009 18:36:34 +0000 (18:36 +0000)]
Unit tests for got_break and got_exit.
svn:r1521
Nick Mathewson [Mon, 9 Nov 2009 18:30:57 +0000 (18:30 +0000)]
Make persistent timeouts more accurate.
Previously, if the user scheduled a persistent timeout for {1,0}, we
would schedule the first one at "now+one second", and then when we
were about to run its callback, we would schedule it again for one
second after that. This would introduce creeping delays to the event
that was supposed to run every second.
Now, we schedule the event for one second after it was _last
scheduled_. To do this, we introduce internal code to add an event at
an _absolute_ tv rather than at now+tv.
svn:r1520
Nick Mathewson [Mon, 9 Nov 2009 18:30:48 +0000 (18:30 +0000)]
Make sure that common timeouts are inserted in-order.
This code should be a no-op, except under strange thread contention
situations.
svn:r1519
Nick Mathewson [Mon, 9 Nov 2009 18:30:33 +0000 (18:30 +0000)]
Add an option to disable the timeval cache.
svn:r1518
Nick Mathewson [Mon, 9 Nov 2009 17:16:30 +0000 (17:16 +0000)]
Implement queued timeouts for case where many timeouts are the same.
Libevent's current timeout code is relatively optimized for the
randomly scattered timeout case, where events are added with their
timeouts in no particular order. We add and remove timeouts with
O(lg n) behavior.
Frequently, however, an application will want to have many timeouts
of the same value. For example, we might have 1000 bufferevents,
each with a 2 second timeout on reading or writing. If we knew this
were always the case, we could just put timeouts in a queue and get
O(1) add and remove behavior. Of course, a queue would give O(n)
performance for a scattered timeout pattern, so we don't want to
just switch the implementation.
This patch gives the user the ability to explicitly tag certain
timeout values as being "very common". These timeout values have a
cookie encoded in the high bits of their tv_usec field to indicate
which queue they belong on. The queues themselves are each
triggered by an entry in the minheap.
See the regress_main.c code for an example use.
svn:r1517
Nick Mathewson [Fri, 6 Nov 2009 21:46:57 +0000 (21:46 +0000)]
We do not work any more without an event-config.h; stop pretending that it is meaningful to check for HAVE_CONFIG_H
svn:r1516
Nick Mathewson [Fri, 6 Nov 2009 21:13:25 +0000 (21:13 +0000)]
Fix kqueue.c build on GNU/kFreeBSD systems.
Yes, some people like to have a BSD-family kernel (thus getting
kqueue) with a GNU-family libc (thus occasionally mandating
_GNU_SOURCE).
Thanks to Debian for noticing this.
svn:r1514
Nick Mathewson [Fri, 6 Nov 2009 17:12:39 +0000 (17:12 +0000)]
Fix a miscalculated realloc() size in win32select.c.
This bug was introduced by the code to make the backend able to safely release the base lock while calling select().
Also, we change win32select.c to the same 32-fds-to-start default as the rest of the backends, so that the main/many_events test can test it. It was at 64-to-start, so the test wasn't hitting it.
svn:r1513
Nick Mathewson [Thu, 5 Nov 2009 22:24:21 +0000 (22:24 +0000)]
Actually add the new dns-example.c code. :p
svn:r1512
Nick Mathewson [Thu, 5 Nov 2009 22:19:09 +0000 (22:19 +0000)]
Move the evdns sample code into the sample directory and fix it not to use any deprecated APIs.
svn:r1511
Nick Mathewson [Thu, 5 Nov 2009 21:22:23 +0000 (21:22 +0000)]
Fix some build warnings on MSVC, mostly related to signed/unsigned comparisons.
svn:r1510
Nick Mathewson [Thu, 5 Nov 2009 20:45:07 +0000 (20:45 +0000)]
Export nmakefiles in source distribution.
svn:r1509
Nick Mathewson [Thu, 5 Nov 2009 20:40:11 +0000 (20:40 +0000)]
Add nmake files to build with MSVC.
Right now, they just make static libraries and unit tests. They probably set lots of options wrong.
svn:r1507
Nick Mathewson [Thu, 5 Nov 2009 20:37:19 +0000 (20:37 +0000)]
Build fixes for MSVC
svn:r1506
Nick Mathewson [Thu, 5 Nov 2009 18:49:08 +0000 (18:49 +0000)]
Rename win32.c to win32select.c, and take it out of the WIN32-Code ghetto.
svn:r1504
Nick Mathewson [Thu, 5 Nov 2009 18:25:46 +0000 (18:25 +0000)]
Fix another ssize_t user
svn:r1503
Nick Mathewson [Thu, 5 Nov 2009 18:07:27 +0000 (18:07 +0000)]
Remove win32-code/config.h. It was apparently confusing.
svn:r1502
Nick Mathewson [Thu, 5 Nov 2009 15:57:22 +0000 (15:57 +0000)]
Fix a few types to use compatible versions
svn:r1501
Nick Mathewson [Wed, 4 Nov 2009 20:17:32 +0000 (20:17 +0000)]
Implement size limits on HTTP header length and body length.
Patch from Constantine Verutin, simplified a little.
svn:r1500
Nick Mathewson [Wed, 4 Nov 2009 05:19:26 +0000 (05:19 +0000)]
Commit ConnectEx code to get connect working with async bufferevents.
This is code by Chris Davis, with changes to get the unit tests failing less aggressively.
The unit tests for this code do not completely pass yet; Chris is looking into that. If they aren't passing by the next release, I'll turn off this code.
svn:r1499
Nick Mathewson [Wed, 4 Nov 2009 03:54:05 +0000 (03:54 +0000)]
Add a missing include for win32.
svn:r1498
Nick Mathewson [Tue, 3 Nov 2009 20:42:32 +0000 (20:42 +0000)]
Oops; add missing bufferevent_evdns.c file
svn:r1497
Nick Mathewson [Tue, 3 Nov 2009 20:40:48 +0000 (20:40 +0000)]
Add a bufferevent function to resolve a name then connect to it.
This function, bufferevent_socket_connect_hostname() can either use
evdns to do the resolve, or use a new function (evutil_resolve) that
uses getaddrinfo or gethostbyname, like http.c does now.
This function is meant to eventually replace the hostname resolution mess in
http.c.
svn:r1496
Nick Mathewson [Tue, 3 Nov 2009 20:05:06 +0000 (20:05 +0000)]
Fix one IOCP-callback signature I missed
svn:r1495
Nick Mathewson [Tue, 3 Nov 2009 19:54:56 +0000 (19:54 +0000)]
Remove compat/sys/_time.h
I've gone through everything that it declared to see where it was used,
and it seems that we probably don't need it anywhere.
Here's what it declared, and why I think we're okay dropping it.
o struct timeval {}
(Used all over, and we can't really get away with declaring it ourselves;
we need the same definition the system uses. If we can't find struct
timeval, we're pretty much sunk.)
o struct timespec {}
(Used in event.c, evdns.c, kqueue.c, evport.c. Of these,
kqueue.c and event.c include sys/_time.h. event.c conditions its use on
_EVENT_HAVE_CLOCK_GETTIME, and kqueue() only works if timespec is defined.)
o TIMEVAL_TO_TIMESPEC
(Used in kqueue.c, but every place with kqueue has sys/time.h)
o struct timezone {}
(event2/util.h has a forward declaration; only evutil.c references it and
doesn't look at its contents.)
o timerclear, timerisset, timercmp, timeradd, timersub
(Everything now uses the evutil_timer* variants.)
o ITIMER_REAL, ITIMER_VIRTUAL, ITIMER_PROF, struct itemerval
(These are only used in test/regress.c, which does not include _time.h)
o CLOCK_REALTIME
(Only used in evdns.c, which does not include _time.h)
o TIMESPEC_TO_TIMEVAL
o DST_*
o timespecclear, timespecisset, timespeccmp, timespecadd, timespecsub
o struct clockinfo {}
o CLOCK_VIRTUAL, CLOCK_PROF
o TIMER_RELTIME, TIMER_ABSTIME
(unused)
svn:r1494
Nick Mathewson [Mon, 2 Nov 2009 20:59:13 +0000 (20:59 +0000)]
Fix remaining AcceptEx issues.
svn:r1492
Nick Mathewson [Mon, 2 Nov 2009 20:20:40 +0000 (20:20 +0000)]
Clean up acceptex code some more: add locking, single-threading, enable/disable.
svn:r1491
Nick Mathewson [Mon, 2 Nov 2009 19:51:26 +0000 (19:51 +0000)]
Refactor IOCP callback interface
Chris Davis points out that GetQueuedCompletionStatus
sometimes returns false not to report "No events for
you!" but instead to report "An overlapped operation
failed." Add a way to tell an event_overlapped that
its operation failed.
svn:r1490
Nick Mathewson [Mon, 2 Nov 2009 19:31:29 +0000 (19:31 +0000)]
Compilation and correctness fixes for IOCP listener code.
svn:r1489
Nick Mathewson [Mon, 2 Nov 2009 19:30:25 +0000 (19:30 +0000)]
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
Nick Mathewson [Mon, 2 Nov 2009 17:42:16 +0000 (17:42 +0000)]
Add more IOCP tests. They might not pass yet.
svn:r1487
Nick Mathewson [Mon, 2 Nov 2009 17:42:09 +0000 (17:42 +0000)]
More refactoring for IOCP listener code
svn:r1486
Nick Mathewson [Mon, 2 Nov 2009 16:31:13 +0000 (16:31 +0000)]
Call the bufferevent_flush_mode variable "mode" more consistently in the documentation. Spotted by Alex.
svn:r1485
Nick Mathewson [Mon, 2 Nov 2009 16:17:06 +0000 (16:17 +0000)]
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
Nick Mathewson [Fri, 30 Oct 2009 22:43:53 +0000 (22:43 +0000)]
Remove some duplicate code in kqueue.c and fix a small memory leak.
svn:r1483
Nick Mathewson [Fri, 30 Oct 2009 22:43:30 +0000 (22:43 +0000)]
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
Nick Mathewson [Fri, 30 Oct 2009 21:08:29 +0000 (21:08 +0000)]
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
Nick Mathewson [Thu, 29 Oct 2009 19:25:33 +0000 (19:25 +0000)]
Refactor evconnlistener to allow multiple implementations; add an (incomplete, not-yet-integrated) IOCP implementation.
svn:r1480
Nick Mathewson [Thu, 29 Oct 2009 18:30:43 +0000 (18:30 +0000)]
Extract XP-only functions when initializing the IOCP port
svn:r1479
Nick Mathewson [Thu, 29 Oct 2009 17:11:12 +0000 (17:11 +0000)]
reformat weird indentation in dns tests
svn:r1478
Nick Mathewson [Thu, 29 Oct 2009 17:10:36 +0000 (17:10 +0000)]
Unit test for reverse ipv6 lookup
svn:r1477
Nick Mathewson [Thu, 29 Oct 2009 16:35:20 +0000 (16:35 +0000)]
More documentation and unit tests for event_tagging.
svn:r1476
Nick Mathewson [Thu, 29 Oct 2009 16:35:15 +0000 (16:35 +0000)]
Actually use the logic in select.c designed to make the out_sets threadsafe.
svn:r1475
Nick Mathewson [Thu, 29 Oct 2009 16:35:09 +0000 (16:35 +0000)]
Unit test for strlcpy
svn:r1474
Nick Mathewson [Tue, 27 Oct 2009 18:25:19 +0000 (18:25 +0000)]
Fix from Chris Davis: get error-logging to be happy on win32.
svn:r1473
Nick Mathewson [Tue, 27 Oct 2009 06:47:25 +0000 (06:47 +0000)]
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
Nick Mathewson [Tue, 27 Oct 2009 05:16:32 +0000 (05:16 +0000)]
Do not call the locking variant of event_add or event_active in some cases when we know we have the lock.
svn:r1471
Nick Mathewson [Tue, 27 Oct 2009 05:16:23 +0000 (05:16 +0000)]
Refactor event_assing even more to avoid unnecessary calls
svn:r1470
Nick Mathewson [Tue, 27 Oct 2009 04:25:45 +0000 (04:25 +0000)]
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
Nick Mathewson [Tue, 27 Oct 2009 04:04:07 +0000 (04:04 +0000)]
Do not add a newline to the end of log statements.
svn:r1468
Nick Mathewson [Tue, 27 Oct 2009 04:03:58 +0000 (04:03 +0000)]
Refactor kq_init error handling.
svn:r1467
Nick Mathewson [Tue, 27 Oct 2009 04:03:50 +0000 (04:03 +0000)]
New test flag to suppress logging for one test.
svn:r1466
Nick Mathewson [Mon, 26 Oct 2009 20:07:06 +0000 (20:07 +0000)]
Note assert-related change in changelog
svn:r1465
Nick Mathewson [Mon, 26 Oct 2009 20:00:43 +0000 (20:00 +0000)]
Use EVUTIL_ASSERT() consistently instead of assert.
svn:r1464
Nick Mathewson [Mon, 26 Oct 2009 20:00:08 +0000 (20:00 +0000)]
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
Nick Mathewson [Mon, 26 Oct 2009 19:59:51 +0000 (19:59 +0000)]
API to replace all calls to exit() with a user-supplied fatal-error handler.
Also, add unit tests for logging.
svn:r1462
Nick Mathewson [Fri, 23 Oct 2009 22:38:35 +0000 (22:38 +0000)]
Tweaks to IOCP interface.
svn:r1461
Nick Mathewson [Fri, 23 Oct 2009 22:07:05 +0000 (22:07 +0000)]
Fix my fix for the bufferevent_connect_fail() test.
svn:r1460
Nick Mathewson [Fri, 23 Oct 2009 22:00:29 +0000 (22:00 +0000)]
Expose a narrow window to the IOCP code.
svn:r1459
Nick Mathewson [Fri, 23 Oct 2009 17:40:00 +0000 (17:40 +0000)]
Improve the behavior of le-proxy in a few cases.
svn:r1458
Nick Mathewson [Wed, 21 Oct 2009 19:21:05 +0000 (19:21 +0000)]
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
Nick Mathewson [Wed, 21 Oct 2009 18:48:22 +0000 (18:48 +0000)]
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
Nick Mathewson [Wed, 21 Oct 2009 07:00:19 +0000 (07:00 +0000)]
Test failing case of bufferevent_connect().
Code by Chris Davis.
svn:r1455
Nick Mathewson [Wed, 21 Oct 2009 07:00:14 +0000 (07:00 +0000)]
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
Nick Mathewson [Wed, 21 Oct 2009 06:03:00 +0000 (06:03 +0000)]
OSX compilation issues
svn:r1453
Nick Mathewson [Wed, 21 Oct 2009 05:36:27 +0000 (05:36 +0000)]
Remove an EVBASE_RELEASE_LOCK that I missed.
svn:r1452
Nick Mathewson [Wed, 21 Oct 2009 04:45:59 +0000 (04:45 +0000)]
Fix win32 compilation.
svn:r1451
Nick Mathewson [Wed, 21 Oct 2009 03:54:00 +0000 (03:54 +0000)]
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
Nick Mathewson [Wed, 21 Oct 2009 02:14:16 +0000 (02:14 +0000)]
Fix windows compilation warnings.
svn:r1449
Niels Provos [Mon, 19 Oct 2009 16:20:12 +0000 (16:20 +0000)]
Fix compilation for listener.h for C++ - missing extern "C". Patch from Ferenc Szalai.
svn:r1448
Nick Mathewson [Fri, 16 Oct 2009 13:20:16 +0000 (13:20 +0000)]
Add a note that we should change previous_to_last to last_with_space
svn:r1447