lib/mpi: Remove unused scalar_copied
authorHerbert Xu <herbert@gondor.apana.org.au>
Thu, 8 Oct 2020 05:58:18 +0000 (16:58 +1100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 30 Oct 2020 06:34:45 +0000 (17:34 +1100)
The scalar_copied variable is not as the scalar is never copied
in that block.  This patch removes it.

Fixes: d58bb7e55a8a ("lib/mpi: Introduce ec implementation to...")
Reported-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
lib/mpi/ec.c

index c214701..40f5908 100644 (file)
@@ -1252,7 +1252,6 @@ void mpi_ec_mul_point(MPI_POINT result,
                MPI_POINT q1, q2, prd, sum;
                unsigned long sw;
                mpi_size_t rsize;
-               int scalar_copied = 0;
 
                /* Compute scalar point multiplication with Montgomery Ladder.
                 * Note that we don't use Y-coordinate in the points at all.
@@ -1314,8 +1313,6 @@ void mpi_ec_mul_point(MPI_POINT result,
                point_free(&p2);
                point_free(&p1_);
                point_free(&p2_);
-               if (scalar_copied)
-                       mpi_free(scalar);
                return;
        }