Merge pull request #572 from libtom/deprecate-apis

Deprecate APIs

Author: sjaeckel

bn_deprecated.c

@@ -74,6 +74,12 @@ mp_err mp_balance_mul(const mp_int *a, const mp_int *b, mp_int *c)
    return s_mp_balance_mul(a, b, c);
 }
 #endif
+#ifdef BN_MP_DIV_3_C
+mp_err mp_div_3(const mp_int *a, mp_int *c, mp_digit *d)
+{
+   return s_mp_div_3(a, c, d);
+}
+#endif
 #ifdef BN_MP_EXPTMOD_FAST_C
 mp_err mp_exptmod_fast(const mp_int *G, const mp_int *X, const mp_int *P, mp_int *Y, int redmode)
 {
@@ -184,51 +190,97 @@ unsigned long mp_get_long(const mp_int *a)
 #ifdef BN_MP_GET_LONG_LONG_C
 unsigned long long mp_get_long_long(const mp_int *a)
 {
-   return mp_get_mag_ull(a);
+   return (unsigned long long)mp_get_mag_u64(a);
 }
 #endif
+#ifdef BN_MP_GET_LL_C
+MP_GET_SIGNED(mp_get_ll, mp_get_mag_u64, long long, uint64_t)
+#endif
+#ifdef BN_MP_GET_MAG_ULL_C
+MP_GET_MAG(mp_get_mag_ull, unsigned long long)
+#endif
+#ifdef BN_MP_INIT_LL_C
+MP_INIT_INT(mp_init_ll, mp_set_i64, long long)
+#endif
+#ifdef BN_MP_SET_LL_C
+MP_SET_SIGNED(mp_set_ll, mp_set_i64, long long, unsigned long long)
+#endif
+#ifdef BN_MP_INIT_ULL_C
+MP_INIT_INT(mp_init_ull, mp_set_u64, unsigned long long)
+#endif
+#ifdef BN_MP_SET_ULL_C
+MP_SET_UNSIGNED(mp_set_ull, unsigned long long)
+#endif
 #ifdef BN_MP_PRIME_IS_DIVISIBLE_C
 mp_err mp_prime_is_divisible(const mp_int *a, mp_bool *result)
 {
    return s_mp_prime_is_divisible(a, result);
 }
 #endif
+#ifdef BN_MP_LOG_U32_C
+mp_err mp_log_u32(const mp_int *a, uint32_t base, uint32_t *c)
+{
+   mp_err e;
+   int c_;
+   if (base > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
+      return MP_VAL;
+   }
+   e = mp_log_n(a, (int)base, &c_);
+   *c = c_;
+   return e;
+}
+#endif
 #ifdef BN_MP_EXPT_D_EX_C
 mp_err mp_expt_d_ex(const mp_int *a, mp_digit b, mp_int *c, int fast)
 {
    (void)fast;
-   if (b > MP_MIN(MP_DIGIT_MAX, UINT32_MAX)) {
+   if (b > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
       return MP_VAL;
    }
-   return mp_expt_u32(a, (uint32_t)b, c);
+   return mp_expt_n(a, (int)b, c);
 }
 #endif
 #ifdef BN_MP_EXPT_D_C
 mp_err mp_expt_d(const mp_int *a, mp_digit b, mp_int *c)
 {
-   if (b > MP_MIN(MP_DIGIT_MAX, UINT32_MAX)) {
+   if (b > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
       return MP_VAL;
    }
-   return mp_expt_u32(a, (uint32_t)b, c);
+   return mp_expt_n(a, (int)b, c);
+}
+#endif
+#ifdef BN_MP_EXPT_U32_C
+mp_err mp_expt_u32(const mp_int *a, uint32_t b, mp_int *c)
+{
+   if (b > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
+      return MP_VAL;
+   }
+   return mp_expt_n(a, (int)b, c);
 }
 #endif
 #ifdef BN_MP_N_ROOT_EX_C
 mp_err mp_n_root_ex(const mp_int *a, mp_digit b, mp_int *c, int fast)
 {
    (void)fast;
-   if (b > MP_MIN(MP_DIGIT_MAX, UINT32_MAX)) {
+   if (b > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
       return MP_VAL;
    }
-   return mp_root_u32(a, (uint32_t)b, c);
+   return mp_root_n(a, (int)b, c);
 }
 #endif
 #ifdef BN_MP_N_ROOT_C
 mp_err mp_n_root(const mp_int *a, mp_digit b, mp_int *c)
 {
-   if (b > MP_MIN(MP_DIGIT_MAX, UINT32_MAX)) {
+   if (b > MP_MIN(MP_DIGIT_MAX, INT_MAX)) {
       return MP_VAL;
    }
-   return mp_root_u32(a, (uint32_t)b, c);
+   return mp_root_n(a, (int)b, c);
+}
+#endif
+#ifdef BN_MP_ROOT_U32_C
+mp_err mp_root_u32(const mp_int *a, uint32_t b, mp_int *c)
+{
+   return mp_root_n(a, (int)b, c);
 }
 #endif
 #ifdef BN_MP_UNSIGNED_BIN_SIZE_C

bn_mp_div_d.c

@@ -44,8 +44,8 @@ mp_err mp_div_d(const mp_int *a, mp_digit b, mp_int *c, mp_digit *d)
    }
 
    /* three? */
-   if (MP_HAS(MP_DIV_3) && (b == 3u)) {
-      return mp_div_3(a, c, d);
+   if (MP_HAS(S_MP_DIV_3) && (b == 3u)) {
+      return s_mp_div_3(a, c, d);
    }
 
    /* no easy answer [c'est la vie].  Just division */

bn_mp_expt_u32.c renamed to bn_mp_expt_n.c

@@ -1,13 +1,12 @@
 #include "tommath_private.h"
-#ifdef BN_MP_EXPT_U32_C
+#ifdef BN_MP_EXPT_N_C
 /* LibTomMath, multiple-precision integer library -- Tom St Denis */
 /* SPDX-License-Identifier: Unlicense */
 
 /* calculate c = a**b  using a square-multiply algorithm */
-mp_err mp_expt_u32(const mp_int *a, uint32_t b, mp_int *c)
+mp_err mp_expt_n(const mp_int *a, int b, mp_int *c)
 {
    mp_err err;
-
    mp_int  g;
 
    if ((err = mp_init_copy(&g, a)) != MP_OKAY) {
@@ -17,16 +16,16 @@ mp_err mp_expt_u32(const mp_int *a, uint32_t b, mp_int *c)
    /* set initial result */
    mp_set(c, 1uL);
 
-   while (b > 0u) {
+   while (b > 0) {
       /* if the bit is set multiply */
-      if ((b & 1u) != 0u) {
+      if ((b & 1) != 0) {
          if ((err = mp_mul(c, &g, c)) != MP_OKAY) {
             goto LBL_ERR;
          }
       }
 
       /* square */
-      if (b > 1u) {
+      if (b > 1) {
          if ((err = mp_sqr(&g, &g)) != MP_OKAY) {
             goto LBL_ERR;
          }
@@ -36,8 +35,6 @@ mp_err mp_expt_u32(const mp_int *a, uint32_t b, mp_int *c)
       b >>= 1;
    }
 
-   err = MP_OKAY;
-
 LBL_ERR:
    mp_clear(&g);
    return err;

bn_mp_get_ll.c

@@ -0,0 +1,29 @@
+#include "tommath_private.h"
+#ifdef BN_MP_LOG_N_C
+/* LibTomMath, multiple-precision integer library -- Tom St Denis */
+/* SPDX-License-Identifier: Unlicense */
+
+mp_err mp_log_n(const mp_int *a, int base, int *c)
+{
+   if (mp_isneg(a) || mp_iszero(a) || (base < 2) || (unsigned)base > (unsigned)MP_DIGIT_MAX) {
+      return MP_VAL;
+   }
+
+   if (MP_HAS(S_MP_LOG_2EXPT) && MP_IS_2EXPT((mp_digit)base)) {
+      *c = s_mp_log_2expt(a, (mp_digit)base);
+      return MP_OKAY;
+   }
+
+   if (MP_HAS(S_MP_LOG_D) && (a->used == 1)) {
+      *c = s_mp_log_d((mp_digit)base, a->dp[0]);
+      return MP_OKAY;
+   }
+
+   if (MP_HAS(S_MP_LOG)) {
+      return s_mp_log(a, (mp_digit)base, c);
+   }
+
+   return MP_VAL;
+}
+
+#endif