Fix CTAP2 batch attestation

This commit is contained in:
Jean-Michel Picod
2020-11-25 17:26:08 +01:00
parent 3dbfae972f
commit 026b4a66ac

View File

@@ -522,10 +522,12 @@ where
let mut signature_data = auth_data.clone();
signature_data.extend(client_data_hash);
// We currently use the presence of the attestation private key in the persistent storage to
// decide whether batch attestation is needed.
let (signature, x5c) = match self.persistent_store.attestation_private_key()? {
Some(attestation_private_key) => {
let (signature, x5c) = if USE_BATCH_ATTESTATION {
let attestation_private_key = self
.persistent_store
.attestation_private_key()?
.ok_or(Ctap2StatusCode::CTAP2_ERR_VENDOR_INTERNAL_ERROR)?;
let attestation_key =
crypto::ecdsa::SecKey::from_bytes(attestation_private_key).unwrap();
let attestation_certificate = self
@@ -536,11 +538,11 @@ where
attestation_key.sign_rfc6979::<crypto::sha256::Sha256>(&signature_data),
Some(vec![attestation_certificate]),
)
}
None => (
} else {
(
sk.sign_rfc6979::<crypto::sha256::Sha256>(&signature_data),
None,
),
)
};
let attestation_statement = PackedAttestationStatement {
alg: SignatureAlgorithm::ES256 as i64,