diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2019-03-29 14:09:55 +0100 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2019-04-08 14:36:16 +0800 |
| commit | a2e9cc3199518db7b7fa5af83d4e215ab474bbb7 (patch) | |
| tree | 41a9f4a31132df735ae3eb2f343c8ddd20ff29b0 /crypto/scompress.c | |
| parent | a6cebd68adf4092573bcd51209bce9e0631bad7a (diff) | |
| download | linux-crypto-a2e9cc3199518db7b7fa5af83d4e215ab474bbb7.tar.gz linux-crypto-a2e9cc3199518db7b7fa5af83d4e215ab474bbb7.zip | |
crypto: scompress - return proper error code for allocation failure
If scomp_acomp_comp_decomp() fails to allocate memory for the
destination then we never copy back the data we compressed.
It is probably best to return an error code instead 0 in case of
failure.
I haven't found any user that is using acomp_request_set_params()
without the `dst' buffer so there is probably no harm.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/scompress.c')
| -rw-r--r-- | crypto/scompress.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/crypto/scompress.c b/crypto/scompress.c index 6f8305f8..aea1a8e5 100644 --- a/crypto/scompress.c +++ b/crypto/scompress.c @@ -171,8 +171,10 @@ static int scomp_acomp_comp_decomp(struct acomp_req *req, int dir) if (!ret) { if (!req->dst) { req->dst = sgl_alloc(req->dlen, GFP_ATOMIC, NULL); - if (!req->dst) + if (!req->dst) { + ret = -ENOMEM; goto out; + } } scatterwalk_map_and_copy(scratch_dst, req->dst, 0, req->dlen, 1); |