[openssl-commits] [openssl] master update
Richard Levitte
levitte at openssl.org
Sun Dec 10 10:24:43 UTC 2017
The branch master has been updated
via df364297495e1d7409f1fcb0f4432c8671019cd8 (commit)
from 397e23f8db5aecfaef4d470b0c421d2cd84da6f7 (commit)
- Log -----------------------------------------------------------------
commit df364297495e1d7409f1fcb0f4432c8671019cd8
Author: FdaSilvaYY <fdasilvayy at gmail.com>
Date: Sat Dec 9 15:49:13 2017 +0100
Useless conf != NULL test
check is already made 10 line above.
clean commented code
Reviewed-by: Rich Salz <rsalz at openssl.org>
Reviewed-by: Richard Levitte <levitte at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4885)
-----------------------------------------------------------------------
Summary of changes:
apps/ca.c | 35 ++++++++++++++---------------------
1 file changed, 14 insertions(+), 21 deletions(-)
diff --git a/apps/ca.c b/apps/ca.c
index d08db6b..ea2750a 100644
--- a/apps/ca.c
+++ b/apps/ca.c
@@ -442,30 +442,23 @@ end_of_options:
&& (section = lookup_conf(conf, BASE_SECTION, ENV_DEFAULT_CA)) == NULL)
goto end;
- if (conf != NULL) {
- p = NCONF_get_string(conf, NULL, "oid_file");
- if (p == NULL)
- ERR_clear_error();
- if (p != NULL) {
- BIO *oid_bio;
+ p = NCONF_get_string(conf, NULL, "oid_file");
+ if (p == NULL)
+ ERR_clear_error();
+ if (p != NULL) {
+ BIO *oid_bio = BIO_new_file(p, "r");
- oid_bio = BIO_new_file(p, "r");
- if (oid_bio == NULL) {
- /*-
- BIO_printf(bio_err,"problems opening %s for extra oid's\n",p);
- ERR_print_errors(bio_err);
- */
- ERR_clear_error();
- } else {
- OBJ_create_objects(oid_bio);
- BIO_free(oid_bio);
- }
- }
- if (!add_oid_section(conf)) {
- ERR_print_errors(bio_err);
- goto end;
+ if (oid_bio == NULL) {
+ ERR_clear_error();
+ } else {
+ OBJ_create_objects(oid_bio);
+ BIO_free(oid_bio);
}
}
+ if (!add_oid_section(conf)) {
+ ERR_print_errors(bio_err);
+ goto end;
+ }
app_RAND_load_conf(conf, BASE_SECTION);
More information about the openssl-commits
mailing list