[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] substantive changes on nick"s rendezvous commit.



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

Modified Files:
	rendservice.c 
Log Message:
substantive changes on nick's rendezvous commit.
nick, can you fix these?


Index: rendservice.c
===================================================================
RCS file: /home2/or/cvsroot/tor/src/or/rendservice.c,v
retrieving revision 1.127
retrieving revision 1.128
diff -u -d -r1.127 -r1.128
--- rendservice.c	29 Jun 2005 21:46:55 -0000	1.127
+++ rendservice.c	14 Jul 2005 08:45:19 -0000	1.128
@@ -483,7 +483,7 @@
       rp_nickname = buf;
       version = 0;
     }
-    /* XXX when 0.1.0.x is obsolete, change this to reject version != 2. */
+    /* XXX when 0.1.0.x is obsolete, change this to reject version < 2. */
     ptr=memchr(rp_nickname,0,nickname_field_len);
     if (!ptr || ptr == rp_nickname) {
       log_fn(LOG_WARN, "Couldn't find a null-padded nickname in INTRODUCE2 cell");
@@ -593,6 +593,7 @@
       oldcirc->build_state->expiry_time < time(NULL)) {
     log_fn(LOG_INFO,"Attempt to build circuit to %s for rendezvous has failed too many times or expired; giving up.",
            oldcirc->build_state->chosen_exit->nickname);
+    /* XXX bug: if the first clause of the if triggers, we'll seg fault. */
     return;
   }
 
@@ -830,6 +831,7 @@
   while ((circ = circuit_get_next_by_pk_and_purpose(circ,pk_digest,
                                                   CIRCUIT_PURPOSE_S_INTRO))) {
     tor_assert(circ->cpath);
+    /* XXX this is a bug. ->nickname will always be there. */
     if (circ->build_state->chosen_exit->nickname &&
         !strcasecmp(circ->build_state->chosen_exit->nickname, router->nickname)) {
       return circ;
@@ -840,6 +842,7 @@
   while ((circ = circuit_get_next_by_pk_and_purpose(circ,pk_digest,
                                         CIRCUIT_PURPOSE_S_ESTABLISH_INTRO))) {
     tor_assert(circ->cpath);
+    /* XXX this is a bug. ->nickname will always be there. */
     if (circ->build_state->chosen_exit->nickname &&
         !strcasecmp(circ->build_state->chosen_exit->nickname, router->nickname)) {
       return circ;