Lukas Wunner c82bb825f9 crypto: ecdsa - Avoid signed integer overflow on signature decoding
[ Upstream commit 3b0565c703503f832d6cd7ba805aafa3b330cb9d ]

When extracting a signature component r or s from an ASN.1-encoded
integer, ecdsa_get_signature_rs() subtracts the expected length
"bufsize" from the ASN.1 length "vlen" (both of unsigned type size_t)
and stores the result in "diff" (of signed type ssize_t).

This results in a signed integer overflow if vlen > SSIZE_MAX + bufsize.

The kernel is compiled with -fno-strict-overflow, which implies -fwrapv,
meaning signed integer overflow is not undefined behavior.  And the
function does check for overflow:

       if (-diff >= bufsize)
               return -EINVAL;

So the code is fine in principle but not very obvious.  In the future it
might trigger a false-positive with CONFIG_UBSAN_SIGNED_WRAP=y.

Avoid by comparing the two unsigned variables directly and erroring out
if "vlen" is too large.

Signed-off-by: Lukas Wunner <lukas@wunner.de>
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2024-12-14 20:03:36 +01:00
..
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2023-12-08 11:59:46 +08:00
2024-02-02 18:08:12 +08:00
2024-04-02 10:49:38 +08:00
2024-08-17 13:55:50 +08:00
2023-11-17 19:16:29 +08:00
2024-04-02 10:49:38 +08:00
2024-08-24 21:36:07 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2024-06-07 19:46:39 +08:00
2020-02-28 08:43:21 +08:00
2023-02-14 13:39:33 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2022-12-02 18:12:40 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2024-04-02 10:49:38 +08:00
2022-12-02 18:12:40 +08:00
2024-06-07 19:46:39 +08:00
2022-06-10 16:40:16 +08:00
2024-08-02 20:53:25 +08:00
2021-11-08 16:55:21 -08:00