[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #5129 [Tor]: Avoid fcntl(O_NONBLOCK) calls on Linux
#5129: Avoid fcntl(O_NONBLOCK) calls on Linux
----------------------------+-----------------------------------------------
Reporter: nickm | Owner:
Type: enhancement | Status: needs_revision
Priority: minor | Milestone: Tor: 0.2.5.x-final
Component: Tor | Version:
Keywords: easy tor-relay | Parent:
Points: | Actualpoints:
----------------------------+-----------------------------------------------
Changes (by nickm):
* status: needs_review => needs_revision
Comment:
Okay, looks better. Three things I want to do before merging this:
* The eventdns changes should be wrapped in an #if 1 or something --
eventdns.c is imported code (snarfed from Libevent 1.x and then patched
sporadically with fixes from later Libevent versions), and we should
minimize our changes to it. In fact, unless the old code is _wrong_ , we
should really just leave it alone.
* In several places, the patch uses an #ifdef _WIN32 to decide whether to
call close() or closesocket(). Ideally, we should have a function or macro
for that. This would be tor_close_socket, but without the socket
accounting.
* This needs a changes/ file.
I'll see if I can do these in the next day or two, unless you want to.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/5129#comment:9>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs