diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2020-07-02 13:32:21 +1000 |
|---|---|---|
| committer | Sasha Levin <sashal@kernel.org> | 2024-03-26 18:22:43 -0400 |
| commit | 546e3961b5d4c6db82cfb441fabb4353940c8f95 (patch) | |
| tree | d1a3f44fc3dbed2ac967b0176afa1b939b75aae6 | |
| parent | 2342b05ec5342a519e00524a507f7a6ea6791a38 (diff) | |
| download | linux-stable-546e3961b5d4c6db82cfb441fabb4353940c8f95.tar.gz linux-stable-546e3961b5d4c6db82cfb441fabb4353940c8f95.tar.bz2 linux-stable-546e3961b5d4c6db82cfb441fabb4353940c8f95.zip | |
crypto: af_alg - Fix regression on empty requests
commit 662bb52f50bca16a74fe92b487a14d7dccb85e1a upstream.
Some user-space programs rely on crypto requests that have no
control metadata. This broke when a check was added to require
the presence of control metadata with the ctx->init flag.
This patch fixes the regression by setting ctx->init as long as
one sendmsg(2) has been made, with or without a control message.
Reported-by: Sachin Sant <sachinp@linux.vnet.ibm.com>
Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Fixes: f3c802a1f300 ("crypto: algif_aead - Only wake up when...")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | crypto/af_alg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/af_alg.c b/crypto/af_alg.c index 78f041754f04..07835c3309c5 100644 --- a/crypto/af_alg.c +++ b/crypto/af_alg.c @@ -864,6 +864,7 @@ int af_alg_sendmsg(struct socket *sock, struct msghdr *msg, size_t size, err = -EINVAL; goto unlock; } + ctx->init = true; if (init) { ctx->enc = enc; @@ -871,7 +872,6 @@ int af_alg_sendmsg(struct socket *sock, struct msghdr *msg, size_t size, memcpy(ctx->iv, con.iv->iv, ivsize); ctx->aead_assoclen = con.aead_assoclen; - ctx->init = true; } while (size) { |