dnsproxy: Conditional jump depends on uninitialized value

This came from Daniel when running ConnMan using valgrind

==20635== Conditional jump or move depends on uninitialised value(s)
==20635==    at 0x49ECF4: get_name (dnsproxy.c:867)
==20635==    by 0x49F3F7: parse_response (dnsproxy.c:1096)
==20635==    by 0x49FD40: cache_update (dnsproxy.c:1410)
==20635==    by 0x4A0FF3: forward_dns_reply (dnsproxy.c:1803)
==20635==    by 0x4A15D9: udp_server_event (dnsproxy.c:1912)
==20635==    by 0x3331E47A54: g_main_context_dispatch (gmain.c:2715)
==20635==    by 0x3331E47D87: g_main_context_iterate.isra.24 (gmain.c:3290)
==20635==    by 0x3331E48181: g_main_loop_run (gmain.c:3484)
==20635==    by 0x44B600: main (main.c:670)

diff --git a/src/dnsproxy.c b/src/dnsproxy.c
index 8ed34a7a3b59cde16ee1075ae230f3fbba153a48..4ad4eb666af99cc100857eff16bec471a8223615 100644 (file)
--- a/src/dnsproxy.c
+++ b/src/dnsproxy.c
@@ -1083,7 +1083,7 @@ static int parse_response(unsigned char *buf, int buflen,
                         * of the alias and cache that.
                         */
                        unsigned char *end = NULL;
-                       int name_len = 0, output_len;
+                       int name_len = 0, output_len = 0;
 
                        memset(rsp, 0, sizeof(rsp));
                        rsp_len = sizeof(rsp) - 1;