Skip to content

Commit debc47a

Browse files
LiedtkeV8-internal LUCI CQ
authored andcommitted
Fix assertion in InliningReducer for explicit resource management
This fixes #545 (by just doing the same for the other kinds of disposable variables that we already do for `loadDisposableVariable`) Change-Id: I11ddb6323124deb7f99dbf110fee214be62b33a9 Reviewed-on: https://chrome-internal-review.googlesource.com/c/v8/fuzzilli/+/8956877 Commit-Queue: Matthias Liedtke <mliedtke@google.com> Auto-Submit: Matthias Liedtke <mliedtke@google.com> Reviewed-by: Michael Achenbach <machenbach@google.com>
1 parent 7e668c9 commit debc47a

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Sources/Fuzzilli/Minimization/InliningReducer.swift

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,10 @@ struct InliningReducer: Reducer {
118118

119119
// Can't inline functions that are passed as arguments to other functions.
120120
deleteCandidates(instr.inputs.dropFirst())
121-
case .loadDisposableVariable:
121+
case .loadDisposableVariable,
122+
.createNamedDisposableVariable,
123+
.loadAsyncDisposableVariable,
124+
.createNamedAsyncDisposableVariable:
122125
// Can't inline functions that are also used as a disposable variable.
123126
deleteCandidates(instr.inputs)
124127
fallthrough

0 commit comments

Comments
 (0)