[DTrace-devel] [PATCH] Avoid valgrind warning about conditional jump based on uninitialized value
    Kris Van Hees 
    kris.van.hees at oracle.com
       
    Wed Apr  8 08:00:35 PDT 2020
    
    
  
The dt_modsym_update_alt() has conditionals based on the content of
sym_name, which is initialized from  a sscanf() statement.  While the
implementation ensures that a value is assigned, valgrind does not seem
to know that.  By providing a default initializer value, complaints go
away.
Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
---
 libdtrace/dt_module.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libdtrace/dt_module.c b/libdtrace/dt_module.c
index d9eb3a73..e66567f9 100644
--- a/libdtrace/dt_module.c
+++ b/libdtrace/dt_module.c
@@ -1315,7 +1315,7 @@ dt_modsym_update_alt(dtrace_hdl_t *dtp, const char *line)
 	int sym_text;
 	dt_module_t *dmp;
 	dtrace_addr_range_t *range = NULL;
-	char sym_name[KSYM_NAME_MAX];
+	char sym_name[KSYM_NAME_MAX] = "";
 	char mod_name[PATH_MAX] = "vmlinux]";	/* note trailing ] */
 	int skip = 0;
 
-- 
2.26.0
    
    
More information about the DTrace-devel
mailing list