commit 3409b02b27b618edf3c171947ab3081c03a6e646
author Steven Cooreman <steven.cooreman@silabs.com> Thu May 06 18:08:30 2021 +0200
committer Steven Cooreman <steven.cooreman@silabs.com> Fri May 07 23:33:50 2021 +0200
tree e90befde9289e063622f6f324051c9c71f49e5d8
parent 8f37004bd71f674d47c8d79594bfe0101af5ffee

Remove superfluous check

As psa_mac_sign_finish / psa_mac_verify_finish already checks that the
operation structure is valid (id is non-zero), the driver itself doesn't
have to check for that anymore. If the operation has a driver ID assigned,
it means that driver has returned success from its setup function, so the
algorithm value will be set correctly.

Signed-off-by: Steven Cooreman <steven.cooreman@silabs.com>

library/psa_crypto_mac.c

diff --git a/library/psa_crypto_mac.c b/library/psa_crypto_mac.c
index dc07d45..3d7f70b 100644
--- a/library/psa_crypto_mac.c
+++ b/library/psa_crypto_mac.c
@@ -452,8 +452,6 @@
                                          uint8_t *mac,
                                          size_t mac_size )
 {
-    if( operation->alg == 0 )
-        return( PSA_ERROR_BAD_STATE );
     if( mac_size < operation->mac_size )
         return( PSA_ERROR_BUFFER_TOO_SMALL );