diff options
| author | Ruslan Kabatsayev <b7.10110111@gmail.com> | 2017-12-30 22:14:41 +0300 |
|---|---|---|
| committer | Ruslan Kabatsayev <b7.10110111@gmail.com> | 2017-12-31 11:55:19 +0300 |
| commit | 47fea877452b84b94ac6ffb26f194f12845526fa (patch) | |
| tree | 26d5c3d1db3a6b10c3be66998ead02a1f130a648 /gdb/psymtab.c | |
| parent | cfa27c399ec9236a100ef794505d35f60da41a6d (diff) | |
Avoid indexing std::vector past the end
The code here wants to find address of an element, and often this
element is one past the end of std::vector. Dereferencing that element
leads to undefined behavior, so it's better to simply use pointer
arithmetic instead of taking address of invalid dereference.
gdb/ChangeLog:
* psymtab.c (recursively_search_psymtabs): Use pointer arithmetic
instead of dereferencing std::vector past the end.
Diffstat (limited to 'gdb/psymtab.c')
| -rw-r--r-- | gdb/psymtab.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gdb/psymtab.c b/gdb/psymtab.c index c87ef25d61..1271e18296 100644 --- a/gdb/psymtab.c +++ b/gdb/psymtab.c @@ -1337,21 +1337,21 @@ recursively_search_psymtabs } partial_symbol **gbound - = &objfile->global_psymbols[ps->globals_offset + ps->n_global_syms]; + = objfile->global_psymbols.data () + ps->globals_offset + ps->n_global_syms; partial_symbol **sbound - = &objfile->static_psymbols[ps->statics_offset + ps->n_static_syms]; + = objfile->static_psymbols.data () + ps->statics_offset + ps->n_static_syms; partial_symbol **bound = gbound; /* Go through all of the symbols stored in a partial symtab in one loop. */ - partial_symbol **psym = &objfile->global_psymbols[ps->globals_offset]; + partial_symbol **psym = objfile->global_psymbols.data () + ps->globals_offset; while (keep_going) { if (psym >= bound) { if (bound == gbound && ps->n_static_syms != 0) { - psym = &objfile->static_psymbols[ps->statics_offset]; + psym = objfile->static_psymbols.data () + ps->statics_offset; bound = sbound; } else |