From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B3DDC32549E; Mon, 27 Oct 2025 18:54:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761591244; cv=none; b=kN8yVFGw532vBcm0JXr0SHNy2/eE75SQOmYC64vxoBlRiqAmdVlhxxSfEp+jfc1OobB3FXNAfWY05Sz1e9eWCzBk36hYCwuBKWeBsMXCHjUph+FeONfXaX+UpqKo+Q02YPIFrIvVSVOlgt2423MlDOdnoY08aw4DYjHwS0c0Lrg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761591244; c=relaxed/simple; bh=3HmRWmSLrbMgePqkCHEOWULAchlEOw7jFDeVPVGJH1c=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=SScWeuFDrIkJ6wM7JRF88jlEoC80DSrjgMbAR3y9mCGUjtDzG6kBpLkh3VfKlX6qGtn+2ofLtEHPww7ClGe5wmSlsYNnOCNGU2cW07zEXoq7TFQac6NSC3IDPiWXz9Tcb1qPaV6JsR+qej2zcFOsLES03PgfkD4+2KCUHIWEwiw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=YyUz4jPp; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="YyUz4jPp" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 455EAC4CEF1; Mon, 27 Oct 2025 18:54:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1761591244; bh=3HmRWmSLrbMgePqkCHEOWULAchlEOw7jFDeVPVGJH1c=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=YyUz4jPpG5LWXytzdxL59lujp782od/A7VP1SiVZnM5B8H/gjq7q5cuc04otfNqQT A0jfbT7sZKn3Y93a5LcuA0nMLBJmNcxNP7228XPsW3qvLQpjetdbGd2v7amHIttGnj JI7hSnxfeeEr8VgKDpamJ66ijjYAU14ASrA6wvXI= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Muhammad Alifa Ramdhan , Herbert Xu , Sasha Levin Subject: [PATCH 5.10 123/332] crypto: essiv - Check ssize for decryption and in-place encryption Date: Mon, 27 Oct 2025 19:32:56 +0100 Message-ID: <20251027183527.858386804@linuxfoundation.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251027183524.611456697@linuxfoundation.org> References: <20251027183524.611456697@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 5.10-stable review patch. If anyone has any objections, please let me know. ------------------ From: Herbert Xu [ Upstream commit 6bb73db6948c2de23e407fe1b7ef94bf02b7529f ] Move the ssize check to the start in essiv_aead_crypt so that it's also checked for decryption and in-place encryption. Reported-by: Muhammad Alifa Ramdhan Fixes: be1eb7f78aa8 ("crypto: essiv - create wrapper template for ESSIV generation") Signed-off-by: Herbert Xu Signed-off-by: Sasha Levin --- crypto/essiv.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/crypto/essiv.c b/crypto/essiv.c index 85bb624e32b9b..7db206fc97f21 100644 --- a/crypto/essiv.c +++ b/crypto/essiv.c @@ -185,9 +185,14 @@ static int essiv_aead_crypt(struct aead_request *req, bool enc) const struct essiv_tfm_ctx *tctx = crypto_aead_ctx(tfm); struct essiv_aead_request_ctx *rctx = aead_request_ctx(req); struct aead_request *subreq = &rctx->aead_req; + int ivsize = crypto_aead_ivsize(tfm); + int ssize = req->assoclen - ivsize; struct scatterlist *src = req->src; int err; + if (ssize < 0) + return -EINVAL; + crypto_cipher_encrypt_one(tctx->essiv_cipher, req->iv, req->iv); /* @@ -197,19 +202,12 @@ static int essiv_aead_crypt(struct aead_request *req, bool enc) */ rctx->assoc = NULL; if (req->src == req->dst || !enc) { - scatterwalk_map_and_copy(req->iv, req->dst, - req->assoclen - crypto_aead_ivsize(tfm), - crypto_aead_ivsize(tfm), 1); + scatterwalk_map_and_copy(req->iv, req->dst, ssize, ivsize, 1); } else { u8 *iv = (u8 *)aead_request_ctx(req) + tctx->ivoffset; - int ivsize = crypto_aead_ivsize(tfm); - int ssize = req->assoclen - ivsize; struct scatterlist *sg; int nents; - if (ssize < 0) - return -EINVAL; - nents = sg_nents_for_len(req->src, ssize); if (nents < 0) return -EINVAL; -- 2.51.0