diff options
| author | Tianjia Zhang <tianjia.zhang@linux.alibaba.com> | 2023-12-14 11:08:34 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-02-23 08:55:11 +0100 |
| commit | 2bb86817b33c9d704e127f92b838035a72c315b6 (patch) | |
| tree | f4be6a492387e10de37b20e1d1f60a48580c1e4e | |
| parent | 44a8a2c92e5af865ae983ab18df0b05574d39d05 (diff) | |
| download | linux-2bb86817b33c9d704e127f92b838035a72c315b6.tar.gz linux-2bb86817b33c9d704e127f92b838035a72c315b6.tar.bz2 linux-2bb86817b33c9d704e127f92b838035a72c315b6.zip | |
crypto: lib/mpi - Fix unexpected pointer access in mpi_ec_init
[ Upstream commit ba3c5574203034781ac4231acf117da917efcd2a ]
When the mpi_ec_ctx structure is initialized, some fields are not
cleared, causing a crash when referencing the field when the
structure was released. Initially, this issue was ignored because
memory for mpi_ec_ctx is allocated with the __GFP_ZERO flag.
For example, this error will be triggered when calculating the
Za value for SM2 separately.
Fixes: d58bb7e55a8a ("lib/mpi: Introduce ec implementation to MPI library")
Cc: stable@vger.kernel.org # v6.5
Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | lib/mpi/ec.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/mpi/ec.c b/lib/mpi/ec.c index 40f5908e57a4..e16dca1e23d5 100644 --- a/lib/mpi/ec.c +++ b/lib/mpi/ec.c @@ -584,6 +584,9 @@ void mpi_ec_init(struct mpi_ec_ctx *ctx, enum gcry_mpi_ec_models model, ctx->a = mpi_copy(a); ctx->b = mpi_copy(b); + ctx->d = NULL; + ctx->t.two_inv_p = NULL; + ctx->t.p_barrett = use_barrett > 0 ? mpi_barrett_init(ctx->p, 0) : NULL; mpi_ec_get_reset(ctx); |