summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authortb <>2024-08-31 09:56:50 +0000
committertb <>2024-08-31 09:56:50 +0000
commitc92ddde3194a70c609ab5dd8987a10b6cf10c12f (patch)
tree88a649b66a44d0f9c4b6bd862981be47f4ac18c5 /src/lib
parentdd1c0533fa84d7ee8bbf2e4d256d3225e2057fe0 (diff)
downloadopenbsd-c92ddde3194a70c609ab5dd8987a10b6cf10c12f.tar.gz
openbsd-c92ddde3194a70c609ab5dd8987a10b6cf10c12f.tar.bz2
openbsd-c92ddde3194a70c609ab5dd8987a10b6cf10c12f.zip
Retire X509V3_EXT_{,CRL_,REQ_}add_conf()
Fortunately all projects who want to configure their extensions using a dangerous string DSL/API figured out the fact that one was supposed to be using the nconf version of these (the hint is the 'n', as in new). ok beck jsing
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/libcrypto/Symbols.list3
-rw-r--r--src/lib/libcrypto/hidden/openssl/x509v3.h5
-rw-r--r--src/lib/libcrypto/x509/x509_conf.c29
-rw-r--r--src/lib/libcrypto/x509/x509v3.h8
4 files changed, 3 insertions, 42 deletions
diff --git a/src/lib/libcrypto/Symbols.list b/src/lib/libcrypto/Symbols.list
index 31e9480c0d..c5187d65ee 100644
--- a/src/lib/libcrypto/Symbols.list
+++ b/src/lib/libcrypto/Symbols.list
@@ -2352,11 +2352,8 @@ WHIRLPOOL_Init
2352WHIRLPOOL_Update 2352WHIRLPOOL_Update
2353X25519 2353X25519
2354X25519_keypair 2354X25519_keypair
2355X509V3_EXT_CRL_add_conf
2356X509V3_EXT_CRL_add_nconf 2355X509V3_EXT_CRL_add_nconf
2357X509V3_EXT_REQ_add_conf
2358X509V3_EXT_REQ_add_nconf 2356X509V3_EXT_REQ_add_nconf
2359X509V3_EXT_add_conf
2360X509V3_EXT_add_nconf 2357X509V3_EXT_add_nconf
2361X509V3_EXT_add_nconf_sk 2358X509V3_EXT_add_nconf_sk
2362X509V3_EXT_conf 2359X509V3_EXT_conf
diff --git a/src/lib/libcrypto/hidden/openssl/x509v3.h b/src/lib/libcrypto/hidden/openssl/x509v3.h
index d0d4e97ec9..5912726791 100644
--- a/src/lib/libcrypto/hidden/openssl/x509v3.h
+++ b/src/lib/libcrypto/hidden/openssl/x509v3.h
@@ -1,4 +1,4 @@
1/* $OpenBSD: x509v3.h,v 1.12 2024/08/28 08:59:03 tb Exp $ */ 1/* $OpenBSD: x509v3.h,v 1.13 2024/08/31 09:56:50 tb Exp $ */
2/* 2/*
3 * Copyright (c) 2022 Bob Beck <beck@openbsd.org> 3 * Copyright (c) 2022 Bob Beck <beck@openbsd.org>
4 * 4 *
@@ -139,9 +139,6 @@ LCRYPTO_USED(X509V3_EXT_REQ_add_nconf);
139LCRYPTO_USED(X509V3_EXT_CRL_add_nconf); 139LCRYPTO_USED(X509V3_EXT_CRL_add_nconf);
140LCRYPTO_USED(X509V3_EXT_conf_nid); 140LCRYPTO_USED(X509V3_EXT_conf_nid);
141LCRYPTO_USED(X509V3_EXT_conf); 141LCRYPTO_USED(X509V3_EXT_conf);
142LCRYPTO_UNUSED(X509V3_EXT_add_conf);
143LCRYPTO_UNUSED(X509V3_EXT_REQ_add_conf);
144LCRYPTO_UNUSED(X509V3_EXT_CRL_add_conf);
145LCRYPTO_USED(X509V3_add_value_bool_nf); 142LCRYPTO_USED(X509V3_add_value_bool_nf);
146LCRYPTO_USED(X509V3_get_value_bool); 143LCRYPTO_USED(X509V3_get_value_bool);
147LCRYPTO_USED(X509V3_get_value_int); 144LCRYPTO_USED(X509V3_get_value_int);
diff --git a/src/lib/libcrypto/x509/x509_conf.c b/src/lib/libcrypto/x509/x509_conf.c
index 27f89072c8..dee9bf65e0 100644
--- a/src/lib/libcrypto/x509/x509_conf.c
+++ b/src/lib/libcrypto/x509/x509_conf.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: x509_conf.c,v 1.23 2024/08/31 09:21:44 tb Exp $ */ 1/* $OpenBSD: x509_conf.c,v 1.24 2024/08/31 09:56:50 tb Exp $ */
2/* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL 2/* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
3 * project 1999. 3 * project 1999.
4 */ 4 */
@@ -489,30 +489,3 @@ X509V3_set_conf_lhash(X509V3_CTX *ctx, LHASH_OF(CONF_VALUE) *lhash)
489{ 489{
490} 490}
491LCRYPTO_ALIAS(X509V3_set_conf_lhash); 491LCRYPTO_ALIAS(X509V3_set_conf_lhash);
492
493int
494X509V3_EXT_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
495 const char *section, X509 *cert)
496{
497 X509V3error(ERR_R_DISABLED);
498 return 0;
499}
500LCRYPTO_ALIAS(X509V3_EXT_add_conf);
501
502int
503X509V3_EXT_CRL_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
504 const char *section, X509_CRL *crl)
505{
506 X509V3error(ERR_R_DISABLED);
507 return 0;
508}
509LCRYPTO_ALIAS(X509V3_EXT_CRL_add_conf);
510
511int
512X509V3_EXT_REQ_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
513 const char *section, X509_REQ *req)
514{
515 X509V3error(ERR_R_DISABLED);
516 return 0;
517}
518LCRYPTO_ALIAS(X509V3_EXT_REQ_add_conf);
diff --git a/src/lib/libcrypto/x509/x509v3.h b/src/lib/libcrypto/x509/x509v3.h
index b38e175954..6470f059ee 100644
--- a/src/lib/libcrypto/x509/x509v3.h
+++ b/src/lib/libcrypto/x509/x509v3.h
@@ -1,4 +1,4 @@
1/* $OpenBSD: x509v3.h,v 1.30 2024/08/28 08:22:57 tb Exp $ */ 1/* $OpenBSD: x509v3.h,v 1.31 2024/08/31 09:56:50 tb Exp $ */
2/* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL 2/* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
3 * project 1999. 3 * project 1999.
4 */ 4 */
@@ -647,12 +647,6 @@ X509_EXTENSION *X509V3_EXT_conf_nid(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
647 int ext_nid, const char *value); 647 int ext_nid, const char *value);
648X509_EXTENSION *X509V3_EXT_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx, 648X509_EXTENSION *X509V3_EXT_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
649 const char *name, const char *value); 649 const char *name, const char *value);
650int X509V3_EXT_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
651 const char *section, X509 *cert);
652int X509V3_EXT_REQ_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
653 const char *section, X509_REQ *req);
654int X509V3_EXT_CRL_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx,
655 const char *section, X509_CRL *crl);
656 650
657int X509V3_add_value_bool_nf(const char *name, int asn1_bool, 651int X509V3_add_value_bool_nf(const char *name, int asn1_bool,
658 STACK_OF(CONF_VALUE) **extlist); 652 STACK_OF(CONF_VALUE) **extlist);