Backport to 0.2.0.x: Never allow a circuit to be created with the same circid as...

svn:r16463

diff --git a/ChangeLog b/ChangeLog
index 87865dab95dfe3c37a7dd6bb5a5821f1b43a966f..868753b525e023e193049d13373a28a3453fa6d4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,9 @@
 Changes in version 0.2.0.31 - 2008-08-??
+ o Major bugfixes:
+    - Make sure that two circuits can never exist on the same connection
+      with the same circuit ID, even if one is marked for close.  This
+      is conceivably a bugfix for bug 779; fixes a bug on 0.1.0.4-rc.
+
  o Minor bugfixes:
     - Fix a small alignment and memory-wasting bug on buffer chunks.  Spotted
       by rovv.

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index bb302836f706d568654578b0f441042c221e75cc..3be26899954f0c8e4098f7e54ecdba813dff206b 100644 (file)
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -99,7 +99,7 @@ get_unique_circ_id_by_conn(or_connection_t *conn)
       return 0;
     }
     test_circ_id |= high_bit;
-  } while (circuit_get_by_circid_orconn(test_circ_id, conn));
+  } while (circuit_id_in_use_on_orconn(test_circ_id, conn));
   return test_circ_id;
 }
 

diff --git a/src/or/circuitlist.c b/src/or/circuitlist.c
index 7bd3829f622a298fdf57f929eecf3d97faa6e8a1..adb0d7867e364c140a761fd0258efe30b366904d 100644 (file)
--- a/src/or/circuitlist.c
+++ b/src/or/circuitlist.c
@@ -661,6 +661,14 @@ circuit_get_by_circid_orconn(uint16_t circ_id, or_connection_t *conn)
     return circ;
 }
 
+/** Return true iff the circuit ID <b>circ_id</b> is currently used by a
+ * circuit, marked or not, on <b>conn</b>. */
+int
+circuit_id_in_use_on_orconn(uint16_t circ_id, or_connection_t *conn)
+{
+  return circuit_get_by_circid_orconn_impl(circ_id, conn) != NULL;
+}
+
 /** Return the circuit that a given edge connection is using. */
 circuit_t *
 circuit_get_by_edge_conn(edge_connection_t *conn)

diff --git a/src/or/command.c b/src/or/command.c
index 552624fb76812afaa0a10a705e40a12d5a4f6273..f125f47690825fa030d636482d733f975fa04026 100644 (file)
--- a/src/or/command.c
+++ b/src/or/command.c
@@ -252,7 +252,7 @@ command_process_create_cell(cell_t *cell, or_connection_t *conn)
     return;
   }
 
-  if (circuit_get_by_circid_orconn(cell->circ_id, conn)) {
+  if (circuit_id_in_use_on_orconn(cell->circ_id, conn)) {
     routerinfo_t *router = router_get_by_digest(conn->identity_digest);
     log_fn(LOG_PROTOCOL_WARN, LD_PROTOCOL,
            "Received CREATE cell (circID %d) for known circ. "

diff --git a/src/or/or.h b/src/or/or.h
index e1a2fe6d9e6eceb246b9e7b7ffe66b0375ff3653..6f7d92cabf9462a66494a3824cb99fce3ad46c03 100644 (file)
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -2568,6 +2568,7 @@ origin_circuit_t *origin_circuit_new(void);
 or_circuit_t *or_circuit_new(uint16_t p_circ_id, or_connection_t *p_conn);
 circuit_t *circuit_get_by_circid_orconn(uint16_t circ_id,
                                         or_connection_t *conn);
+int circuit_id_in_use_on_orconn(uint16_t circ_id, or_connection_t *conn);
 circuit_t *circuit_get_by_edge_conn(edge_connection_t *conn);
 void circuit_unlink_all_from_or_conn(or_connection_t *conn, int reason);
 origin_circuit_t *circuit_get_by_global_id(uint32_t id);