[tor-bugs] #4519 [Tor]: circuit_unlink_all_from_orconn is too expensive

Tor Bug Tracker & Wiki blackhole at torproject.org
Tue Jun 18 13:05:47 UTC 2013


#4519: circuit_unlink_all_from_orconn is too expensive
----------------------------+-----------------------------------------------
 Reporter:  nickm           |          Owner:                  
     Type:  defect          |         Status:  needs_review    
 Priority:  major           |      Milestone:  Tor: unspecified
Component:  Tor             |        Version:                  
 Keywords:  easy tor-relay  |         Parent:                  
   Points:                  |   Actualpoints:                  
----------------------------+-----------------------------------------------
Changes (by marek):

  * status:  new => needs_review


Comment:

 If we want to be able to unlink `circuit_t` without the need of walking
 through all of the circuits, first we need to make `global_circuitlist` a
 doubly-linked list. Attached patch does just that, and improves some other
 minor things:

  - Doesn't expose `global_circuitlist` via 'extern', instead calls
 `circuit_get_global_list_()` to get the structure.
  - Removes unused `circuit_dump_by_chan` and its helper
 `circuit_dump_chan_details`.
  - Adds `#include tor_queue.h` to `or.h` as it's needed.

 I don't think this patch is enough to "resolve" this bug, but is
 definitely necessary.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/4519#comment:6>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list