From 6f45101327592333dcc54e08800fbc2cb68ccd49 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Fri, 30 Jul 2010 18:55:24 -0400 Subject: Clear cell queues when marking or truncating a circuit. At best, this patch helps us avoid sending queued relayed cells that would get ignored during the time between when a destroy cell is sent and when the circuit is finally freed. At worst, it lets us release some memory a little earlier than it would otherwise. Fix for bug #1184. Bugfix on 0.2.0.1-alpha. --- src/or/connection_or.c | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/or/connection_or.c') diff --git a/src/or/connection_or.c b/src/or/connection_or.c index 405df1578b..c94325a5b7 100644 --- a/src/or/connection_or.c +++ b/src/or/connection_or.c @@ -1291,10 +1291,6 @@ connection_or_send_destroy(circid_t circ_id, or_connection_t *conn, int reason) cell.payload[0] = (uint8_t) reason; log_debug(LD_OR,"Sending destroy (circID %d).", circ_id); - /* XXXX It's possible that under some circumstances, we want the destroy - * to take precedence over other data waiting on the circuit's cell queue. - */ - connection_or_write_cell_to_buf(&cell, conn); return 0; } -- cgit v1.2.3-54-g00ecf