fix: lower regions count limit (#354)

skylot · Sep 8, 2018 · 9be62fb · 9be62fb
1 parent f6f883b
commit 9be62fb
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/jadx-core/src/main/java/jadx/core/dex/visitors/regions/RegionMaker.java b/jadx-core/src/main/java/jadx/core/dex/visitors/regions/RegionMaker.java
@@ -55,16 +55,16 @@
 public class RegionMaker {
 	private static final Logger LOG = LoggerFactory.getLogger(RegionMaker.class);
 
-	// 'dumb' guard to prevent endless loop in regions processing
-	private static final int REGIONS_LIMIT = 1000 * 1000;
-
 	private final MethodNode mth;
+	private final int regionsLimit;
 	private int regionsCount;
 	private BitSet processedBlocks;
 
 	public RegionMaker(MethodNode mth) {
 		this.mth = mth;
-		this.processedBlocks = new BitSet(mth.getBasicBlocks().size());
+		int blocksCount = mth.getBasicBlocks().size();
+		this.processedBlocks = new BitSet(blocksCount);
+		this.regionsLimit = blocksCount * 100;
 	}
 
 	public Region makeRegion(BlockNode startBlock, RegionStack stack) {
@@ -84,7 +84,7 @@ public Region makeRegion(BlockNode startBlock, RegionStack stack) {
 		while (next != null) {
 			next = traverse(r, next, stack);
 			regionsCount++;
-			if (regionsCount > REGIONS_LIMIT) {
+			if (regionsCount > regionsLimit) {
 				throw new JadxRuntimeException("Regions count limit reached");
 			}
 		}