Browse Source

Merge remote-tracking branch 'public/feature21496'

Nick Mathewson 8 years ago
parent
commit
a783c5cbae
2 changed files with 7 additions and 0 deletions
  1. 4 0
      changes/bug21496
  2. 3 0
      src/or/routerparse.c

+ 4 - 0
changes/bug21496

@@ -0,0 +1,4 @@
+  o Minor features (safety):
+    - Add an explict check to extrainfo_parse_entry_from_string() for NULL
+      inputs. We don't believe this can actually happen, but it may help
+      silence a warning from the Clang analyzer. Closes ticket 21496.

+ 3 - 0
src/or/routerparse.c

@@ -2030,6 +2030,9 @@ extrainfo_parse_entry_from_string(const char *s, const char *end,
    * parse that's covered by the hash. */
   int can_dl_again = 0;
 
+  if (BUG(s == NULL))
+    return NULL;
+
   if (!end) {
     end = s + strlen(s);
   }