Skip to content

Commit

Permalink
Don't ignore ECDSA tests without GF2m support
Browse files Browse the repository at this point in the history
This looks like a typo. We can't do ECDSA without EC support, but ECDSA
for the prime curve P-256 works just fine without GF2m support.
  • Loading branch information
botovq committed Oct 20, 2023
1 parent 753811d commit ac2640d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions openssl/src/ecdsa.rs
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ mod test {
}

#[test]
#[cfg_attr(osslconf = "OPENSSL_NO_EC2M", ignore)]
#[cfg_attr(osslconf = "OPENSSL_NO_EC", ignore)]
fn sign_and_verify() {
let group = EcGroup::from_curve_name(Nid::X9_62_PRIME256V1).unwrap();
let private_key = EcKey::generate(&group).unwrap();
Expand Down Expand Up @@ -186,7 +186,7 @@ mod test {
}

#[test]
#[cfg_attr(osslconf = "OPENSSL_NO_EC2M", ignore)]
#[cfg_attr(osslconf = "OPENSSL_NO_EC", ignore)]
fn check_private_components() {
let group = EcGroup::from_curve_name(Nid::X9_62_PRIME256V1).unwrap();
let private_key = EcKey::generate(&group).unwrap();
Expand All @@ -206,7 +206,7 @@ mod test {
}

#[test]
#[cfg_attr(osslconf = "OPENSSL_NO_EC2M", ignore)]
#[cfg_attr(osslconf = "OPENSSL_NO_EC", ignore)]
fn serialize_deserialize() {
let group = EcGroup::from_curve_name(Nid::X9_62_PRIME256V1).unwrap();
let private_key = EcKey::generate(&group).unwrap();
Expand Down

0 comments on commit ac2640d

Please sign in to comment.