[tor-bugs] #1184 [Tor - Tor client]: Sending useless relay cells after the destroy cell

[Tor Bug Tracker & Wiki]

#1184: Sending useless relay cells after the destroy cell
--------------------------------+-------------------------------------------
 Reporter:  arma                |         Type:  defect          
   Status:  needs_review        |     Priority:  minor           
Milestone:  Tor: 0.2.2.x-final  |    Component:  Tor - Tor client
  Version:  0.2.2.6-alpha       |   Resolution:  None            
 Keywords:                      |       Parent:                  
--------------------------------+-------------------------------------------

Comment(by nickm):

 Well, for all cases except the TRUNCATE case, it (might) improve
 conformance with the spec, which frowns on sending any cells once we've
 gotten a destroy.  Even after fixing 1784, we won't change anything in the
 non-TRUNCATE cases.

 In the TRUNCATE case, the patch (maybe) trades a severely nonconformant
 broken behavior for a less visibly nonconformant  broken behavior.
 Currently, on a TRUNCATE, we send a DESTROY, maybe send a couple of queued
 cells after the destroy (bug 1184, fixed here), and  drop queued cells
 when we close the circuit (bug 1784).  This patch fixes the "We might send
 cells on a destroyed circuit" problem, and leaves the "TRUNCATE destroys
 the circuit too early" problem for later.

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