diff options
| author | Tom Tromey <tom@tromey.com> | 2019-06-23 16:07:12 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2019-06-25 07:48:51 -0600 |
| commit | f7952c5774671414d9e0e8d3524c2921daa6f28e (patch) | |
| tree | f6b30b00d226b3936e0b88166fcac16bb97a29f8 /gdb/tui/tui-winsource.c | |
| parent | 17568d782d96434537fe9698c5ebcb09f35c20cb (diff) | |
| download | fsf-binutils-gdb-f7952c5774671414d9e0e8d3524c2921daa6f28e.zip fsf-binutils-gdb-f7952c5774671414d9e0e8d3524c2921daa6f28e.tar.gz fsf-binutils-gdb-f7952c5774671414d9e0e8d3524c2921daa6f28e.tar.bz2 | |
Fix latent bug in set_is_exec_point_at
valgrind pointed out that the TUI was using uninitialized memory in
set_is_exec_point_at. The bug is a missing check against LOA_ADDRESS,
causing gdb to examine the uninitialized bits of the "addr" field.
gdb/ChangeLog
2019-06-25 Tom Tromey <tom@tromey.com>
* tui/tui-winsource.c
(tui_source_window_base::set_is_exec_point_at): Add check against
LOA_ADDRESS.
Diffstat (limited to 'gdb/tui/tui-winsource.c')
| -rw-r--r-- | gdb/tui/tui-winsource.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/tui/tui-winsource.c b/gdb/tui/tui-winsource.c index 93c6253..6ec1f1b 100644 --- a/gdb/tui/tui-winsource.c +++ b/gdb/tui/tui-winsource.c @@ -365,7 +365,7 @@ tui_source_window_base::set_is_exec_point_at (struct tui_line_or_address l) || content_loa.loa == LOA_ADDRESS); if (content_loa.loa == l.loa && ((l.loa == LOA_LINE && content_loa.u.line_no == l.u.line_no) - || (content_loa.u.addr == l.u.addr))) + || (l.loa == LOA_ADDRESS && content_loa.u.addr == l.u.addr))) new_state = true; else new_state = false; |