diff options
| author | Tom Tromey <tom@tromey.com> | 2019-01-02 11:41:44 -0700 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2019-01-03 15:39:56 -0700 |
| commit | 986041cda76a312e4285c9ba1e51b1778fcb15ce (patch) | |
| tree | b8fc1ae6140786bf205c2476892662144a976123 /gprof/configure | |
| parent | 7c711119166fc1c60c756059f39c6703d57eedbf (diff) | |
| download | gdb-986041cda76a312e4285c9ba1e51b1778fcb15ce.zip gdb-986041cda76a312e4285c9ba1e51b1778fcb15ce.tar.gz gdb-986041cda76a312e4285c9ba1e51b1778fcb15ce.tar.bz2 | |
Use xstrdup in tui_set_source_content
valgrind pointed out that the string copy in tui_set_source_content
was not allocating space for the trailing \0:
==3941== Invalid write of size 1
==3941== at 0x4C3239F: strcpy (vg_replace_strmem.c:512)
==3941== by 0x72036B: strcpy (string_fortified.h:90)
==3941== by 0x72036B: tui_set_source_content(symtab*, int, int) (tui-source.c:203)
Looking closer, I don't think there's a need to check the line width
here, so this patch changes it to use xstrdup.
Tested by re-running the TUI under valgrind. There are still other
valgrind reports from TUI code, but this one is gone.
gdb/ChangeLog
2019-01-03 Tom Tromey <tom@tromey.com>
* tui/tui-source.c (tui_set_source_content): Use xstrdup.
Diffstat (limited to 'gprof/configure')
0 files changed, 0 insertions, 0 deletions