diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2025-11-21 13:54:20 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2025-11-24 17:53:02 +0800 |
| commit | ebbdf6466b30e3b37f3b360826efd21f0633fb9e (patch) | |
| tree | 9e4fe8d685b57a61b463770c79b439c3b3fd4fdc | |
| parent | b0356b75f42fde15d4be268c5891f2cee6eb65bf (diff) | |
crypto: ahash - Zero positive err value in ahash_update_finish
The partial block length returned by a block-only driver should
not be passed up to the caller since ahash itself deals with the
partial block data.
Set err to zero in ahash_update_finish if it was positive.
Reported-by: T Pratham <t-pratham@ti.com>
Tested-by: T Pratham <t-pratham@ti.com>
Fixes: 9d7a0ab1c753 ("crypto: ahash - Handle partial blocks in API")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
| -rw-r--r-- | crypto/ahash.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/crypto/ahash.c b/crypto/ahash.c index 819b484a1a00..66492ae75fcf 100644 --- a/crypto/ahash.c +++ b/crypto/ahash.c @@ -423,7 +423,11 @@ static int ahash_update_finish(struct ahash_request *req, int err) req->nbytes += nonzero - blen; - blen = err < 0 ? 0 : err + nonzero; + blen = 0; + if (err >= 0) { + blen = err + nonzero; + err = 0; + } if (ahash_request_isvirt(req)) memcpy(buf, req->svirt + req->nbytes - blen, blen); else |