Always initialize addr in parse_port_config()

It was always analyzed before use, but scan-build wasn't able to
persuade itself of that.

Closes ticket 28881.

diff --git a/changes/ticket28881 b/changes/ticket28881
new file mode 100644 (file)
index 0000000..1b015a6
--- /dev/null
+++ b/changes/ticket28881
@@ -0,0 +1,4 @@
+  o Code simplification and refactoring:
+    -  When parsing a port configuration, make it more
+       obvious to static analyzer tools that we will always initialize the
+       address. Closes ticket 28881.

diff --git a/src/app/config/config.c b/src/app/config/config.c
index 81cc3e378f193741bb7e2e1bc2004fb7fd74ce98..4a8f94da0e2098f67a704dc79c4813dbc66b1fa5 100644 (file)
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -6913,6 +6913,8 @@ parse_port_config(smartlist_t *out,
 
   for (; ports; ports = ports->next) {
     tor_addr_t addr;
+    tor_addr_make_unspec(&addr);
+
     int port;
     int sessiongroup = SESSION_GROUP_UNSET;
     unsigned isolation = ISO_DEFAULT;