]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
crypto: shash - fold shash_digest_unaligned() into crypto_shash_digest()
authorEric Biggers <ebiggers@google.com>
Mon, 9 Oct 2023 07:32:14 +0000 (00:32 -0700)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 20 Oct 2023 05:39:25 +0000 (13:39 +0800)
Fold shash_digest_unaligned() into its only remaining caller.  Also,
avoid a redundant check of CRYPTO_TFM_NEED_KEY by replacing the call to
crypto_shash_init() with shash->init(desc).  Finally, replace
shash_update_unaligned() + shash_final_unaligned() with
shash_finup_unaligned() which does exactly that.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
crypto/shash.c

index d99dc2f94c65f52afd770ced9751f6750d872c03..15fee57cca8efd8b227de8f467c89d8db32ab9f5 100644 (file)
@@ -225,14 +225,6 @@ int crypto_shash_finup(struct shash_desc *desc, const u8 *data,
 }
 EXPORT_SYMBOL_GPL(crypto_shash_finup);
 
-static int shash_digest_unaligned(struct shash_desc *desc, const u8 *data,
-                                 unsigned int len, u8 *out)
-{
-       return crypto_shash_init(desc) ?:
-              shash_update_unaligned(desc, data, len) ?:
-              shash_final_unaligned(desc, out);
-}
-
 static int shash_default_digest(struct shash_desc *desc, const u8 *data,
                                unsigned int len, u8 *out)
 {
@@ -260,7 +252,8 @@ int crypto_shash_digest(struct shash_desc *desc, const u8 *data,
        if (crypto_shash_get_flags(tfm) & CRYPTO_TFM_NEED_KEY)
                err = -ENOKEY;
        else if (((unsigned long)data | (unsigned long)out) & alignmask)
-               err = shash_digest_unaligned(desc, data, len, out);
+               err = shash->init(desc) ?:
+                     shash_finup_unaligned(desc, data, len, out);
        else
                err = shash->digest(desc, data, len, out);