diff options
| author | Pedro Alves <palves@redhat.com> | 2017-03-27 12:28:03 +0100 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2017-03-27 12:28:03 +0100 |
| commit | 210477268d4ac5cad937e811888a5c932206794c (patch) | |
| tree | 70565a15b6b77a2cfe1d9c028d2b2f33c3b0544d /gdb/ChangeLog | |
| parent | 9b7539374617a94c2d646f49e1bbfc954b11891d (diff) | |
| download | gdb-210477268d4ac5cad937e811888a5c932206794c.zip gdb-210477268d4ac5cad937e811888a5c932206794c.tar.gz gdb-210477268d4ac5cad937e811888a5c932206794c.tar.bz2 | |
Fix gdb_xml_debug/gdb_xml_error ATTRIBUTE_PRINTF use
The declarations of gdb_xml_debug and gdb_xml_error are passing "0" as
"first-to-check" argument to ATTRIBUTE_PRINTF, as if they were va_args
functions. Consequently, the arguments to gdb_xml_debug /
gdb_xml_error aren't being checked against the format strings.
With that fixed, a couple obvious bugs are exposed, both fixed by this
commit.
gdb/ChangeLog:
2017-03-27 Pedro Alves <palves@redhat.com>
* xml-support.h (gdb_xml_debug): Pass a "first-to-check" argument
to ATTRIBUTE_PRINTF.
* solib-target.c (library_list_start_list): Print "string" not
"version".
* xml-tdesc.c (tdesc_start_field): Pass "field_name" to
gdb_xml_error call.
Diffstat (limited to 'gdb/ChangeLog')
| -rw-r--r-- | gdb/ChangeLog | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 82213e8..b4d995a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,14 @@ 2017-03-27 Pedro Alves <palves@redhat.com> + * xml-support.h (gdb_xml_debug): Pass a "first-to-check" argument + to ATTRIBUTE_PRINTF. + * solib-target.c (library_list_start_list): Print "string" not + "version". + * xml-tdesc.c (tdesc_start_field): Pass "field_name" to + gdb_xml_error call. + +2017-03-27 Pedro Alves <palves@redhat.com> + * dwarf2read.c (struct file_and_directory): New. (dwarf2_get_dwz_file): Adjust to use std::string. (dw2_get_file_names_reader): Adjust to use file_and_directory. |