Skip to content

Commit

Permalink
fixup! APPS/pkeyutl: add missing high-level check for -verifyrecover …
Browse files Browse the repository at this point in the history
…not being compatible with EdDSA
  • Loading branch information
DDvO committed Nov 8, 2024
1 parent e9a55ae commit fb1796d
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions test/recipes/20-test_pkeyutl.t
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use File::Compare qw/compare_text compare/;

setup("test_pkeyutl");

plan tests => 23;
plan tests => 24;

# For the tests below we use the cert itself as the TBS file

Expand Down Expand Up @@ -54,7 +54,7 @@ SKIP: {
}

SKIP: {
skip "Skipping tests that require ECX", 6
skip "Skipping tests that require ECX", 7
if disabled("ecx");

# Ed25519
Expand All @@ -68,6 +68,9 @@ SKIP: {
'-inkey', srctop_file('test', 'certs', 'server-ed25519-cert.pem'),
'-sigfile', 'Ed25519.sig']))),
"Verify an Ed25519 signature against a piece of data");
ok(!run(app(([ 'openssl', 'pkeyutl', '-verifyrecover', '-in', 'Ed25519.sig',
'-inkey', srctop_file('test', 'certs', 'server-ed25519-key.pem')]))),
"Cannot use -verifyrecover with EdDSA");

# Ed448
ok(run(app(([ 'openssl', 'pkeyutl', '-sign', '-in',
Expand Down

0 comments on commit fb1796d

Please sign in to comment.