diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2023-01-13 18:27:51 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2023-01-20 18:29:31 +0800 |
| commit | 32e62025e5e52fbe4812ef044759de7010b15dbc (patch) | |
| tree | 9cebafbbd2bab3badda0a070684d32609f305b75 /crypto | |
| parent | b5a772adf45a32c68bef28e60621f12617161556 (diff) | |
| download | linux-32e62025e5e52fbe4812ef044759de7010b15dbc.tar.gz linux-32e62025e5e52fbe4812ef044759de7010b15dbc.tar.bz2 linux-32e62025e5e52fbe4812ef044759de7010b15dbc.zip | |
crypto: seqiv - Handle EBUSY correctly
As it is seqiv only handles the special return value of EINPROGERSS,
which means that in all other cases it will free data related to the
request.
However, as the caller of seqiv may specify MAY_BACKLOG, we also need
to expect EBUSY and treat it in the same way. Otherwise backlogged
requests will trigger a use-after-free.
Fixes: 0a270321dbf9 ("[CRYPTO] seqiv: Add Sequence Number IV Generator")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/seqiv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/seqiv.c b/crypto/seqiv.c index 0899d527c284..b1bcfe537daf 100644 --- a/crypto/seqiv.c +++ b/crypto/seqiv.c @@ -23,7 +23,7 @@ static void seqiv_aead_encrypt_complete2(struct aead_request *req, int err) struct aead_request *subreq = aead_request_ctx(req); struct crypto_aead *geniv; - if (err == -EINPROGRESS) + if (err == -EINPROGRESS || err == -EBUSY) return; if (err) |