Provide X509_PUBKEY_get0() by splitting X509_PUBKEY_get() and turning it

into a wrapper that calls X509_PUBKEY_get0() and up refs.

1 files changed, 16 insertions, 6 deletions

diff --git a/src/lib/libcrypto/asn1/x_pubkey.c b/src/lib/libcrypto/asn1/x_pubkey.c
index 738507bbb6..ea67419cb2 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.26 2017/01/29 17:49:22 beck Exp $ */
+/* $OpenBSD: x_pubkey.c,v 1.27 2018/03/17 14:55:39 jsing Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -175,17 +175,15 @@ error:
 }
 
 EVP_PKEY *
-X509_PUBKEY_get(X509_PUBKEY *key)
+X509_PUBKEY_get0(X509_PUBKEY *key)
 {
         EVP_PKEY *ret = NULL;
 
         if (key == NULL)
                 goto error;
 
-        if (key->pkey != NULL) {
-                CRYPTO_add(&key->pkey->references, 1, CRYPTO_LOCK_EVP_PKEY);
+        if (key->pkey != NULL)
                 return key->pkey;
-        }
 
         if (key->public_key == NULL)
                 goto error;
@@ -220,7 +218,6 @@ X509_PUBKEY_get(X509_PUBKEY *key)
                 key->pkey = ret;
                 CRYPTO_w_unlock(CRYPTO_LOCK_EVP_PKEY);
         }
-        CRYPTO_add(&ret->references, 1, CRYPTO_LOCK_EVP_PKEY);
 
         return ret;
 
@@ -229,6 +226,19 @@ error:
         return (NULL);
 }
 
+EVP_PKEY *
+X509_PUBKEY_get(X509_PUBKEY *key)
+{
+        EVP_PKEY *pkey;
+
+        if ((pkey = X509_PUBKEY_get0(key)) == NULL)
+                return (NULL);
+
+        CRYPTO_add(&pkey->references, 1, CRYPTO_LOCK_EVP_PKEY);
+
+        return pkey;
+}
+
 /* Now two pseudo ASN1 routines that take an EVP_PKEY structure
  * and encode or decode as X509_PUBKEY
  */