[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