Improve readability of f_impl.c and f_impl.h

Reviewed-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
(Merged from https://github.com/openssl/openssl/pull/5105)

diff --git a/crypto/ec/curve448/arch_32/f_impl.c b/crypto/ec/curve448/arch_32/f_impl.c
index 423e8a803a43da47df72a820a5024f7889f3deb7..8a89d276edb6a4d2a435182bb838c8a00948e946 100644 (file)
--- a/crypto/ec/curve448/arch_32/f_impl.c
+++ b/crypto/ec/curve448/arch_32/f_impl.c
@@ -69,7 +69,7 @@ void gf_mulw_unsigned(gf_s * RESTRICT cs, const gf as, uint32_t b)
     uint32_t mask = (1 << 28) - 1;
     int i;
 
-    assert(b < 1 << 28);
+    assert(b <= mask);
 
     for (i = 0; i < 8; i++) {
         accum0 += widemul(b, a[i]);

diff --git a/crypto/ec/curve448/arch_32/f_impl.h b/crypto/ec/curve448/arch_32/f_impl.h
index a8b26621822e8c37772d20a721d896a7569bc3de..1543d10663621036d4ebcf01844063dabd39f630 100644 (file)
--- a/crypto/ec/curve448/arch_32/f_impl.h
+++ b/crypto/ec/curve448/arch_32/f_impl.h
@@ -42,16 +42,16 @@ void gf_bias(gf a, int amt)
     uint32_t co1 = ((1 << 28) - 1) * amt, co2 = co1 - amt;
 
     for (i = 0; i < NLIMBS; i++)
-        a->limb[i] += (i == sizeof(*a) / sizeof(a->limb[0]) / 2) ? co2 : co1;
+        a->limb[i] += (i == NLIMBS / 2) ? co2 : co1;
 }
 
 void gf_weak_reduce(gf a)
 {
     uint32_t mask = (1 << 28) - 1;
-    uint32_t tmp = a->limb[15] >> 28;
+    uint32_t tmp = a->limb[NLIMBS - 1] >> 28;
     unsigned int i;
 
-    a->limb[8] += tmp;
+    a->limb[NLIMBS / 2] += tmp;
     for (i = 15; i > 0; i--)
         a->limb[i] = (a->limb[i] & mask) + (a->limb[i - 1] >> 28);
     a->limb[0] = (a->limb[0] & mask) + tmp;