save 3-7 bytes per edge connection

svn:r7044

diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c
index 6e971328af2e4c87880b1e8cc934cd40cd391180..dac0a5c8934f0708de2bd2cfde0dd6a03afaa241 100644 (file)
--- a/src/or/connection_edge.c
+++ b/src/or/connection_edge.c
@@ -417,7 +417,8 @@ connection_ap_expire_beginning(void)
     circ->timestamp_dirty -= options->MaxCircuitDirtiness;
     /* give our stream another 'cutoff' seconds to try */
     conn->_base.timestamp_lastread += cutoff;
-    conn->num_socks_retries++;
+    if (conn->num_socks_retries < 250) /* avoid overflow */
+      conn->num_socks_retries++;
     /* move it back into 'pending' state, and try to attach. */
     if (connection_ap_detach_retriable(conn, TO_ORIGIN_CIRCUIT(circ))<0) {
       connection_mark_unattached_ap(conn, END_STREAM_REASON_CANT_ATTACH);

diff --git a/src/or/or.h b/src/or/or.h
index b07cec267bc388a99ba4b14e4824c894b40e68ff..cb21a3c0bd3a60f48a8bce2c7b02224c0dd8f33d 100644 (file)
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -713,7 +713,7 @@ typedef struct edge_connection_t {
   /** Number of times we've reassigned this application connection to
    * a new circuit. We keep track because the timeout is longer if we've
    * already retried several times. */
-  int num_socks_retries;
+  uint8_t num_socks_retries;
 
   /** Nickname of planned exit node -- used with .exit support. */
   char *chosen_exit_name;