Skip to content

Commit

Permalink
Merge pull request #485 from plotchy/patch-1
Browse files Browse the repository at this point in the history
fix: jumpi coverage condition matches evm
  • Loading branch information
publicqi authored May 22, 2024
2 parents 9840724 + 57669c2 commit 2ffa214
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/evm/middlewares/coverage.rs
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,7 @@ where
self.pc_coverage.entry(address).or_default().insert(pc);

if *interp.instruction_pointer == JUMPI {
let condition = is_zero(interp.stack.peek(1).unwrap());
let condition = if is_zero(interp.stack.peek(1).unwrap()) { false } else { true };
self.jumpi_coverage.entry(address).or_default().insert((pc, condition));
}
}
Expand Down

0 comments on commit 2ffa214

Please sign in to comment.