Skip to content

Commit c4eefc3

Browse files
authored
fix OOB panic (crytic#303)
1 parent ac0b42f commit c4eefc3

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

fuzzing/coverage/coverage_maps.go

+2-3
Original file line numberDiff line numberDiff line change
@@ -344,10 +344,9 @@ func (cm *CoverageMapBytecodeData) update(coverageMap *CoverageMapBytecodeData)
344344
return true, nil
345345
}
346346

347-
// Update each byte which represents a position in the bytecode which was covered. We ignore any size
348-
// differences as init bytecode can have arbitrary length arguments appended.
347+
// Update each byte which represents a position in the bytecode which was covered.
349348
changed := false
350-
for i := 0; i < len(cm.executedFlags) || i < len(coverageMap.executedFlags); i++ {
349+
for i := 0; i < len(cm.executedFlags) && i < len(coverageMap.executedFlags); i++ {
351350
if cm.executedFlags[i] == 0 && coverageMap.executedFlags[i] != 0 {
352351
cm.executedFlags[i] = 1
353352
changed = true

0 commit comments

Comments
 (0)