Rewrite X509_PUBKEY_set0_param() to use asn1_abs_set_unused_bits()

This streamlines the logic and uses ASN1_STRING_set0() and
asn1_abs_set_unused_bits() instead of inlining them.

ok jsing

1 files changed, 8 insertions, 10 deletions

diff --git a/src/lib/libcrypto/asn1/x_pubkey.c b/src/lib/libcrypto/asn1/x_pubkey.c
index 69be6ed1f9..a2c3d88e8a 100644
--- a/src/lib/libcrypto/asn1/x_pubkey.c
+++ b/src/lib/libcrypto/asn1/x_pubkey.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: x_pubkey.c,v 1.31 2021/12/25 13:17:48 jsing Exp $ */
+/* $OpenBSD: x_pubkey.c,v 1.32 2022/05/24 19:59:14 tb Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -716,15 +716,13 @@ X509_PUBKEY_set0_param(X509_PUBKEY *pub, ASN1_OBJECT *aobj, int ptype,
 {
         if (!X509_ALGOR_set0(pub->algor, aobj, ptype, pval))
                 return 0;
-        if (penc) {
-                free(pub->public_key->data);
-                pub->public_key->data = penc;
-                pub->public_key->length = penclen;
-                /* Set number of unused bits to zero */
-                pub->public_key->flags&= ~(ASN1_STRING_FLAG_BITS_LEFT|0x07);
-                pub->public_key->flags |= ASN1_STRING_FLAG_BITS_LEFT;
-        }
-        return 1;
+
+        if (penc == NULL)
+                return 1;
+
+        ASN1_STRING_set0(pub->public_key, penc, penclen);
+
+        return asn1_abs_set_unused_bits(pub->public_key, 0);
 }
 
 int