[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