[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: classfile parsing bug?
Ondrej Lhotak wrote:
> Some fixes to jsr elimination have been made since Soot 2.1.0. Do you
> get the same error when processing that file with the Subversion trunk?
Ah, this commit looks very promising... :-)
> r1396 | jjorge1 | 2004-02-17 22:29:20 -0600 (Tue, 17 Feb 2004) | 13 lines
>
> When comparing the bytecode ranges of inlined subroutines with
> exception table entries, get the originalIndex of the first
> instruction in the subroutine from the successor of the astore
> instruction that begins the original subroutine, rather than from
> the first instruction in the cloned code.
>
> This change fixes a bug I introduced in revision 1126: inlining
> an empty subroutine was triggering a NullPointerException
> because in an empty subroutine the first instruction is not
> cloned from the original code, but is instead a brand new goto
> replacing the subroutine's ret; the goto has a bogus default
> value for originalIndex.
Thanks,
-Archie
__________________________________________________________________________
Archie Cobbs * CTO, Awarix * http://www.awarix.com