From 07fa1e73aee5fb447a989d049037204d9e133fd0 Mon Sep 17 00:00:00 2001
From: tb <>
Date: Thu, 14 Dec 2023 14:33:23 +0000
Subject: Dedup OBJ_nid2{obj,sn,ln}()

First get the obj corresponding to nid, then inspect its sn and ln.
Shaves off 40 lines of code and will simplify locking.
---
 src/lib/libcrypto/objects/obj_dat.c | 58 ++++++-------------------------------
 1 file changed, 9 insertions(+), 49 deletions(-)

(limited to 'src')

diff --git a/src/lib/libcrypto/objects/obj_dat.c b/src/lib/libcrypto/objects/obj_dat.c
index 94841b4265..5a2c34278e 100644
--- a/src/lib/libcrypto/objects/obj_dat.c
+++ b/src/lib/libcrypto/objects/obj_dat.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: obj_dat.c,v 1.68 2023/12/14 14:04:46 tb Exp $ */
+/* $OpenBSD: obj_dat.c,v 1.69 2023/12/14 14:33:23 tb Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -313,64 +313,24 @@ LCRYPTO_ALIAS(OBJ_nid2obj);
 const char *
 OBJ_nid2sn(int nid)
 {
-	if (nid >= 0 && nid < NUM_NID) {
-		if (nid == NID_undef || nid_objs[nid].nid != NID_undef)
-			return nid_objs[nid].sn;
-
-		goto unknown;
-	}
+	ASN1_OBJECT *aobj;
 
-	/* XXX - locking. */
-	if (added != NULL) {
-		ASN1_OBJECT aobj = {
-			.nid = nid,
-		};
-		ADDED_OBJ needle = {
-			.type = ADDED_NID,
-			.obj = &aobj,
-		};
-		ADDED_OBJ *found;
-
-		if ((found = lh_ADDED_OBJ_retrieve(added, &needle)) != NULL)
-			return found->obj->sn;
-	}
-
- unknown:
-	OBJerror(OBJ_R_UNKNOWN_NID);
+	if ((aobj = OBJ_nid2obj(nid)) == NULL)
+		return NULL;
 
-	return NULL;
+	return aobj->sn;
 }
 LCRYPTO_ALIAS(OBJ_nid2sn);
 
 const char *
 OBJ_nid2ln(int nid)
 {
-	if (nid >= 0 && nid < NUM_NID) {
-		if (nid == NID_undef || nid_objs[nid].nid != NID_undef)
-			return nid_objs[nid].ln;
-
-		goto unknown;
-	}
+	ASN1_OBJECT *aobj;
 
-	/* XXX - locking. */
-	if (added != NULL) {
-		ASN1_OBJECT aobj = {
-			.nid = nid,
-		};
-		ADDED_OBJ needle = {
-			.type = ADDED_NID,
-			.obj = &aobj,
-		};
-		ADDED_OBJ *found;
-
-		if ((found = lh_ADDED_OBJ_retrieve(added, &needle)) != NULL)
-			return found->obj->ln;
-	}
-
- unknown:
-	OBJerror(OBJ_R_UNKNOWN_NID);
+	if ((aobj = OBJ_nid2obj(nid)) == NULL)
+		return NULL;
 
-	return NULL;
+	return aobj->ln;
 }
 LCRYPTO_ALIAS(OBJ_nid2ln);
 
-- 
cgit v1.2.3-55-g6feb