Move BN_bn2mpi()/BN_mpi2bn() into bn_convert.c

3 files changed, 74 insertions, 138 deletions

diff --git a/src/lib/libcrypto/Makefile b/src/lib/libcrypto/Makefile
index 2026360212..d2cc11244e 100644
--- a/src/lib/libcrypto/Makefile
+++ b/src/lib/libcrypto/Makefile
@@ -1,4 +1,4 @@
-# $OpenBSD: Makefile,v 1.105 2023/04/17 05:54:20 tb Exp $
+# $OpenBSD: Makefile,v 1.106 2023/04/17 12:36:59 jsing Exp $
 
 LIB=    crypto
 LIBREBUILD=y
@@ -195,7 +195,6 @@ SRCS+= bn_lib.c
 SRCS+= bn_mod.c
 SRCS+= bn_mod_sqrt.c
 SRCS+= bn_mont.c
-SRCS+= bn_mpi.c
 SRCS+= bn_mul.c
 SRCS+= bn_nist.c
 SRCS+= bn_prime.c
diff --git a/src/lib/libcrypto/bn/bn_convert.c b/src/lib/libcrypto/bn/bn_convert.c
index 2d15b29893..ab6c7924f1 100644
--- a/src/lib/libcrypto/bn/bn_convert.c
+++ b/src/lib/libcrypto/bn/bn_convert.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: bn_convert.c,v 1.1 2023/04/14 11:10:11 jsing Exp $ */
+/* $OpenBSD: bn_convert.c,v 1.2 2023/04/17 12:36:59 jsing Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -351,6 +351,78 @@ BN_asc2bn(BIGNUM **bn, const char *a)
         return 1;
 }
 
+int
+BN_bn2mpi(const BIGNUM *a, unsigned char *d)
+{
+        int bits;
+        int num = 0;
+        int ext = 0;
+        long l;
+
+        bits = BN_num_bits(a);
+        num = (bits + 7) / 8;
+        if (bits > 0) {
+                ext = ((bits & 0x07) == 0);
+        }
+        if (d == NULL)
+                return (num + 4 + ext);
+
+        l = num + ext;
+        d[0] = (unsigned char)(l >> 24) & 0xff;
+        d[1] = (unsigned char)(l >> 16) & 0xff;
+        d[2] = (unsigned char)(l >> 8) & 0xff;
+        d[3] = (unsigned char)(l) & 0xff;
+        if (ext)
+                d[4] = 0;
+        num = BN_bn2bin(a, &(d[4 + ext]));
+        if (a->neg)
+                d[4] |= 0x80;
+        return (num + 4 + ext);
+}
+
+BIGNUM *
+BN_mpi2bn(const unsigned char *d, int n, BIGNUM *ain)
+{
+        BIGNUM *a = ain;
+        long len;
+        int neg = 0;
+
+        if (n < 4) {
+                BNerror(BN_R_INVALID_LENGTH);
+                return (NULL);
+        }
+        len = ((long)d[0] << 24) | ((long)d[1] << 16) | ((int)d[2] << 8) |
+            (int)d[3];
+        if ((len + 4) != n) {
+                BNerror(BN_R_ENCODING_ERROR);
+                return (NULL);
+        }
+
+        if (a == NULL)
+                a = BN_new();
+        if (a == NULL)
+                return (NULL);
+
+        if (len == 0) {
+                a->neg = 0;
+                a->top = 0;
+                return (a);
+        }
+        d += 4;
+        if ((*d) & 0x80)
+                neg = 1;
+        if (BN_bin2bn(d, (int)len, a) == NULL) {
+                if (ain == NULL)
+                        BN_free(a);
+                return (NULL);
+        }
+        BN_set_negative(a, neg);
+        if (neg) {
+                BN_clear_bit(a, BN_num_bits(a) - 1);
+        }
+        return (a);
+}
+
 #ifndef OPENSSL_NO_BIO
 int
 BN_print_fp(FILE *fp, const BIGNUM *a)
diff --git a/src/lib/libcrypto/bn/bn_mpi.c b/src/lib/libcrypto/bn/bn_mpi.c
deleted file mode 100644
index e3b9ba0dd9..0000000000
--- a/src/lib/libcrypto/bn/bn_mpi.c
+++ /dev/null
@@ -1,135 +0,0 @@
-/* $OpenBSD: bn_mpi.c,v 1.12 2023/02/13 04:25:37 jsing Exp $ */
-/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
- * All rights reserved.
- *
- * This package is an SSL implementation written
- * by Eric Young (eay@cryptsoft.com).
- * The implementation was written so as to conform with Netscapes SSL.
- *
- * This library is free for commercial and non-commercial use as long as
- * the following conditions are aheared to.  The following conditions
- * apply to all code found in this distribution, be it the RC4, RSA,
- * lhash, DES, etc., code; not just the SSL code.  The SSL documentation
- * included with this distribution is covered by the same copyright terms
- * except that the holder is Tim Hudson (tjh@cryptsoft.com).
- *
- * Copyright remains Eric Young's, and as such any Copyright notices in
- * the code are not to be removed.
- * If this package is used in a product, Eric Young should be given attribution
- * as the author of the parts of the library used.
- * This can be in the form of a textual message at program startup or
- * in documentation (online or textual) provided with the package.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *    "This product includes cryptographic software written by
- *     Eric Young (eay@cryptsoft.com)"
- *    The word 'cryptographic' can be left out if the rouines from the library
- *    being used are not cryptographic related :-).
- * 4. If you include any Windows specific code (or a derivative thereof) from
- *    the apps directory (application code) you must include an acknowledgement:
- *    "This product includes software written by Tim Hudson (tjh@cryptsoft.com)"
- *
- * THIS SOFTWARE IS PROVIDED BY ERIC YOUNG ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- *
- * The licence and distribution terms for any publically available version or
- * derivative of this code cannot be changed.  i.e. this code cannot simply be
- * copied and put under another distribution licence
- * [including the GNU Public Licence.]
- */
-
-#include <stdio.h>
-
-#include <openssl/err.h>
-
-#include "bn_local.h"
-
-int
-BN_bn2mpi(const BIGNUM *a, unsigned char *d)
-{
-        int bits;
-        int num = 0;
-        int ext = 0;
-        long l;
-
-        bits = BN_num_bits(a);
-        num = (bits + 7) / 8;
-        if (bits > 0) {
-                ext = ((bits & 0x07) == 0);
-        }
-        if (d == NULL)
-                return (num + 4 + ext);
-
-        l = num + ext;
-        d[0] = (unsigned char)(l >> 24) & 0xff;
-        d[1] = (unsigned char)(l >> 16) & 0xff;
-        d[2] = (unsigned char)(l >> 8) & 0xff;
-        d[3] = (unsigned char)(l) & 0xff;
-        if (ext)
-                d[4] = 0;
-        num = BN_bn2bin(a, &(d[4 + ext]));
-        if (a->neg)
-                d[4] |= 0x80;
-        return (num + 4 + ext);
-}
-
-BIGNUM *
-BN_mpi2bn(const unsigned char *d, int n, BIGNUM *ain)
-{
-        BIGNUM *a = ain;
-        long len;
-        int neg = 0;
-
-        if (n < 4) {
-                BNerror(BN_R_INVALID_LENGTH);
-                return (NULL);
-        }
-        len = ((long)d[0] << 24) | ((long)d[1] << 16) | ((int)d[2] << 8) |
-            (int)d[3];
-        if ((len + 4) != n) {
-                BNerror(BN_R_ENCODING_ERROR);
-                return (NULL);
-        }
-
-        if (a == NULL)
-                a = BN_new();
-        if (a == NULL)
-                return (NULL);
-
-        if (len == 0) {
-                a->neg = 0;
-                a->top = 0;
-                return (a);
-        }
-        d += 4;
-        if ((*d) & 0x80)
-                neg = 1;
-        if (BN_bin2bn(d, (int)len, a) == NULL) {
-                if (ain == NULL)
-                        BN_free(a);
-                return (NULL);
-        }
-        BN_set_negative(a, neg);
-        if (neg) {
-                BN_clear_bit(a, BN_num_bits(a) - 1);
-        }
-        return (a);
-}