]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
Fix kbkdf bug if MAC is set to KMAC and then something else
authorPauli <ppzgs1@gmail.com>
Mon, 15 Jul 2024 03:26:50 +0000 (13:26 +1000)
committerPauli <ppzgs1@gmail.com>
Wed, 17 Jul 2024 04:14:41 +0000 (14:14 +1000)
A context that is set to KMAC sets the is_kmac flag and this cannot be reset.
So a user that does kbkdf using KMAC and then wants to use HMAC or CMAC will
experience a failure.

Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24883)

providers/implementations/kdfs/kbkdf.c

index e6855d573240410ac299c66539908cff7a4006e0..ebc0e58e5b0e457d003b3ba4e8c450d9532e862c 100644 (file)
@@ -354,7 +354,8 @@ static int kbkdf_set_ctx_params(void *vctx, const OSSL_PARAM params[])
     if (!ossl_prov_macctx_load_from_params(&ctx->ctx_init, params, NULL,
                                            NULL, NULL, libctx))
         return 0;
-    else if (ctx->ctx_init != NULL) {
+    if (ctx->ctx_init != NULL) {
+        ctx->is_kmac = 0;
         if (EVP_MAC_is_a(EVP_MAC_CTX_get0_mac(ctx->ctx_init),
                          OSSL_MAC_NAME_KMAC128)
             || EVP_MAC_is_a(EVP_MAC_CTX_get0_mac(ctx->ctx_init),