Skip to content

Commit 377cbe5

Browse files
LiedtkeV8-internal LUCI CQ
authored andcommitted
Set a lower limit for max variables
Bug: 433646300 Change-Id: Icbcf88d431a255a3f6df6ca113e6c784e5b96e2c Reviewed-on: https://chrome-internal-review.googlesource.com/c/v8/fuzzilli/+/8446876 Reviewed-by: Carl Smith <[email protected]> Reviewed-by: Michael Achenbach <[email protected]> Commit-Queue: Matthias Liedtke <[email protected]>
1 parent ceaed44 commit 377cbe5

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

Sources/Fuzzilli/Base/ProgramBuilder.swift

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4000,7 +4000,13 @@ public class ProgramBuilder {
40004000

40014001
/// Returns the next free variable.
40024002
func nextVariable() -> Variable {
4003-
assert(numVariables < Code.maxNumberOfVariables, "Too many variables")
4003+
// Temporarily introduce a lower limit: Programs shouldn't get exceedingly large.
4004+
// TODO(mliedtke): Undo this once the issues are fixed.
4005+
let maxNumberOfVariables = 10_000
4006+
assert(maxNumberOfVariables <= Code.maxNumberOfVariables)
4007+
if numVariables >= maxNumberOfVariables {
4008+
fatalError("Too many variables \(numVariables). Contributors: \(contributors.map {$0.name})")
4009+
}
40044010
numVariables += 1
40054011
return Variable(number: numVariables - 1)
40064012
}

0 commit comments

Comments
 (0)