[AbRT PATCH] abrt-gdb-exploitable: print current instruction
Richard Marko
rmarko at redhat.com
Mon Sep 9 13:49:20 UTC 2013
Pushed, thanks!
On 09/01/2013 08:09 PM, Denys Vlasenko wrote:
> I've seen strange BZ created by abrt where "exploitable" element
> looked bogus. It claimed that the crash was jumping to bogus address
> yet crashing insn was a benign reg-to-reg move.
>
> This patch makes analyzer to record what it thinks current insn is.
>
> Signed-off-by: Denys Vlasenko <dvlasenk at redhat.com>
> ---
> src/plugins/abrt-gdb-exploitable | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/plugins/abrt-gdb-exploitable b/src/plugins/abrt-gdb-exploitable
> index 950797b..abc1fa4 100755
> --- a/src/plugins/abrt-gdb-exploitable
> +++ b/src/plugins/abrt-gdb-exploitable
> @@ -705,6 +705,8 @@ class AbrtExploitable(gdb.Command):
> f = open(args[1], 'w')
> f.write(_("Likely crash reason: ") + si.exploitable_desc + "\n")
> f.write(_("Exploitable rating (0-9 scale): ") + str(si.exploitable_rating) + "\n")
> + if si.current_instruction:
> + f.write(_("Current instruction: ") + si.current_instruction + "\n")
> else:
> sys.stderr.write(_("Exploitability analysis came up empty\n"))
>
--
Richard Marko
More information about the Crash-catcher
mailing list