status-im
/
nimbus-eth2
mirror of https://github.com/status-im/nimbus-eth2.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

add errIgnore helper and refactor errReject helper (#2808)

This commit is contained in:
tersec 2021-08-23 10:39:06 +00:00 committed by GitHub
parent d9f7ba7153
commit 0a61d1112e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
beacon_chain/gossip_processing/gossip_validation.nim
View File

@ -162,7 +162,7 @@ template errReject(msg: cstring): untyped =
# an internal consistency/correctness check only, and effectively never has
# false positives. These don't, for example, arise from timeouts.
raiseAssert $msg
err((ValidationResult.Reject, msg))
err((ValidationResult.Reject, cstring msg))
template errReject(error: (ValidationResult, cstring)): untyped =
doAssert error[0] == ValidationResult.Reject
@ -173,6 +173,9 @@ template errReject(error: (ValidationResult, cstring)): untyped =
raiseAssert $error[1]
err(error)
template errIgnore(msg: cstring): untyped =
err((ValidationResult.Ignore, cstring msg))
# https://github.com/ethereum/eth2.0-specs/blob/v1.0.1/specs/phase0/p2p-interface.md#beacon_attestation_subnet_id
proc validateAttestation*(
pool: ref AttestationPool,
@ -314,7 +317,7 @@ proc validateAttestation*(
var x = (await cryptoFut)
case x
of BatchResult.Invalid:
return errReject(cstring("validateAttestation: invalid signature"))
return errReject("validateAttestation: invalid signature")
of BatchResult.Timeout:
beacon_attestations_dropped_queue_full.inc()
return err((ValidationResult.Ignore, cstring("validateAttestation: timeout checking signature")))
@ -387,8 +390,7 @@ proc validateAggregate*(
pool.nextAttestationEpoch[
aggregate_and_proof.aggregator_index].aggregate >
aggregate.data.target.epoch:
return err((ValidationResult.Ignore, cstring(
"Validator has already aggregated in epoch")))
return errIgnore("validateAggregate: Validator has already aggregated in epoch")
# [REJECT] The attestation has participants -- that is,
# len(get_attesting_indices(state, aggregate.data, aggregate.aggregation_bits)) >= 1.
@ -477,8 +479,7 @@ proc validateAggregate*(
var x = await cryptoFuts.aggregatorCheck
case x
of BatchResult.Invalid:
return errReject(cstring(
"validateAggregate: invalid aggregator signature"))
return errReject("validateAggregate: invalid aggregator signature")
of BatchResult.Timeout:
beacon_aggregates_dropped_queue_full.inc()
return err((ValidationResult.Reject, cstring("validateAggregate: timeout checking aggregator signature")))
@ -490,8 +491,7 @@ proc validateAggregate*(
var x = await cryptoFuts.aggregateCheck
case x
of BatchResult.Invalid:
return errReject(cstring(
"validateAggregate: invalid aggregate signature"))
return errReject("validateAggregate: invalid aggregate signature")
of BatchResult.Timeout:
beacon_aggregates_dropped_queue_full.inc()
return err((ValidationResult.Reject, cstring("validateAggregate: timeout checking aggregate signature")))