[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