diff options
| author | Don Breazeal <donb@codesourcery.com> | 2016-03-15 15:18:28 -0700 |
|---|---|---|
| committer | Don Breazeal <donb@codesourcery.com> | 2016-03-15 15:25:15 -0700 |
| commit | 1cafadb4e4eb329dea01355c6632b3077113d80c (patch) | |
| tree | 2aa982ab01ff1f0649de210f67bdada496a42e0a /gdb/ChangeLog | |
| parent | 79427bd2f8edb59110f16a9c1652e3d6e41e08e7 (diff) | |
| download | gdb-1cafadb4e4eb329dea01355c6632b3077113d80c.zip gdb-1cafadb4e4eb329dea01355c6632b3077113d80c.tar.gz gdb-1cafadb4e4eb329dea01355c6632b3077113d80c.tar.bz2 | |
PR 18303, Tolerate malformed input for lookup_symbol-called functions
lookup_symbol is often called with user input. Consequently, any
function called from lookup_symbol{,_in_language} should attempt to
deal with malformed input gracefully. After all, malformed user
input is not a programming/API error.
This patch does not attempt to find/correct all instances of this. It
only fixes locations in the code that trigger test suite failures.
This patch fixes PR breakpoints/18303, "Assertion: -breakpoint-insert
with windows paths of file in non-current directory".
The patch includes three new tests related to this. One is just
gdb.linespec/ls-errs.exp copied and converted to use C++ instead of C, and
to add a case using a file name containing a Windows-style logical drive
specifier. The others include an MI test to provide a regression test for
the specific case reported in PR 18303, and a C++ test for proper error
handling of access to a program variable when using a file scope specifier
that refers to a non-existent file.
Tested on x86_64 native Linux.
gdb/ChangeLog
2016-01-28 Keith Seitz <keiths@redhat.com>
PR breakpoints/18303
* cp-namespace.c (cp_lookup_bare_symbol): Change assertion to
look for "::" instead of simply ":".
(cp_search_static_and_baseclasses): Return null_block_symbol for
malformed input.
Remove assertions.
* cp-support.c (cp_find_first_component_aux): Do not return
a prefix length for ':' unless the next character is also ':'.
gdb/testsuite/ChangeLog
2016-01-28 Don Breazeal <donb@codesourcery.com>
* gdb.cp/scope-err.cc: New test program.
* gdb.cp/scope-err.exp: New test script.
* gdb.linespec/ls-errs.c (myfunction): Expanded to have multiple
lines and "set breakpoint here" comment.
* gdb.linespec/ls-errs.exp: Added C++ testing and new test case.
Fixed some whitespace and format issues.
* gdb.mi/mi-linespec-err-cp.cc: New test program.
* gdb.mi/mi-linespec-err-cp.exp: New test script.
Diffstat (limited to 'gdb/ChangeLog')
| -rw-r--r-- | gdb/ChangeLog | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 35e7eee..63909dd 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,14 @@ +2016-03-15 Keith Seitz <keiths@redhat.com> + + PR breakpoints/18303 + * cp-namespace.c (cp_lookup_bare_symbol): Change assertion to + look for "::" instead of simply ":". + (cp_search_static_and_baseclasses): Return null_block_symbol for + malformed input. + Remove assertions. + * cp-support.c (cp_find_first_component_aux): Do not return + a prefix length for ':' unless the next character is also ':'. + 2016-03-15 Doug Evans <dje@google.com> * features/aarch64-core.xml (cpsr_flags): New flags type. |