[or-cvs] r10229: Address points in r10227. (in tor/trunk: . src/common src/or)

[nickm@xxxxxxxx]

Author: nickm
Date: 2007-05-20 13:43:55 -0400 (Sun, 20 May 2007)
New Revision: 10229

Modified:
   tor/trunk/
   tor/trunk/src/common/mempool.c
   tor/trunk/src/or/config.c
   tor/trunk/src/or/dns.c
   tor/trunk/src/or/router.c
   tor/trunk/src/or/routerlist.c
   tor/trunk/src/or/routerparse.c
Log:
 r13017@Kushana:  nickm | 2007-05-20 13:40:45 -0400
 Address points in r10227.



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r13017] on c95137ef-5f19-0410-b913-86e773d04f59

Modified: tor/trunk/src/common/mempool.c
===================================================================
--- tor/trunk/src/common/mempool.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/common/mempool.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -385,11 +385,11 @@
 {
   mp_chunk_t *chunk, **first_to_free;
   if (n < 0) {
+    /* As said in the documentation, "negative n" means "leave an additional
+     * -n chunks". So replace n with a positive number. */
     n = pool->min_empty_chunks + (-n);
     if (n < pool->n_empty_chunks)
       pool->min_empty_chunks = n;
-    /* XXX020 don't we want some sort of return here, given the
-     * assert that follows? -RD */
   }
   ASSERT(n>=0);
 

Modified: tor/trunk/src/or/config.c
===================================================================
--- tor/trunk/src/or/config.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/or/config.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -3119,9 +3119,7 @@
 
   if (argc > 1 && (!strcmp(argv[1],"--version"))) {
     char vbuf[128];
-    // XXX020 below, tor_svn_revision will always be defined, right?
-    // So we can take out that check? Also in router.c. -RD
-    if (tor_svn_revision != NULL && strlen(tor_svn_revision)) {
+    if (strlen(tor_svn_revision)) {
       tor_snprintf(vbuf, sizeof(vbuf), " (r%s)", tor_svn_revision);
     } else {
       vbuf[0] = 0;

Modified: tor/trunk/src/or/dns.c
===================================================================
--- tor/trunk/src/or/dns.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/or/dns.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -562,6 +562,9 @@
       if (!exitconn->_base.marked_for_close) {
         connection_free(TO_CONN(exitconn));
         //XXX020 ... and we just leak exitconn otherwise? -RD
+        // If it's marked for close, it's on closeable_connection_lst in
+        // main.c.  If it's on the closeable list, it will get freed from
+        // main.c. -NM
       }
       break;
     default:

Modified: tor/trunk/src/or/router.c
===================================================================
--- tor/trunk/src/or/router.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/or/router.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -1249,8 +1249,7 @@
 get_platform_str(char *platform, size_t len)
 {
   char svn_version_buf[128];
-  if (!strcmpend(VERSION, "-dev") &&
-      tor_svn_revision != NULL && strlen(tor_svn_revision)) {
+  if (!strcmpend(VERSION, "-dev") && strlen(tor_svn_revision)) {
     tor_snprintf(svn_version_buf, sizeof(svn_version_buf), " (r%s)",
                  tor_svn_revision);
   } else {

Modified: tor/trunk/src/or/routerlist.c
===================================================================
--- tor/trunk/src/or/routerlist.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/or/routerlist.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -4856,8 +4856,8 @@
   if (ei->bad_sig)
     return 1;
 
-  /* XXX020 below we should explain why this is strcmp and not strcasecmp,
-   * since it differs from how we usually compare nicknames. -RD */
+  /* The nickname must match exactly to have been generated at the same time
+   * by the same rotuer.  */
   if (strcmp(ri->nickname, ei->nickname) ||
       memcmp(ri->cache_info.identity_digest, ei->cache_info.identity_digest,
              DIGEST_LEN))

Modified: tor/trunk/src/or/routerparse.c
===================================================================
--- tor/trunk/src/or/routerparse.c	2007-05-20 17:43:29 UTC (rev 10228)
+++ tor/trunk/src/or/routerparse.c	2007-05-20 17:43:55 UTC (rev 10229)
@@ -2142,8 +2142,6 @@
   for (i = 0; table[i].t; ++i) {
     if (counts[table[i].v] < table[i].min_cnt) {
       log_warn(LD_DIR, "Parse error: missing %s element.", table[i].t);
-      tor_assert(0); /* XXX020 is this assert a remote crash waiting to
-                      * happen? -RD */
       return -1;
     }
     if (counts[table[i].v] > table[i].max_cnt) {