Initialize libsym early in trace.c to help the compiler.
authorMark Wielaard <mjw@redhat.com>
Thu, 9 Jan 2014 21:56:35 +0000 (22:56 +0100)
committerChanho Park <chanho61.park@samsung.com>
Fri, 22 Aug 2014 11:38:24 +0000 (20:38 +0900)
GCC 4.4.7 isn't smart enough to realize own_libsym will always be zero
when it sees the goto done which might jump over the initialization of
libsym. And so will produce a warning like:

cc1: warnings being treated as errors
trace.c: In function ‘ifunc_ret_hit’:
trace.c:1433: error: ‘libsym’ may be used uninitialized in this function

sysdeps/linux-gnu/trace.c

index e648b8f..0abb545 100644 (file)
@@ -1390,6 +1390,7 @@ ifunc_ret_hit(struct breakpoint *bp, struct process *proc)
 
        struct breakpoint *nbp = NULL;
        int own_libsym = 0;
+       struct library_symbol *libsym = NULL;
 
        struct value value;
        value_init(&value, proc, NULL, type_get_voidptr(), 0);
@@ -1430,8 +1431,7 @@ ifunc_ret_hit(struct breakpoint *bp, struct process *proc)
 
        /* Look if we already have a symbol with this address.
         * Otherwise create a new one.  */
-       struct library_symbol *libsym
-               = library_each_symbol(lib, NULL, libsym_at_address, &u.a);
+       libsym = library_each_symbol(lib, NULL, libsym_at_address, &u.a);
        if (libsym == NULL) {
                libsym = malloc(sizeof *libsym);
                char *name = strdup(bp->os.ret_libsym->name);