[tor-commits] [tlsdate/master] Don't log raw dates, in case they contain junk.

ioerror at torproject.org ioerror at torproject.org
Thu Oct 31 10:25:58 UTC 2013


commit 2dafddce642c497477c27a3f7a005be2865755f2
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Oct 10 15:08:33 2013 -0400

    Don't log raw dates, in case they contain junk.
---
 src/tlsdate-helper.c |   15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/tlsdate-helper.c b/src/tlsdate-helper.c
index 2e0e2fb..a5017d5 100644
--- a/src/tlsdate-helper.c
+++ b/src/tlsdate-helper.c
@@ -215,6 +215,19 @@ write_all_to_bio(BIO *bio, const char *string)
   return 1;
 }
 
+/* If the string is all nice clean ascii that it's safe to log, return
+ * it. Otherwise return a placeholder "This is junk" string. */
+static const char *
+sanitize_string(const char *s)
+{
+  const unsigned char *cp;
+  for (cp = (const unsigned char *)s; *cp; cp++) {
+    if (*cp < 32 || *cp > 127)
+      return "string with invalid characters";
+  }
+  return s;
+}
+
 static int
 handle_date_line(const char *dateline, uint32_t *result)
 {
@@ -241,7 +254,7 @@ handle_date_line(const char *dateline, uint32_t *result)
     return 0;
 
   dateline += 8;
-  verb("V: The alleged date is <%s>\n", dateline);
+  verb("V: The alleged date is <%s>\n", sanitize_string(dateline));
 
   while (*dateline == ' ')
     ++dateline;





More information about the tor-commits mailing list