[tor-bugs] #6475 [Tor Client]: circuit_send_next_onion_skin(): Bug: Unexpectedly high circuit_successes

[Tor Bug Tracker & Wiki]

#6475: circuit_send_next_onion_skin(): Bug: Unexpectedly high circuit_successes
-----------------------------+----------------------------------------------
 Reporter:  grarpamp         |          Owner:  mikeperry         
     Type:  defect           |         Status:  needs_revision    
 Priority:  major            |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Client       |        Version:  Tor: 0.2.3.19-rc  
 Keywords:  MikePerry201208  |         Parent:                    
   Points:                   |   Actualpoints:                    
-----------------------------+----------------------------------------------

Comment(by mikeperry):

 nickm: In circuit_finish_handshake() there are log messages and side
 effects (the 'hop' assignment) outside of the main codepath. Should I just
 duplicate the checks in the new function?

 Also, this has been running all night fetching hidserv urls and I haven't
 reproduced or even hit any of the log messages. My tor client has built
 about 170 circuits of hidserv fetches since I started last night... I am
 just doing a few wgets in a shell loop.

 grarpamp: Can you perhaps try reproducing with my branch? Or would you
 like me to directly attach a diff after addressing nick's comments?

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