[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #11069 [Tor]: Tor with unreachable PTs should not hang the bootstrap process



#11069: Tor with unreachable PTs should not hang the bootstrap process
------------------------+--------------------------------
     Reporter:  asn     |      Owner:
         Type:  defect  |     Status:  needs_review
     Priority:  normal  |  Milestone:  Tor: 0.2.5.x-final
    Component:  Tor     |    Version:
   Resolution:          |   Keywords:  tor-pt tor-client
Actual Points:          |  Parent ID:
       Points:          |
------------------------+--------------------------------
Changes (by asn):

 * status:  new => needs_review


Comment:

 OK. See branch `bug11069` in
 `https://git.torproject.org/user/asn/tor.git`.

 After some testing it seems to work fine.

 A thing I'm not so sure about is that I moved
 `connection_or_notify_error()` after the connection has been marked for
 closing. There doesn't seem to be anything wrong with that, but that
 function also calls `channel_close_for_error()` and I'm not sure if the
 channel subsystem cares about whether the connection has been closed or
 not.

 Also, are there any other attributes of the connection that we should be
 checking in `any_active_or_conns()`? We could check whether its state is
 `OR_CONN_STATE_CONNECTING` for example, but I'm not sure if this is a good
 idea.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/11069#comment:11>
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