diff options
| author |   Herbert Xu <herbert@gondor.apana.org.au> | 2022-09-02 18:19:27 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2022-09-02 18:19:27 +0800 |
| commit | 2ad548ebb85cc416587ba68c0e530a2f00b2273a (patch) | |
| tree | e32e8a004e5f8098fdbb687f171c7b2c3b66b446 /drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c | |
| parent | Revert "crypto: gemini - Fix error check for dma_map_sg" (diff) | |
| download | linux-dev-2ad548ebb85cc416587ba68c0e530a2f00b2273a.tar.xz linux-dev-2ad548ebb85cc416587ba68c0e530a2f00b2273a.zip | |
Revert "crypto: allwinner - Fix dma_map_sg error check"
This reverts commit 2b02187bdb0bb75000850bd0309e70eb8664159e.
The original code was correct and arguably more robust than the
patched version.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to '')
| -rw-r--r-- | drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c b/drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c index fcb8c41cc957..36a82b22953c 100644 --- a/drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c +++ b/drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c @@ -527,7 +527,7 @@ int sun8i_ss_hash_run(struct crypto_engine *engine, void *breq) rctx->method = ss->variant->alg_hash[algt->ss_algo_id]; nr_sgs = dma_map_sg(ss->dev, areq->src, sg_nents(areq->src), DMA_TO_DEVICE); - if (!nr_sgs || nr_sgs > MAX_SG) { + if (nr_sgs <= 0 || nr_sgs > MAX_SG) { dev_err(ss->dev, "Invalid sg number %d\n", nr_sgs); err = -EINVAL; goto theend; |