diff options
| author | Tom Tromey <tom@tromey.com> | 2023-03-31 09:17:13 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2023-03-31 09:20:01 -0600 |
| commit | 5d49758d2c04dcf0d685f99e85a553e7c3891e34 (patch) | |
| tree | d48fda5458e52fbce0d910a86aa9ccabd2c80013 /gdb | |
| parent | 103409bb38502b95cb60475939a49c2b059673d5 (diff) | |
| download | gdb-5d49758d2c04dcf0d685f99e85a553e7c3891e34.zip gdb-5d49758d2c04dcf0d685f99e85a553e7c3891e34.tar.gz gdb-5d49758d2c04dcf0d685f99e85a553e7c3891e34.tar.bz2 | |
Fix maybe-uninitialized warning in frame.c
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.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/frame.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/gdb/frame.c b/gdb/frame.c index 4f420f1..36fb02f 100644 --- a/gdb/frame.c +++ b/gdb/frame.c @@ -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); } |