[or-cvs] r8545: Now, all log messages should be distinct. (in tor/trunk: . src/common src/or)

[nickm at seul.org]

Author: nickm
Date: 2006-09-29 18:51:47 -0400 (Fri, 29 Sep 2006)
New Revision: 8545

Modified:
   tor/trunk/
   tor/trunk/src/common/torgzip.c
   tor/trunk/src/or/relay.c
   tor/trunk/src/or/rendservice.c
   tor/trunk/src/or/router.c
   tor/trunk/src/or/routerparse.c
Log:
 r9032 at Kushana:  nickm | 2006-09-29 18:51:42 -0400
 Now, all log messages should be distinct.



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

Modified: tor/trunk/src/common/torgzip.c
===================================================================
--- tor/trunk/src/common/torgzip.c	2006-09-29 22:39:08 UTC (rev 8544)
+++ tor/trunk/src/common/torgzip.c	2006-09-29 22:51:47 UTC (rev 8545)
@@ -217,7 +217,7 @@
           goto err;
         }
         if (inflateInit2(stream, method_bits(method)) != Z_OK) {
-          log_warn(LD_GENERAL, "Error from inflateInit2: %s",
+          log_warn(LD_GENERAL, "Error from second inflateInit2: %s",
                    stream->msg?stream->msg:"<no message>");
           goto err;
         }

Modified: tor/trunk/src/or/relay.c
===================================================================
--- tor/trunk/src/or/relay.c	2006-09-29 22:39:08 UTC (rev 8544)
+++ tor/trunk/src/or/relay.c	2006-09-29 22:51:47 UTC (rev 8545)
@@ -589,7 +589,8 @@
     case END_STREAM_REASON_NET_UNREACHABLE:
       return SOCKS5_NET_UNREACHABLE;
     default:
-      log_warn(LD_PROTOCOL,"Reason for ending (%d) not recognized.",reason);
+      log_warn(LD_PROTOCOL,"Reason for ending (%d) not recognized; "
+               "sending generic socks error.",reason);
       return SOCKS5_GENERAL_ERROR;
   }
 }

Modified: tor/trunk/src/or/rendservice.c
===================================================================
--- tor/trunk/src/or/rendservice.c	2006-09-29 22:39:08 UTC (rev 8544)
+++ tor/trunk/src/or/rendservice.c	2006-09-29 22:51:47 UTC (rev 8545)
@@ -345,7 +345,8 @@
     /* Load key */
     if (strlcpy(fname,s->directory,sizeof(fname)) >= sizeof(fname) ||
         strlcat(fname,"/private_key",sizeof(fname)) >= sizeof(fname)) {
-      log_warn(LD_CONFIG, "Directory name too long: \"%s\".", s->directory);
+      log_warn(LD_CONFIG, "Directory name too long to store key file: \"%s\".",
+               s->directory);
       return -1;
     }
     s->private_key = init_key_from_file(fname);
@@ -363,7 +364,8 @@
     }
     if (strlcpy(fname,s->directory,sizeof(fname)) >= sizeof(fname) ||
         strlcat(fname,"/hostname",sizeof(fname)) >= sizeof(fname)) {
-      log_warn(LD_CONFIG, "Directory name too long: \"%s\".", s->directory);
+      log_warn(LD_CONFIG, "Directory name too long to store hostname file:"
+               " \"%s\".", s->directory);
       return -1;
     }
     tor_snprintf(buf, sizeof(buf),"%s.onion\n", s->service_id);

Modified: tor/trunk/src/or/router.c
===================================================================
--- tor/trunk/src/or/router.c	2006-09-29 22:39:08 UTC (rev 8544)
+++ tor/trunk/src/or/router.c	2006-09-29 22:51:47 UTC (rev 8545)
@@ -127,7 +127,7 @@
   tor_snprintf(fname_prev,sizeof(fname_prev),
            "%s/keys/secret_onion_key.old",get_options()->DataDirectory);
   if (!(prkey = crypto_new_pk_env())) {
-    log_err(LD_GENERAL,"Error creating crypto environment.");
+    log_err(LD_GENERAL,"Error constructing rotated onion key");
     goto error;
   }
   if (crypto_pk_generate_key(prkey)) {
@@ -187,7 +187,7 @@
   FILE *file = NULL;
 
   if (!(prkey = crypto_new_pk_env())) {
-    log_err(LD_GENERAL,"Error creating crypto environment.");
+    log_err(LD_GENERAL,"Error constructing key");
     goto error;
   }
 

Modified: tor/trunk/src/or/routerparse.c
===================================================================
--- tor/trunk/src/or/routerparse.c	2006-09-29 22:39:08 UTC (rev 8544)
+++ tor/trunk/src/or/routerparse.c	2006-09-29 22:51:47 UTC (rev 8545)
@@ -1016,12 +1016,14 @@
   strlcpy(rs->nickname, tok->args[0], sizeof(rs->nickname));
 
   if (digest_from_base64(rs->identity_digest, tok->args[1])) {
-    log_warn(LD_DIR, "Error decoding digest %s", escaped(tok->args[1]));
+    log_warn(LD_DIR, "Error decoding identity digest %s",
+             escaped(tok->args[1]));
     goto err;
   }
 
   if (digest_from_base64(rs->descriptor_digest, tok->args[2])) {
-    log_warn(LD_DIR, "Error decoding digest %s", escaped(tok->args[2]));
+    log_warn(LD_DIR, "Error decoding descriptor digest %s",
+             escaped(tok->args[2]));
     goto err;
   }
 
@@ -1747,7 +1749,7 @@
   char *start, *end;
   start = strstr(s, start_str);
   if (!start) {
-    log_warn(LD_DIR,"couldn't find \"%s\"",start_str);
+    log_warn(LD_DIR,"couldn't find start of hashed material \"%s\"",start_str);
     return -1;
   }
   if (start != s && *(start-1) != '\n') {
@@ -1758,7 +1760,7 @@
   }
   end = strstr(start+strlen(start_str), end_str);
   if (!end) {
-    log_warn(LD_DIR,"couldn't find \"%s\"",end_str);
+    log_warn(LD_DIR,"couldn't find end of hashed material \"%s\"",end_str);
     return -1;
   }
   end = strchr(end+strlen(end_str), '\n');