test/evp_test.c: don't misuse pkey_test_ctrl() in mac_test_run()
authorRichard Levitte <levitte@openssl.org>
Fri, 26 Oct 2018 11:42:40 +0000 (13:42 +0200)
committerRichard Levitte <levitte@openssl.org>
Mon, 29 Oct 2018 16:32:27 +0000 (17:32 +0100)
commitd308458ef138dfbe925203b5cb01a015f0aa93ac
tree5580fb029e1e6edfef95b5338b34389473846ff9
parent86743ef857ae3323e0d5afe73282d79b7245586f
test/evp_test.c: don't misuse pkey_test_ctrl() in mac_test_run()

pkey_test_ctrl() was designed for parsing values, not for using in
test runs.  Relying on its returned value when it returned 1 even for
control errors made it particularly useless for mac_test_run().

Here, it gets replaced with a MAC specific control function, that
parses values the same way but is designed for use in a _run() rather
than a _parse() function.

This uncovers a SipHash test with an invalid control that wasn't
caught properly.  After all, that stanza is supposed to test that
invalid control values do generate an error.  Now we catch that.

Reviewed-by: Tim Hudson <tjh@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/7500)

(cherry picked from commit ce5d64c79c4d809ece8fe28a5b62915467a1c0e1)
test/evp_test.c
test/recipes/30-test_evp_data/evpmac.txt