[or-cvs] start to track the "unexpected relay cell" warning

[arma@seul.org (Roger Dingledine)]

Update of /home/or/cvsroot/src/or
In directory moria.mit.edu:/home2/arma/work/onion/cvs/src/or

Modified Files:
	connection_edge.c onion.c 
Log Message:
start to track the 'unexpected relay cell' warning


Index: connection_edge.c
===================================================================
RCS file: /home/or/cvsroot/src/or/connection_edge.c,v
retrieving revision 1.71
retrieving revision 1.72
diff -u -d -r1.71 -r1.72
--- connection_edge.c	7 Dec 2003 22:03:47 -0000	1.71
+++ connection_edge.c	14 Dec 2003 06:50:44 -0000	1.72
@@ -6,6 +6,7 @@
 #include "tree.h"
 
 extern or_options_t options; /* command-line and config-file options */
+extern char *conn_state_to_string[][_CONN_TYPE_MAX+1];
 
 static int connection_ap_handshake_process_socks(connection_t *conn);
 static int connection_ap_handshake_attach_circuit(connection_t *conn);
@@ -194,7 +195,8 @@
       conn->has_sent_end = 1;
       return 0;
     } else {
-      log_fn(LOG_WARN,"Got an unexpected relay cell, not in 'open' state. Closing.");
+      log_fn(LOG_WARN,"Got an unexpected relay command %d, in state %d (%s). Closing.",
+             relay_command, conn->state, conn_state_to_string[conn->type][conn->state]);
       connection_edge_end(conn, END_STREAM_REASON_MISC, conn->cpath_layer);
       return -1;
     }
@@ -304,7 +306,7 @@
         return -1;
       }
       if (circuit_send_next_onion_skin(circ)<0) {
-        log_fn(LOG_WARN,"circuit_send_next_onion_skin() failed.");
+        log_fn(LOG_INFO,"circuit_send_next_onion_skin() failed.");
         return -1;
       }
       return 0;
@@ -842,7 +844,7 @@
   char *address;
   uint32_t addr;
   time_t expires;
-};      
+};
 static int client_dns_size = 0;
 static SPLAY_HEAD(client_dns_tree, client_dns_entry) client_dns_root;
 

Index: onion.c
===================================================================
RCS file: /home/or/cvsroot/src/or/onion.c,v
retrieving revision 1.110
retrieving revision 1.111
diff -u -d -r1.110 -r1.111
--- onion.c	14 Dec 2003 05:08:28 -0000	1.110
+++ onion.c	14 Dec 2003 06:50:44 -0000	1.111
@@ -477,6 +477,7 @@
     /* try the nodes in EntryNodes first */
     sl = smartlist_create(MAX_ROUTERS_IN_DIR);
     add_nickname_list_to_smartlist(sl,options.EntryNodes);
+    /* XXX one day, consider picking chosen_exit knowing what's in EntryNodes */
     remove_twins_from_smartlist(sl,router_get_by_nickname(state->chosen_exit));
     smartlist_subtract(sl,excludednodes);
     choice = smartlist_choose(sl);