Fix maybe-uninitialized warning in frame.c
authorTom Tromey <tom@tromey.com>
Fri, 31 Mar 2023 15:17:13 +0000 (09:17 -0600)
committerTom Tromey <tom@tromey.com>
Fri, 31 Mar 2023 15:20:01 +0000 (09:20 -0600)
A recent patch caused my system gcc (Fedora 36, so gcc 12.2.1) to warn
about sym_addr being possibly uninitialized in frame.c.  It isn't, but
the compiler can't tell.  So, this patch initializes the variable.  I
also fixed a formatting buglet that I missed in review.

gdb/frame.c

index 4f420f138f61e20fde0a1da08ff2ad3653d996de..36fb02f3c8eca4dace8e8c94c99323bab803158a 100644 (file)
@@ -2537,7 +2537,7 @@ inside_main_func (frame_info_ptr this_frame)
   if (current_program_space->symfile_object_file == nullptr)
     return false;
 
-  CORE_ADDR sym_addr;
+  CORE_ADDR sym_addr = 0;
   const char *name = main_name ();
   bound_minimal_symbol msymbol
     = lookup_minimal_symbol (name, NULL,
@@ -2573,8 +2573,9 @@ inside_main_func (frame_info_ptr this_frame)
 
   /* Convert any function descriptor addresses into the actual function
      code address.  */
-  sym_addr = gdbarch_convert_from_func_ptr_addr
-    (get_frame_arch (this_frame), sym_addr, current_inferior ()->top_target ());
+  sym_addr = (gdbarch_convert_from_func_ptr_addr
+             (get_frame_arch (this_frame), sym_addr,
+              current_inferior ()->top_target ()));
 
   return sym_addr == get_frame_func (this_frame);
 }