commit 22be635d13929acba60ada6a018059105b6c74ca
author Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Thu Mar 09 13:02:35 2017 +0100
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Wed Aug 09 11:44:53 2017 +0200
tree 1b35290e90c415d99d2996d17b9c0e5a17d09a94
parent ec5606ad0c8907527b95048e5faad270256d2d9f

Re-order some more code

library/ecp.c

diff --git a/library/ecp.c b/library/ecp.c
index c0b6a1b..cea835d 100644
--- a/library/ecp.c
+++ b/library/ecp.c
@@ -1437,6 +1437,14 @@
     m_is_odd = ( mbedtls_mpi_get_bit( m, 0 ) == 1 );
     MBEDTLS_MPI_CHK( ecp_make_scalar_odd( grp, &M, m, m_is_odd ) );
 
+    /* Is P the base point ? */
+#if MBEDTLS_ECP_FIXED_POINT_OPTIM == 1
+    p_eq_g = ( mbedtls_mpi_cmp_mpi( &P->Y, &grp->G.Y ) == 0 &&
+               mbedtls_mpi_cmp_mpi( &P->X, &grp->G.X ) == 0 );
+#else
+    p_eq_g = 0;
+#endif
+
     /*
      * Minimize the number of multiplications, that is minimize
      * 10 * d * w + 18 * 2^(w-1) + 11 * d + 7 * w, with d = ceil( nbits / w )
@@ -1449,14 +1457,8 @@
      * Just adding one avoids upping the cost of the first mul too much,
      * and the memory cost too.
      */
-#if MBEDTLS_ECP_FIXED_POINT_OPTIM == 1
-    p_eq_g = ( mbedtls_mpi_cmp_mpi( &P->Y, &grp->G.Y ) == 0 &&
-               mbedtls_mpi_cmp_mpi( &P->X, &grp->G.X ) == 0 );
     if( p_eq_g )
         w++;
-#else
-    p_eq_g = 0;
-#endif
 
     /*
      * Make sure w is within bounds.