diff options
| author | Gregory Anders <greg@gpanders.com> | 2023-09-01 16:02:17 -0500 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-09-20 10:59:32 -0600 |
| commit | 155f5df517c962f1317e119a6a5ca317e1202996 (patch) | |
| tree | a059ad9901fc19f17c476c132d5cde33c3fac5f0 | |
| parent | 4abf53c99115dad84720be64432ab4bd933b9df7 (diff) | |
| download | gdb-155f5df517c962f1317e119a6a5ca317e1202996.zip gdb-155f5df517c962f1317e119a6a5ca317e1202996.tar.gz gdb-155f5df517c962f1317e119a6a5ca317e1202996.tar.bz2 | |
gdb/dap: check for breakpoint source before unpacking
Not all breakpoints have a source location. For example, a breakpoint
set on a raw address will have only the "address" field populated, but
"source" will be None, which leads to a RuntimeError when attempting to
unpack the filename and line number.
Before attempting to unpack the filename and line number from the
breakpoint, ensure that the source information is not None. Also
populate the source and line information separately from the
"instructionReference" field, so that breakpoints that include only an
address are still included.
Approved-By: Tom Tromey <tom@tromey.com>
| -rw-r--r-- | gdb/python/lib/gdb/dap/breakpoint.py | 20 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.dap/bt-nodebug.exp | 7 |
2 files changed, 19 insertions, 8 deletions
diff --git a/gdb/python/lib/gdb/dap/breakpoint.py b/gdb/python/lib/gdb/dap/breakpoint.py index 1622ae6..bf06298 100644 --- a/gdb/python/lib/gdb/dap/breakpoint.py +++ b/gdb/python/lib/gdb/dap/breakpoint.py @@ -106,14 +106,18 @@ def _breakpoint_descriptor(bp): # multiple locations. See # https://github.com/microsoft/debug-adapter-protocol/issues/13 loc = bp.locations[0] - (filename, line) = loc.source - result.update( - { - "source": make_source(filename, os.path.basename(filename)), - "line": line, - "instructionReference": hex(loc.address), - } - ) + if loc.source: + (filename, line) = loc.source + result.update( + { + "source": make_source(filename, os.path.basename(filename)), + "line": line, + } + ) + + if loc.address: + result["instructionReference"] = hex(loc.address), + path = loc.fullname if path is not None: result["source"]["path"] = path diff --git a/gdb/testsuite/gdb.dap/bt-nodebug.exp b/gdb/testsuite/gdb.dap/bt-nodebug.exp index e9726d2..abd394a 100644 --- a/gdb/testsuite/gdb.dap/bt-nodebug.exp +++ b/gdb/testsuite/gdb.dap/bt-nodebug.exp @@ -53,4 +53,11 @@ gdb_assert {[llength $frames] == 3} "three frames" gdb_assert {[dict get [lindex $frames 1] name] == "no_debug_info"} \ "name of no-debug frame" +# Breakpoint can be set without source file information +set obj [dap_check_request_and_response "set breakpoint on no_debug_info" \ + setFunctionBreakpoints \ + {o breakpoints [a [o name [s no_debug_info]]]}] +set breakpoints [dict get [lindex $obj 0] body breakpoints] +gdb_assert {[dict exists [lindex $breakpoints 0] instructionReference]} "breakpoint has instructionReference" + dap_shutdown |