diff options
author | tb <> | 2024-08-31 09:56:50 +0000 |
---|---|---|
committer | tb <> | 2024-08-31 09:56:50 +0000 |
commit | c92ddde3194a70c609ab5dd8987a10b6cf10c12f (patch) | |
tree | 88a649b66a44d0f9c4b6bd862981be47f4ac18c5 | |
parent | dd1c0533fa84d7ee8bbf2e4d256d3225e2057fe0 (diff) | |
download | openbsd-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
-rw-r--r-- | src/lib/libcrypto/Symbols.list | 3 | ||||
-rw-r--r-- | src/lib/libcrypto/hidden/openssl/x509v3.h | 5 | ||||
-rw-r--r-- | src/lib/libcrypto/x509/x509_conf.c | 29 | ||||
-rw-r--r-- | src/lib/libcrypto/x509/x509v3.h | 8 |
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 | |||
2352 | WHIRLPOOL_Update | 2352 | WHIRLPOOL_Update |
2353 | X25519 | 2353 | X25519 |
2354 | X25519_keypair | 2354 | X25519_keypair |
2355 | X509V3_EXT_CRL_add_conf | ||
2356 | X509V3_EXT_CRL_add_nconf | 2355 | X509V3_EXT_CRL_add_nconf |
2357 | X509V3_EXT_REQ_add_conf | ||
2358 | X509V3_EXT_REQ_add_nconf | 2356 | X509V3_EXT_REQ_add_nconf |
2359 | X509V3_EXT_add_conf | ||
2360 | X509V3_EXT_add_nconf | 2357 | X509V3_EXT_add_nconf |
2361 | X509V3_EXT_add_nconf_sk | 2358 | X509V3_EXT_add_nconf_sk |
2362 | X509V3_EXT_conf | 2359 | X509V3_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); | |||
139 | LCRYPTO_USED(X509V3_EXT_CRL_add_nconf); | 139 | LCRYPTO_USED(X509V3_EXT_CRL_add_nconf); |
140 | LCRYPTO_USED(X509V3_EXT_conf_nid); | 140 | LCRYPTO_USED(X509V3_EXT_conf_nid); |
141 | LCRYPTO_USED(X509V3_EXT_conf); | 141 | LCRYPTO_USED(X509V3_EXT_conf); |
142 | LCRYPTO_UNUSED(X509V3_EXT_add_conf); | ||
143 | LCRYPTO_UNUSED(X509V3_EXT_REQ_add_conf); | ||
144 | LCRYPTO_UNUSED(X509V3_EXT_CRL_add_conf); | ||
145 | LCRYPTO_USED(X509V3_add_value_bool_nf); | 142 | LCRYPTO_USED(X509V3_add_value_bool_nf); |
146 | LCRYPTO_USED(X509V3_get_value_bool); | 143 | LCRYPTO_USED(X509V3_get_value_bool); |
147 | LCRYPTO_USED(X509V3_get_value_int); | 144 | LCRYPTO_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 | } |
491 | LCRYPTO_ALIAS(X509V3_set_conf_lhash); | 491 | LCRYPTO_ALIAS(X509V3_set_conf_lhash); |
492 | |||
493 | int | ||
494 | X509V3_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 | } | ||
500 | LCRYPTO_ALIAS(X509V3_EXT_add_conf); | ||
501 | |||
502 | int | ||
503 | X509V3_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 | } | ||
509 | LCRYPTO_ALIAS(X509V3_EXT_CRL_add_conf); | ||
510 | |||
511 | int | ||
512 | X509V3_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 | } | ||
518 | LCRYPTO_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); |
648 | X509_EXTENSION *X509V3_EXT_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx, | 648 | X509_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); |
650 | int X509V3_EXT_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx, | ||
651 | const char *section, X509 *cert); | ||
652 | int X509V3_EXT_REQ_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx, | ||
653 | const char *section, X509_REQ *req); | ||
654 | int X509V3_EXT_CRL_add_conf(LHASH_OF(CONF_VALUE) *conf, X509V3_CTX *ctx, | ||
655 | const char *section, X509_CRL *crl); | ||
656 | 650 | ||
657 | int X509V3_add_value_bool_nf(const char *name, int asn1_bool, | 651 | int X509V3_add_value_bool_nf(const char *name, int asn1_bool, |
658 | STACK_OF(CONF_VALUE) **extlist); | 652 | STACK_OF(CONF_VALUE) **extlist); |