ReVanced-Mirror/Apktool
1
0
Fork 0
mirror of https://github.com/revanced/Apktool.git synced 2024-10-12 15:58:33 +02:00
Code Issues Projects Releases Wiki Activity

Allow but ignore multiple exception handlers for the same exception

Browse Source
This commit is contained in:
Ben Gruver 2014-07-21 18:54:30 -07:00 committed by Connor Tumbleson
parent 3d2e935f08
commit 394bb25b7c
2 changed files with 5 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
brut.apktool.smali/dexlib2/src/main/java/org/jf/dexlib2/writer/util/TryListBuilder.java
View File

@ -166,7 +166,6 @@ public class TryListBuilder<EH extends ExceptionHandler>
String existingType = existingHandler.getExceptionType();
String newType = handler.getExceptionType();
// Don't add it if we already have a handler of the same type
if (existingType == null) {
if (newType == null) {
if (existingHandler.getHandlerCodeAddress() != handler.getHandlerCodeAddress()) {
@ -176,10 +175,9 @@ public class TryListBuilder<EH extends ExceptionHandler>
return;
}
} else if (existingType.equals(newType)) {
if (existingHandler.getHandlerCodeAddress() != handler.getHandlerCodeAddress()) {
throw new InvalidTryException(
"Multiple overlapping catches for %s with different handlers", existingType);
}
// dalvik doesn't reject cases when there are multiple catches with the same exception
// but different handlers. In practice, the first handler "wins". Since the later
// handler will never be used, we don't add it.
return;
}
}

8
brut.apktool.smali/dexlib2/src/test/java/org/jf/dexlib2/writer/util/TryListBuilderTest.java
View File

@ -449,12 +449,8 @@ public class TryListBuilderTest {
TryListBuilder tlb = new TryListBuilder();
tlb.addHandler(5, 10, new ImmutableExceptionHandler("LException1;", 5));
try {
tlb.addHandler(0, 15, new ImmutableExceptionHandler("LException1;", 6));
} catch (TryListBuilder.InvalidTryException ex) {
return;
}
Assert.fail();
tlb.addHandler(0, 15, new ImmutableExceptionHandler("LException1;", 6));
// no exception should be thrown...
}
@Test