[tor-commits] [tor/master] Defensive programming: null [pn]_chan, circ_id in circuit_mark_for_close_

nickm at torproject.org nickm at torproject.org
Sun Mar 23 04:20:53 UTC 2014


commit d769cab3e50979807a526b3ebc5c341c13b12e97
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed Mar 12 10:57:08 2014 -0400

    Defensive programming: null [pn]_chan,circ_id in circuit_mark_for_close_
    
    Doing this as part of the patch for #9683 to prevent possible bugs
    down the line
---
 src/or/circuitlist.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/or/circuitlist.c b/src/or/circuitlist.c
index 0015a68..867a9cd 100644
--- a/src/or/circuitlist.c
+++ b/src/or/circuitlist.c
@@ -1594,6 +1594,7 @@ circuit_mark_for_close_, (circuit_t *circ, int reason, int line,
       channel_send_destroy(circ->n_circ_id, circ->n_chan, reason);
     }
     circuitmux_detach_circuit(circ->n_chan->cmux, circ);
+    circuit_set_n_circid_chan(circ, 0, NULL);
   }
 
   if (! CIRCUIT_IS_ORIGIN(circ)) {
@@ -1627,6 +1628,7 @@ circuit_mark_for_close_, (circuit_t *circ, int reason, int line,
         channel_send_destroy(or_circ->p_circ_id, or_circ->p_chan, reason);
       }
       circuitmux_detach_circuit(or_circ->p_chan->cmux, circ);
+      circuit_set_p_circid_chan(or_circ, 0, NULL);
     }
   } else {
     origin_circuit_t *ocirc = TO_ORIGIN_CIRCUIT(circ);





More information about the tor-commits mailing list