more: no need for null check before free

ok tedu guenther

2 files changed, 7 insertions, 14 deletions

diff --git a/src/lib/libcrypto/x509v3/v3_prn.c b/src/lib/libcrypto/x509v3/v3_prn.c
index ff08534f9a..d06932aef1 100644
--- a/src/lib/libcrypto/x509v3/v3_prn.c
+++ b/src/lib/libcrypto/x509v3/v3_prn.c
@@ -143,8 +143,7 @@ X509V3_EXT_print(BIO *out, X509_EXTENSION *ext, unsigned long flag, int indent)
 
 err:
         sk_CONF_VALUE_pop_free(nval, X509V3_conf_free);
-        if (value)
-                free(value);
+        free(value);
         if (method->it)
                 ASN1_item_free(ext_str, ASN1_ITEM_ptr(method->it));
         else
diff --git a/src/lib/libcrypto/x509v3/v3_utl.c b/src/lib/libcrypto/x509v3/v3_utl.c
index 0b389807dd..3134feff21 100644
--- a/src/lib/libcrypto/x509v3/v3_utl.c
+++ b/src/lib/libcrypto/x509v3/v3_utl.c
@@ -103,12 +103,9 @@ X509V3_add_value(const char *name, const char *value,
 
 err:
         X509V3err(X509V3_F_X509V3_ADD_VALUE, ERR_R_MALLOC_FAILURE);
-        if (vtmp)
-                free(vtmp);
-        if (tname)
-                free(tname);
-        if (tvalue)
-                free(tvalue);
+        free(vtmp);
+        free(tname);
+        free(tvalue);
         return 0;
 }
 
@@ -126,12 +123,9 @@ X509V3_conf_free(CONF_VALUE *conf)
 {
         if (!conf)
                 return;
-        if (conf->name)
-                free(conf->name);
-        if (conf->value)
-                free(conf->value);
-        if (conf->section)
-                free(conf->section);
+        free(conf->name);
+        free(conf->value);
+        free(conf->section);
         free(conf);
 }