diff options
| author | Markus Metzger <markus.t.metzger@intel.com> | 2016-02-11 11:07:09 +0100 |
|---|---|---|
| committer | Markus Metzger <markus.t.metzger@intel.com> | 2016-02-12 09:46:31 +0100 |
| commit | a038fa3e14a477d4d72a26c2e139fa47d2774be2 (patch) | |
| tree | 6cf48ff732e34f47017006f91ed34c014445454d /gdb/stack.c | |
| parent | 2f3ef606b91298855e9ea13ae0e0316c23f06c9b (diff) | |
stack: check frame_unwind_caller_id
Callers of frame_unwind_caller_* functions are supposed to check
frame_unwind_caller_id.
Add such a check to frame_info and treat an invalid caller ID as if the caller
PC were not available.
gdb/
* stack.c (frame_info): Check frame_unwind_caller_id.
Diffstat (limited to 'gdb/stack.c')
| -rw-r--r-- | gdb/stack.c | 39 |
1 files changed, 22 insertions, 17 deletions
diff --git a/gdb/stack.c b/gdb/stack.c index 89879f38ad..6e3acc7e40 100644 --- a/gdb/stack.c +++ b/gdb/stack.c @@ -1509,27 +1509,32 @@ frame_info (char *addr_exp, int from_tty) wrap_here (" "); printf_filtered ("saved %s = ", pc_regname); - TRY - { - caller_pc = frame_unwind_caller_pc (fi); - caller_pc_p = 1; - } - CATCH (ex, RETURN_MASK_ERROR) + if (!frame_id_p (frame_unwind_caller_id (fi))) + val_print_unavailable (gdb_stdout); + else { - switch (ex.error) + TRY { - case NOT_AVAILABLE_ERROR: - val_print_unavailable (gdb_stdout); - break; - case OPTIMIZED_OUT_ERROR: - val_print_not_saved (gdb_stdout); - break; - default: - fprintf_filtered (gdb_stdout, _("<error: %s>"), ex.message); - break; + caller_pc = frame_unwind_caller_pc (fi); + caller_pc_p = 1; } + CATCH (ex, RETURN_MASK_ERROR) + { + switch (ex.error) + { + case NOT_AVAILABLE_ERROR: + val_print_unavailable (gdb_stdout); + break; + case OPTIMIZED_OUT_ERROR: + val_print_not_saved (gdb_stdout); + break; + default: + fprintf_filtered (gdb_stdout, _("<error: %s>"), ex.message); + break; + } + } + END_CATCH } - END_CATCH if (caller_pc_p) fputs_filtered (paddress (gdbarch, caller_pc), gdb_stdout); |