X-Git-Url: https://git.distorted.org.uk/u/mdw/catacomb/blobdiff_plain/dfdacfdcd7e3376072506d6bdf69271a0e6bd2e0..b0b682aa5f6174deb5bde83701b191556986fe05:/rabin.c

diff --git a/rabin.c b/rabin.c
index 58d5291..e6cd488 100644
--- a/rabin.c
+++ b/rabin.c
@@ -1,6 +1,6 @@
 /* -*-c-*-
  *
- * $Id: rabin.c,v 1.3 1999/12/22 15:50:29 mdw Exp $
+ * $Id: rabin.c,v 1.6 2001/06/16 12:56:38 mdw Exp $
  *
  * Miller-Rabin primality test
  *
@@ -30,6 +30,15 @@
 /*----- Revision history --------------------------------------------------* 
  *
  * $Log: rabin.c,v $
+ * Revision 1.6  2001/06/16 12:56:38  mdw
+ * Fixes for interface change to @mpmont_expr@ and @mpmont_mexpr@.
+ *
+ * Revision 1.5  2000/10/08 12:11:22  mdw
+ * Use @MP_EQ@ instead of @MP_CMP@.
+ *
+ * Revision 1.4  2000/06/22 19:03:02  mdw
+ * Use the new @mp_odd@ function.
+ *
  * Revision 1.3  1999/12/22 15:50:29  mdw
  * Reworking for new prime-search system.  Add function for working out how
  * many iterations to use for a particular number.
@@ -66,24 +75,8 @@
 void rabin_create(rabin *r, mp *m)
 {
   mp *m1 = mp_sub(MP_NEW, m, MP_ONE);
-  mpscan sc;
-  size_t s;
-
-  /* --- Find @r@ and @s@ --- */
-
   mpmont_create(&r->mm, m);
-  mp_scan(&sc, m1);
-  s = 0;
-  while (mp_step(&sc)) {
-    if (mp_bit(&sc))
-      break;
-    s++;
-  }
-  r->s = s;
-  r->r = mp_lsr(MP_NEW, m1, s);
-
-  /* --- Compute %$(m - 1)R \bmod m$% --- */
-
+  r->r = mp_odd(MP_NEW, m1, &r->s);
   r->m1 = mp_sub(MP_NEW, m, r->mm.r);
   mp_drop(m1);
 }
@@ -130,8 +123,9 @@ int rabin_test(rabin *r, mp *g)
    * @y@ here has an extra factor of %$R$%.
    */
 
-  y = mpmont_expr(&r->mm, MP_NEW, g, r->r);
-  if (MP_CMP(y, ==, r->mm.r) || MP_CMP(y, ==, r->m1)) {
+  y = mpmont_mul(&r->mm, MP_NEW, g, r->mm.r2);
+  y = mpmont_expr(&r->mm, y, y, r->r);
+  if (MP_EQ(y, r->mm.r) || MP_EQ(y, r->m1)) {
     rc = PGEN_PASS;
     goto done;
   }
@@ -146,9 +140,9 @@ int rabin_test(rabin *r, mp *g)
     dd = mp_sqr(spare, y);
     dd = mpmont_reduce(&r->mm, dd, dd);
     spare = y; y = dd;
-    if (MP_CMP(y, ==, r->mm.r))
+    if (MP_EQ(y, r->mm.r))
       break;
-    if (MP_CMP(y, ==, r->m1)) {
+    if (MP_EQ(y, r->m1)) {
       rc = PGEN_PASS;
       break;
     }