]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
Use xstrdup in tui_set_source_content
authorTom Tromey <tom@tromey.com>
Wed, 2 Jan 2019 18:41:44 +0000 (11:41 -0700)
committerTom Tromey <tom@tromey.com>
Thu, 3 Jan 2019 22:39:56 +0000 (15:39 -0700)
commit986041cda76a312e4285c9ba1e51b1778fcb15ce
treeb8fc1ae6140786bf205c2476892662144a976123
parent7c711119166fc1c60c756059f39c6703d57eedbf
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.
gdb/ChangeLog
gdb/tui/tui-source.c