[tor-commits] [tor/master] Add some comments to or_circuit_t.workqueue_entry

[nickm@xxxxxxxxxxxxxx]

commit 9667b2b88a1f8e9af08704c8bc95b8399a6e4145
Author: Sebastian Hahn <sebastian@xxxxxxxxxxxxxx>
Date:   Mon Feb 9 16:11:45 2015 +0100

    Add some comments to or_circuit_t.workqueue_entry
    
    These were suggested by dgoulet, thanks!
---
 src/or/cpuworker.c |    1 +
 src/or/or.h        |    3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/or/cpuworker.c b/src/or/cpuworker.c
index 7fe2351..c5e78ed 100644
--- a/src/or/cpuworker.c
+++ b/src/or/cpuworker.c
@@ -556,6 +556,7 @@ cpuworker_cancel_circ_handshake(or_circuit_t *circ)
     tor_free(job);
     tor_assert(total_pending_tasks > 0);
     --total_pending_tasks;
+    /* if (!job), this is done in cpuworker_onion_handshake_replyfn. */
     circ->workqueue_entry = NULL;
   }
 }
diff --git a/src/or/or.h b/src/or/or.h
index 520b7db..5d70798 100644
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -3126,7 +3126,8 @@ typedef struct or_circuit_t {
    * chance to give an onionskin to a cpuworker. Used only in onion.c */
   struct onion_queue_t *onionqueue_entry;
   /** Pointer to a workqueue entry, if this circuit has given an onionskin to
-   * a cpuworker and is waiting for a response. Used only in cpuworker.c */
+   * a cpuworker and is waiting for a response. Used to decide whether it is
+   * safe to free a circuit or if it is still in use by a cpuworker. */
   struct workqueue_entry_s *workqueue_entry;
 
   /** The circuit_id used in the previous (backward) hop of this circuit. */

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits