diff options
| author | tb <> | 2024-08-31 09:18:00 +0000 |
|---|---|---|
| committer | tb <> | 2024-08-31 09:18:00 +0000 |
| commit | f725589730b88830cf4e7f19e9e69b38a87802d9 (patch) | |
| tree | 08439682ae0357dcc698bee785d05e0a152d2e0b | |
| parent | 4d0ecb9ca6915541794e3cc736907b89e9a1dd6c (diff) | |
| download | openbsd-f725589730b88830cf4e7f19e9e69b38a87802d9.tar.gz openbsd-f725589730b88830cf4e7f19e9e69b38a87802d9.tar.bz2 openbsd-f725589730b88830cf4e7f19e9e69b38a87802d9.zip | |
Remove unused {,N}CONF_dump_{bio,fp}()
It's time to start removing some horrors from the conf/ garbage. If the
current top layer wasn't terrible enough, you could always slap another
one on top of it and it would then be truly marvellous.
ok beck jsing
| -rw-r--r-- | src/lib/libcrypto/Symbols.list | 4 | ||||
| -rw-r--r-- | src/lib/libcrypto/conf/conf.h | 6 | ||||
| -rw-r--r-- | src/lib/libcrypto/conf/conf_lib.c | 55 | ||||
| -rw-r--r-- | src/lib/libcrypto/hidden/openssl/conf.h | 6 |
4 files changed, 3 insertions, 68 deletions
diff --git a/src/lib/libcrypto/Symbols.list b/src/lib/libcrypto/Symbols.list index 05c39eddbb..3385b631e7 100644 --- a/src/lib/libcrypto/Symbols.list +++ b/src/lib/libcrypto/Symbols.list | |||
| @@ -589,8 +589,6 @@ CMS_unsigned_get_attr_by_OBJ | |||
| 589 | CMS_unsigned_get_attr_count | 589 | CMS_unsigned_get_attr_count |
| 590 | CMS_verify | 590 | CMS_verify |
| 591 | CMS_verify_receipt | 591 | CMS_verify_receipt |
| 592 | CONF_dump_bio | ||
| 593 | CONF_dump_fp | ||
| 594 | CONF_free | 592 | CONF_free |
| 595 | CONF_get1_default_config_file | 593 | CONF_get1_default_config_file |
| 596 | CONF_get_number | 594 | CONF_get_number |
| @@ -1505,8 +1503,6 @@ NAME_CONSTRAINTS_it | |||
| 1505 | NAME_CONSTRAINTS_new | 1503 | NAME_CONSTRAINTS_new |
| 1506 | NCONF_WIN32 | 1504 | NCONF_WIN32 |
| 1507 | NCONF_default | 1505 | NCONF_default |
| 1508 | NCONF_dump_bio | ||
| 1509 | NCONF_dump_fp | ||
| 1510 | NCONF_free | 1506 | NCONF_free |
| 1511 | NCONF_free_data | 1507 | NCONF_free_data |
| 1512 | NCONF_get_number_e | 1508 | NCONF_get_number_e |
diff --git a/src/lib/libcrypto/conf/conf.h b/src/lib/libcrypto/conf/conf.h index 5d10163bfb..7b30ad453d 100644 --- a/src/lib/libcrypto/conf/conf.h +++ b/src/lib/libcrypto/conf/conf.h | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: conf.h,v 1.16 2022/07/12 14:42:48 kn Exp $ */ | 1 | /* $OpenBSD: conf.h,v 1.17 2024/08/31 09:18:00 tb Exp $ */ |
| 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * | 4 | * |
| @@ -131,8 +131,6 @@ char *CONF_get_string(LHASH_OF(CONF_VALUE) *conf, const char *group, | |||
| 131 | long CONF_get_number(LHASH_OF(CONF_VALUE) *conf, const char *group, | 131 | long CONF_get_number(LHASH_OF(CONF_VALUE) *conf, const char *group, |
| 132 | const char *name); | 132 | const char *name); |
| 133 | void CONF_free(LHASH_OF(CONF_VALUE) *conf); | 133 | void CONF_free(LHASH_OF(CONF_VALUE) *conf); |
| 134 | int CONF_dump_fp(LHASH_OF(CONF_VALUE) *conf, FILE *out); | ||
| 135 | int CONF_dump_bio(LHASH_OF(CONF_VALUE) *conf, BIO *out); | ||
| 136 | 134 | ||
| 137 | void OPENSSL_config(const char *config_name); | 135 | void OPENSSL_config(const char *config_name); |
| 138 | void OPENSSL_no_config(void); | 136 | void OPENSSL_no_config(void); |
| @@ -159,8 +157,6 @@ STACK_OF(CONF_VALUE) *NCONF_get_section(const CONF *conf, const char *section); | |||
| 159 | char *NCONF_get_string(const CONF *conf, const char *group, const char *name); | 157 | char *NCONF_get_string(const CONF *conf, const char *group, const char *name); |
| 160 | int NCONF_get_number_e(const CONF *conf, const char *group, const char *name, | 158 | int NCONF_get_number_e(const CONF *conf, const char *group, const char *name, |
| 161 | long *result); | 159 | long *result); |
| 162 | int NCONF_dump_fp(const CONF *conf, FILE *out); | ||
| 163 | int NCONF_dump_bio(const CONF *conf, BIO *out); | ||
| 164 | 160 | ||
| 165 | #define NCONF_get_number(c,g,n,r) NCONF_get_number_e(c,g,n,r) | 161 | #define NCONF_get_number(c,g,n,r) NCONF_get_number_e(c,g,n,r) |
| 166 | 162 | ||
diff --git a/src/lib/libcrypto/conf/conf_lib.c b/src/lib/libcrypto/conf/conf_lib.c index d1c85dc098..01735906c0 100644 --- a/src/lib/libcrypto/conf/conf_lib.c +++ b/src/lib/libcrypto/conf/conf_lib.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: conf_lib.c,v 1.17 2024/04/09 13:56:30 beck Exp $ */ | 1 | /* $OpenBSD: conf_lib.c,v 1.18 2024/08/31 09:18:00 tb Exp $ */ |
| 2 | /* Written by Richard Levitte (richard@levitte.org) for the OpenSSL | 2 | /* Written by Richard Levitte (richard@levitte.org) for the OpenSSL |
| 3 | * project 2000. | 3 | * project 2000. |
| 4 | */ | 4 | */ |
| @@ -198,32 +198,6 @@ CONF_free(LHASH_OF(CONF_VALUE) *conf) | |||
| 198 | } | 198 | } |
| 199 | LCRYPTO_ALIAS(CONF_free); | 199 | LCRYPTO_ALIAS(CONF_free); |
| 200 | 200 | ||
| 201 | int | ||
| 202 | CONF_dump_fp(LHASH_OF(CONF_VALUE) *conf, FILE *out) | ||
| 203 | { | ||
| 204 | BIO *btmp; | ||
| 205 | int ret; | ||
| 206 | |||
| 207 | if (!(btmp = BIO_new_fp(out, BIO_NOCLOSE))) { | ||
| 208 | CONFerror(ERR_R_BUF_LIB); | ||
| 209 | return 0; | ||
| 210 | } | ||
| 211 | ret = CONF_dump_bio(conf, btmp); | ||
| 212 | BIO_free(btmp); | ||
| 213 | return ret; | ||
| 214 | } | ||
| 215 | LCRYPTO_ALIAS(CONF_dump_fp); | ||
| 216 | |||
| 217 | int | ||
| 218 | CONF_dump_bio(LHASH_OF(CONF_VALUE) *conf, BIO *out) | ||
| 219 | { | ||
| 220 | CONF ctmp; | ||
| 221 | |||
| 222 | CONF_set_nconf(&ctmp, conf); | ||
| 223 | return NCONF_dump_bio(&ctmp, out); | ||
| 224 | } | ||
| 225 | LCRYPTO_ALIAS(CONF_dump_bio); | ||
| 226 | |||
| 227 | /* The following section contains the "New CONF" functions. They are | 201 | /* The following section contains the "New CONF" functions. They are |
| 228 | completely centralised around a new CONF structure that may contain | 202 | completely centralised around a new CONF structure that may contain |
| 229 | basically anything, but at least a method pointer and a table of data. | 203 | basically anything, but at least a method pointer and a table of data. |
| @@ -368,30 +342,3 @@ NCONF_get_number_e(const CONF *conf, const char *group, const char *name, | |||
| 368 | return 1; | 342 | return 1; |
| 369 | } | 343 | } |
| 370 | LCRYPTO_ALIAS(NCONF_get_number_e); | 344 | LCRYPTO_ALIAS(NCONF_get_number_e); |
| 371 | |||
| 372 | int | ||
| 373 | NCONF_dump_fp(const CONF *conf, FILE *out) | ||
| 374 | { | ||
| 375 | BIO *btmp; | ||
| 376 | int ret; | ||
| 377 | if (!(btmp = BIO_new_fp(out, BIO_NOCLOSE))) { | ||
| 378 | CONFerror(ERR_R_BUF_LIB); | ||
| 379 | return 0; | ||
| 380 | } | ||
| 381 | ret = NCONF_dump_bio(conf, btmp); | ||
| 382 | BIO_free(btmp); | ||
| 383 | return ret; | ||
| 384 | } | ||
| 385 | LCRYPTO_ALIAS(NCONF_dump_fp); | ||
| 386 | |||
| 387 | int | ||
| 388 | NCONF_dump_bio(const CONF *conf, BIO *out) | ||
| 389 | { | ||
| 390 | if (conf == NULL) { | ||
| 391 | CONFerror(CONF_R_NO_CONF); | ||
| 392 | return 0; | ||
| 393 | } | ||
| 394 | |||
| 395 | return conf->meth->dump(conf, out); | ||
| 396 | } | ||
| 397 | LCRYPTO_ALIAS(NCONF_dump_bio); | ||
diff --git a/src/lib/libcrypto/hidden/openssl/conf.h b/src/lib/libcrypto/hidden/openssl/conf.h index 9408a543e3..9d073303ca 100644 --- a/src/lib/libcrypto/hidden/openssl/conf.h +++ b/src/lib/libcrypto/hidden/openssl/conf.h | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: conf.h,v 1.1 2024/04/09 14:57:28 tb Exp $ */ | 1 | /* $OpenBSD: conf.h,v 1.2 2024/08/31 09:18:00 tb Exp $ */ |
| 2 | /* | 2 | /* |
| 3 | * Copyright (c) 2024 Bob Beck <beck@openbsd.org> | 3 | * Copyright (c) 2024 Bob Beck <beck@openbsd.org> |
| 4 | * | 4 | * |
| @@ -34,8 +34,6 @@ LCRYPTO_USED(CONF_get_section); | |||
| 34 | LCRYPTO_USED(CONF_get_string); | 34 | LCRYPTO_USED(CONF_get_string); |
| 35 | LCRYPTO_USED(CONF_get_number); | 35 | LCRYPTO_USED(CONF_get_number); |
| 36 | LCRYPTO_USED(CONF_free); | 36 | LCRYPTO_USED(CONF_free); |
| 37 | LCRYPTO_USED(CONF_dump_fp); | ||
| 38 | LCRYPTO_USED(CONF_dump_bio); | ||
| 39 | LCRYPTO_USED(OPENSSL_config); | 37 | LCRYPTO_USED(OPENSSL_config); |
| 40 | LCRYPTO_USED(OPENSSL_no_config); | 38 | LCRYPTO_USED(OPENSSL_no_config); |
| 41 | LCRYPTO_USED(NCONF_new); | 39 | LCRYPTO_USED(NCONF_new); |
| @@ -49,8 +47,6 @@ LCRYPTO_USED(NCONF_load_bio); | |||
| 49 | LCRYPTO_USED(NCONF_get_section); | 47 | LCRYPTO_USED(NCONF_get_section); |
| 50 | LCRYPTO_USED(NCONF_get_string); | 48 | LCRYPTO_USED(NCONF_get_string); |
| 51 | LCRYPTO_USED(NCONF_get_number_e); | 49 | LCRYPTO_USED(NCONF_get_number_e); |
| 52 | LCRYPTO_USED(NCONF_dump_fp); | ||
| 53 | LCRYPTO_USED(NCONF_dump_bio); | ||
| 54 | LCRYPTO_USED(CONF_modules_load); | 50 | LCRYPTO_USED(CONF_modules_load); |
| 55 | LCRYPTO_USED(CONF_modules_load_file); | 51 | LCRYPTO_USED(CONF_modules_load_file); |
| 56 | LCRYPTO_USED(CONF_modules_unload); | 52 | LCRYPTO_USED(CONF_modules_unload); |
