diff options
author | sergiodj <sergiodj> | 2013-10-16 02:55:27 +0000 |
---|---|---|
committer | sergiodj <sergiodj> | 2013-10-16 02:55:27 +0000 |
commit | ce5cb58f8143845e96e73392ce2231b37b6e514b (patch) | |
tree | 223ce8e11bc7070d05ebe90531886291e56f05e4 /gdb/dwarf2read.c | |
parent | 33b0fe7a799d99cfd2a086e3a862ce4fbfa3d679 (diff) | |
download | gdb-ce5cb58f8143845e96e73392ce2231b37b6e514b.tar.gz |
There were two functions who were calling "sizeof" twice.
The first one, dw2_get_real_path from gdb/dwarf2read.c, was actually
making use of OBSTACK_CALLOC which already calls "sizeof" for its third
argument.
The second, download_tracepoint_1 from gdb/gdbserver/tracepoint.c, was
explicitly calling "sizeof" inside another "sizeof".
This patch fixed both functions.
gdb/ChangeLog
2013-10-16 Sergio Durigan Junior <sergiodj@redhat.com>
PR gdb/16014
* dwarf2read.c (dw2_get_real_path): Remove unnecessary call to
sizeof.
gdb/gdbserver/ChangeLog
2013-10-16 Sergio Durigan Junior <sergiodj@redhat.com>
PR gdb/16014
* tracepoint.c (download_tracepoint_1): Remove unnecessary double
call to sizeof.
Diffstat (limited to 'gdb/dwarf2read.c')
-rw-r--r-- | gdb/dwarf2read.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index aa109e0080b..3974d0b2f8a 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -3278,7 +3278,7 @@ dw2_get_real_path (struct objfile *objfile, { if (qfn->real_names == NULL) qfn->real_names = OBSTACK_CALLOC (&objfile->objfile_obstack, - qfn->num_file_names, sizeof (char *)); + qfn->num_file_names, char *); if (qfn->real_names[index] == NULL) qfn->real_names[index] = gdb_realpath (qfn->file_names[index]); |