crypto: ahash - Fix infinite recursion in ahash_def_finup

Invoke the final function directly in the default finup implementation
since crypto_ahash_final is now just a wrapper around finup.

Reported-by: Eric Biggers <ebiggers@kernel.org>
Fixes: 9d7a0ab1c753 ("crypto: ahash - Handle partial blocks in API")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/crypto/ahash.c b/crypto/ahash.c
index e10bc2659ae4f559504dbf7cfb7ab258b0bb99d9..bc84a07c924c3bea4999f199f628668146c35ddf 100644 (file)
--- a/crypto/ahash.c
+++ b/crypto/ahash.c
@@ -600,12 +600,14 @@ static void ahash_def_finup_done2(void *data, int err)
 
 static int ahash_def_finup_finish1(struct ahash_request *req, int err)
 {
+       struct crypto_ahash *tfm = crypto_ahash_reqtfm(req);
+
        if (err)
                goto out;
 
        req->base.complete = ahash_def_finup_done2;
 
-       err = crypto_ahash_final(req);
+       err = crypto_ahash_alg(tfm)->final(req);
        if (err == -EINPROGRESS || err == -EBUSY)
                return err;