Remove skipValidation for block, block_header handlers.

As differences are being identified due to it skipping block parent root
validation.
This commit is contained in:
Nathaniel Jensen 2019-12-30 13:50:59 +11:00 committed by zah
parent e795234504
commit e356ae0405
1 changed files with 3 additions and 2 deletions

View File

@ -109,7 +109,7 @@ proc nfuzz_block(input: openArray[byte], output: ptr byte,
raise newException(FuzzCrashError, "SSZ deserialisation failed, likely bug in preprocessing.", e) raise newException(FuzzCrashError, "SSZ deserialisation failed, likely bug in preprocessing.", e)
try: try:
result = state_transition(data.state, data.beaconBlock, {skipValidation}) result = state_transition(data.state, data.beaconBlock, {})
except IOError as e: except IOError as e:
# TODO why an IOError? # TODO why an IOError?
raise newException(FuzzCrashError, "Unexpected IOError in state transition", e) raise newException(FuzzCrashError, "Unexpected IOError in state transition", e)
@ -140,7 +140,8 @@ proc nfuzz_block_header(input: openArray[byte], output: ptr byte,
raise newException(FuzzCrashError, "SSZ deserialisation failed, likely bug in preprocessing.", e) raise newException(FuzzCrashError, "SSZ deserialisation failed, likely bug in preprocessing.", e)
try: try:
result = process_block_header(data.state, data.beaconBlock, {skipValidation}, cache) # TODO disable bls
result = process_block_header(data.state, data.beaconBlock, {}, cache)
except IOError as e: except IOError as e:
# TODO why an IOError? - is this expected/should we return false? # TODO why an IOError? - is this expected/should we return false?
raise newException(FuzzCrashError, "Unexpected IOError in block header processing", e) raise newException(FuzzCrashError, "Unexpected IOError in block header processing", e)