[tor-commits] [tor/master] Clean up a clangalyzer warning in directory_remove_invalid

nickm at torproject.org nickm at torproject.org
Mon Sep 15 17:54:07 UTC 2014


commit 1dc0d26b50929ad3969a6aa759ce71e0d089bb33
Author: Nick Mathewson <nickm at torproject.org>
Date:   Mon Sep 15 13:52:56 2014 -0400

    Clean up a clangalyzer warning in directory_remove_invalid
    
    "At this point in the code, msg has been set to a string
    constant. But the tor code checks that msg is not NULL, and the
    redundant NULL check confuses the analyser[...] To avoid this
    spurious warning, the patch initialises msg to NULL."
    
    Patch from teor. another part of 13157.
---
 src/or/dirserv.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index a5ad742..9131440 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -664,7 +664,7 @@ directory_remove_invalid(void)
   smartlist_add_all(nodes, nodelist_get_list());
 
   SMARTLIST_FOREACH_BEGIN(nodes, node_t *, node) {
-    const char *msg;
+    const char *msg = NULL;
     routerinfo_t *ent = node->ri;
     char description[NODE_DESC_BUF_LEN];
     uint32_t r;



More information about the tor-commits mailing list