Add more unique errors in validate ssz block

This commit is contained in:
Paul Hauner 2018-09-25 23:35:10 +10:00
parent 310662ee17
commit eedfa8c8ea
No known key found for this signature in database
GPG Key ID: 303E4494BB28068C

View File

@ -41,7 +41,8 @@ pub enum SszBlockValidationError {
UnknownPoWChainRef, UnknownPoWChainRef,
BadAttestationSsz, BadAttestationSsz,
AttestationValidationError(AttestationValidationError), AttestationValidationError(AttestationValidationError),
InvalidAttestation, AttestationSignatureFailed,
FirstAttestationSignatureFailed,
NoProposerSignature, NoProposerSignature,
BadProposerMap, BadProposerMap,
DatabaseError(String), DatabaseError(String),
@ -141,7 +142,8 @@ pub fn validate_ssz_block<T>(b: &SszBlock,
* If the set of voters is None, the attestation was invalid. * If the set of voters is None, the attestation was invalid.
*/ */
let attestation_voters = attestation_voters let attestation_voters = attestation_voters
.ok_or(SszBlockValidationError::InvalidAttestation)?; .ok_or(SszBlockValidationError::
FirstAttestationSignatureFailed)?;
/* /*
* Read the proposer from the map of slot -> validator index. * Read the proposer from the map of slot -> validator index.
@ -180,7 +182,8 @@ pub fn validate_ssz_block<T>(b: &SszBlock,
validator_store.clone(), validator_store.clone(),
attester_map.clone())?; attester_map.clone())?;
if attestation_voters.is_none() { if attestation_voters.is_none() {
return Err(SszBlockValidationError::InvalidAttestation); return Err(SszBlockValidationError::
AttestationSignatureFailed);
} }
} }