[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aUT44MU1ehlRsu3m@gondor.apana.org.au>
Date: Fri, 19 Dec 2025 15:04:00 +0800
From: Herbert Xu <herbert@...dor.apana.org.au>
To: Thorsten Blum <thorsten.blum@...ux.dev>
Cc: Kristen Accardi <kristen.c.accardi@...el.com>,
Vinicius Costa Gomes <vinicius.gomes@...el.com>,
Kanchana P Sridhar <kanchana.p.sridhar@...el.com>,
"David S. Miller" <davem@...emloft.net>,
Tom Zanussi <tom.zanussi@...ux.intel.com>, stable@...r.kernel.org,
linux-crypto@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] crypto: iaa - Fix out-of-bounds index in
find_empty_iaa_compression_mode
On Thu, Nov 27, 2025 at 03:01:57PM +0100, Thorsten Blum wrote:
> The local variable 'i' is initialized with -EINVAL, but the for loop
> immediately overwrites it and -EINVAL is never returned.
>
> If no empty compression mode can be found, the function would return the
> out-of-bounds index IAA_COMP_MODES_MAX, which would cause an invalid
> array access in add_iaa_compression_mode().
>
> Fix both issues by returning either a valid index or -EINVAL.
>
> Cc: stable@...r.kernel.org
> Fixes: b190447e0fa3 ("crypto: iaa - Add compression mode management along with fixed mode")
> Signed-off-by: Thorsten Blum <thorsten.blum@...ux.dev>
> ---
> drivers/crypto/intel/iaa/iaa_crypto_main.c | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
Patch applied. Thanks.
--
Email: Herbert Xu <herbert@...dor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Powered by blists - more mailing lists