Missing else in cipher_hw_aes_ocb_generic_initkey
authorFangming.Fang <fangming.fang@arm.com>
Thu, 26 Sep 2019 02:14:54 +0000 (12:14 +1000)
committerShane Lontis <shane.lontis@oracle.com>
Thu, 26 Sep 2019 02:14:54 +0000 (12:14 +1000)
This came from commit 3837c202 "Add aes_ocb cipher to providers". It
causes the default non-hardware accelerated AES implementation to be
used even if HWAES_CAPABLE is set. Affects all platforms except X86 and
SPARC.

Patch by: Nick Gasson <Nick.Gasson@arm.com>
Change-Id: I26001a3a922ff23f6090fdcefefaecf68e92e2a6

Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/10012)

providers/default/ciphers/cipher_aes_ocb_hw.c

index d7ef3bd28248d29f7c34a3500509172aebfe172d..49f387b5ba695680824a72ca0650c000cdfc50cb 100644 (file)
@@ -37,7 +37,7 @@ static int cipher_hw_aes_ocb_generic_initkey(PROV_CIPHER_CTX *vctx,
         OCB_SET_KEY_FN(HWAES_set_encrypt_key, HWAES_set_decrypt_key,
                        HWAES_encrypt, HWAES_decrypt,
                        HWAES_ocb_encrypt, HWAES_ocb_decrypt);
-    }
+    } else
 # endif
 # ifdef VPAES_CAPABLE
     if (VPAES_CAPABLE) {