Remove a redundant function call
Remove a call to `mbedtls_mpi_bitlen()` since the returned value is
overwritten in the line after. This is redundant since da31fa137a1183d3feed5981af6d05c550a8c005.
Fixes #2377.
diff --git a/ChangeLog b/ChangeLog
index 798e84a..e0a824e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -38,6 +38,9 @@
for the parameter.
* Add a check for MBEDTLS_X509_CRL_PARSE_C in ssl_server2, guarding the crl
sni entry parameter. Reported by inestlerode in #560.
+ * Remove redundant line for getting the bitlen of a bignum, since the variable
+ holding the returned value is overwritten a line after.
+ Found by irwir in #2377.
Changes
* Return from various debugging routines immediately if the
diff --git a/library/bignum.c b/library/bignum.c
index 4194618..20a1432 100644
--- a/library/bignum.c
+++ b/library/bignum.c
@@ -2333,8 +2333,6 @@
MBEDTLS_MPI_CHK( mbedtls_mpi_copy( &R, &W ) );
MBEDTLS_MPI_CHK( mbedtls_mpi_shift_r( &R, s ) );
- i = mbedtls_mpi_bitlen( X );
-
for( i = 0; i < rounds; i++ )
{
/*