Package: vgrabbj
Version: 0.9.6-1
Severity: minor
Tags: patch

When running 'valgrind vgrabbj', this warning is displayed:

==21513== Conditional jump or move depends on uninitialised value(s)
==21513==    at 0x804E042: free_mmap (v_utils.c:46)
==21513==    by 0x80498AF: cleanup (vgrabbj.c:32)
==21513==    by 0x804993E: v_error (vgrabbj.c:124)
==21513==    by 0x804AA3E: check_files (v_config.c:280)
==21513==    by 0x804B87E: v_init (v_config.c:647)
==21513==    by 0x804A120: main (vgrabbj.c:350)

The reason is that vconf->map is not initialized by init_defaults().
Adding 'vconf->map = NULL;' to the function solve the issue, for
example like this:

diff -u vgrabbj-0.9.6/v_config.c vgrabbj-0.9.6/v_config.c
--- vgrabbj-0.9.6/v_config.c
+++ vgrabbj-0.9.6/v_config.c
@@ -183,6 +183,7 @@
   vconf->tmpout     = NULL;
   vconf->buffer     = NULL;
   vconf->o_buffer   = NULL;
+  vconf->map        = NULL;
   l_opt[idx++].var  = &vconf->debug;
   l_opt[idx++].var  = &vconf->loop;
   l_opt[idx++].var  = &vconf->loop;

Happy hacking,
-- 
Petter Reinholdtsen



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to