diff options
| author |   Sergio Durigan Junior <sergiodj@redhat.com> | 2017-12-08 15:33:55 -0500 |
|---|---|---|
| committer | Sergio Durigan Junior <sergiodj@redhat.com> | 2017-12-08 15:33:55 -0500 |
| commit | 92469284a650232768523564f2c715c4ebb57906 (patch) | |
| tree | 3393e764d2e67ab28c03c95f1cf4421881f44094 /gdb/dtrace-probe.c | |
| parent | Clear non-significant bits of address in watchpoint (diff) | |
| download | binutils-gdb-92469284a650232768523564f2c715c4ebb57906.tar.gz binutils-gdb-92469284a650232768523564f2c715c4ebb57906.tar.bz2 binutils-gdb-92469284a650232768523564f2c715c4ebb57906.zip | |
Fix thinko on dtrace-probe.c:dtrace_process_dof_probe
While investigating PR gdb/22557 ("Regression:
gdb.base/dtrace-probe.exp"), I noticed that the code is wrongly
declaring a new "expression_up" variable inside the TRY block in
"dtrace_process_dof_probe". This causes the outter "expr" variable to
be empty, which may have an impact later when evaluating the
expression.
This commit fixes that. Unfortunately the script used to test DTrace
probes (gdb/testsuite/lib/pdtrace.in) is not very reliable so I cannot
say whether this commit fixes the PR mentioned above. Nonetheless,
it's an obvious fix and should go in.
gdb/ChangeLog:
2017-12-08 Sergio Durigan Junior <sergiodj@redhat.com>
* dtrace-probe.c (dtrace_process_dof_probe): Do not declare a new
"expression_up" inside the TRY block.
Diffstat (limited to 'gdb/dtrace-probe.c')
| -rw-r--r-- | gdb/dtrace-probe.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/gdb/dtrace-probe.c b/gdb/dtrace-probe.c index 1c88f890540..3314445f98a 100644 --- a/gdb/dtrace-probe.c +++ b/gdb/dtrace-probe.c @@ -486,17 +486,16 @@ dtrace_process_dof_probe (struct objfile *objfile, TRY { - expression_up expr - = parse_expression_with_language (type_str.c_str (), - language_c); + expr = parse_expression_with_language (type_str.c_str (), + language_c); } CATCH (ex, RETURN_MASK_ERROR) { } END_CATCH - if (expr != NULL && expr->elts[0].opcode == OP_TYPE) - type = expr->elts[1].type; + if (expr != NULL && expr.get ()->elts[0].opcode == OP_TYPE) + type = expr.get ()->elts[1].type; args.emplace_back (type, std::move (type_str), std::move (expr)); } |