BIO_free() returns immediately when the sole input is NULL.
authordoug <doug@openbsd.org>
Fri, 25 Jul 2014 06:05:31 +0000 (06:05 +0000)
committerdoug <doug@openbsd.org>
Fri, 25 Jul 2014 06:05:31 +0000 (06:05 +0000)
Remove unnecessary NULL check.

ok miod@

26 files changed:
lib/libcrypto/asn1/bio_ndef.c
lib/libcrypto/bio/bio_lib.c
lib/libcrypto/bio/bss_acpt.c
lib/libcrypto/cms/cms_lib.c
lib/libcrypto/ocsp/ocsp_ht.c
lib/libcrypto/pkcs7/pk7_doit.c
lib/libcrypto/x509/by_file.c
lib/libssl/src/apps/apps.c
lib/libssl/src/apps/dh.c
lib/libssl/src/apps/dhparam.c
lib/libssl/src/apps/dsa.c
lib/libssl/src/apps/dsaparam.c
lib/libssl/src/apps/ec.c
lib/libssl/src/apps/ecparam.c
lib/libssl/src/apps/enc.c
lib/libssl/src/apps/gendsa.c
lib/libssl/src/apps/passwd.c
lib/libssl/src/apps/s_server.c
lib/libssl/src/apps/sess_id.c
lib/libssl/src/crypto/asn1/bio_ndef.c
lib/libssl/src/crypto/bio/bio_lib.c
lib/libssl/src/crypto/bio/bss_acpt.c
lib/libssl/src/crypto/cms/cms_lib.c
lib/libssl/src/crypto/ocsp/ocsp_ht.c
lib/libssl/src/crypto/pkcs7/pk7_doit.c
lib/libssl/src/crypto/x509/by_file.c

index fd6b0e5..1a23c27 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bio_ndef.c,v 1.8 2014/06/12 15:49:27 deraadt Exp $ */
+/* $OpenBSD: bio_ndef.c,v 1.9 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project.
  */
@@ -144,8 +144,7 @@ BIO_new_NDEF(BIO *out, ASN1_VALUE *val, const ASN1_ITEM *it)
        return sarg.ndef_bio;
 
 err:
-       if (asn_bio)
-               BIO_free(asn_bio);
+       BIO_free(asn_bio);
        free(ndef_aux);
        return NULL;
 }
index a2b34a2..477208e 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bio_lib.c,v 1.20 2014/07/11 15:40:32 miod Exp $ */
+/* $OpenBSD: bio_lib.c,v 1.21 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -575,8 +575,7 @@ BIO_dup_chain(BIO *in)
        }
        return (ret);
 err:
-       if (ret != NULL)
-               BIO_free(ret);
+       BIO_free(ret);
        return (NULL);
 
 }
index 65d4bb2..b6131c4 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bss_acpt.c,v 1.24 2014/07/13 16:03:09 beck Exp $ */
+/* $OpenBSD: bss_acpt.c,v 1.25 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -153,8 +153,7 @@ BIO_ACCEPT_free(BIO_ACCEPT *a)
 
        free(a->param_addr);
        free(a->addr);
-       if (a->bio_chain != NULL)
-               BIO_free(a->bio_chain);
+       BIO_free(a->bio_chain);
        free(a);
 }
 
@@ -358,8 +357,7 @@ acpt_ctrl(BIO *b, int cmd, long num, void *ptr)
                        } else if (num == 1) {
                                data->accept_nbio = (ptr != NULL);
                        } else if (num == 2) {
-                               if (data->bio_chain != NULL)
-                                       BIO_free(data->bio_chain);
+                               BIO_free(data->bio_chain);
                                data->bio_chain = (BIO *)ptr;
                        }
                }
index ee1f06c..4ca38e2 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: cms_lib.c,v 1.3 2014/06/12 15:49:28 deraadt Exp $ */
+/* $OpenBSD: cms_lib.c,v 1.4 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project.
  */
@@ -360,8 +360,7 @@ cms_DigestAlgorithm_init_bio(X509_ALGOR *digestAlgorithm)
        return mdbio;
 
 err:
-       if (mdbio)
-               BIO_free(mdbio);
+       BIO_free(mdbio);
        return NULL;
 }
 
index c895e9d..5d16271 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ocsp_ht.c,v 1.20 2014/07/12 14:58:32 miod Exp $ */
+/* $OpenBSD: ocsp_ht.c,v 1.21 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project 2006.
  */
@@ -111,8 +111,7 @@ OCSP_REQ_CTX_free(OCSP_REQ_CTX *rctx)
        if (rctx == NULL)
                return;
 
-       if (rctx->mem)
-               BIO_free(rctx->mem);
+       BIO_free(rctx->mem);
        free(rctx->iobuf);
        free(rctx);
 }
index cc6fd97..8f1e393 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: pk7_doit.c,v 1.28 2014/07/12 16:03:37 miod Exp $ */
+/* $OpenBSD: pk7_doit.c,v 1.29 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -133,8 +133,7 @@ PKCS7_bio_add_digest(BIO **pbio, X509_ALGOR *alg)
        return 1;
 
 err:
-       if (btmp)
-               BIO_free(btmp);
+       BIO_free(btmp);
        return 0;
 
 }
index 10adcfb..0436646 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: by_file.c,v 1.15 2014/07/11 08:44:49 jsing Exp $ */
+/* $OpenBSD: by_file.c,v 1.16 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -182,8 +182,7 @@ X509_load_cert_file(X509_LOOKUP *ctx, const char *file, int type)
 err:
        if (x != NULL)
                X509_free(x);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        return (ret);
 }
 
@@ -243,8 +242,7 @@ X509_load_crl_file(X509_LOOKUP *ctx, const char *file, int type)
 err:
        if (x != NULL)
                X509_CRL_free(x);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        return (ret);
 }
 
index f9e8452..7e7086a 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: apps.c,v 1.68 2014/07/19 03:40:26 lteo Exp $ */
+/* $OpenBSD: apps.c,v 1.69 2014/07/25 06:05:31 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -669,8 +669,7 @@ end:
                BIO_printf(err, "unable to load certificate\n");
                ERR_print_errors(err);
        }
-       if (cert != NULL)
-               BIO_free(cert);
+       BIO_free(cert);
        return (x);
 }
 
@@ -745,8 +744,7 @@ load_key(BIO *err, const char *file, int format, int maybe_stdin,
                goto end;
        }
 end:
-       if (key != NULL)
-               BIO_free(key);
+       BIO_free(key);
        if (pkey == NULL) {
                BIO_printf(err, "unable to load %s\n", key_descrip);
                ERR_print_errors(err);
@@ -833,8 +831,7 @@ load_pubkey(BIO *err, const char *file, int format, int maybe_stdin,
        }
 
 end:
-       if (key != NULL)
-               BIO_free(key);
+       BIO_free(key);
        if (pkey == NULL)
                BIO_printf(err, "unable to load %s\n", key_descrip);
        return (pkey);
index 1485ded..ce00082 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: dh.c,v 1.25 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: dh.c,v 1.26 2014/07/25 06:05:31 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -297,8 +297,7 @@ bad:
        ret = 0;
 
 end:
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (dh != NULL)
index 23fb2a5..1c41ff6 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: dhparam.c,v 1.33 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: dhparam.c,v 1.34 2014/07/25 06:05:31 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -441,8 +441,7 @@ bad:
        ret = 0;
 
 end:
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (dh != NULL)
index 287ecc4..2b90d4f 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: dsa.c,v 1.28 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: dsa.c,v 1.29 2014/07/25 06:05:31 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -320,8 +320,7 @@ bad:
        } else
                ret = 0;
 end:
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (dsa != NULL)
index 5980da6..46f41a5 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: dsaparam.c,v 1.34 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: dsaparam.c,v 1.35 2014/07/25 06:05:31 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -382,8 +382,7 @@ bad:
        ret = 0;
 
 end:
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (dsa != NULL)
index e60fddf..82a98dd 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ec.c,v 1.16 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: ec.c,v 1.17 2014/07/25 06:05:32 doug Exp $ */
 /*
  * Written by Nils Larsch for the OpenSSL project.
  */
@@ -328,8 +328,7 @@ bad:
        } else
                ret = 0;
 end:
-       if (in)
-               BIO_free(in);
+       BIO_free(in);
        if (out)
                BIO_free_all(out);
        if (eckey)
index e8e843e..9f7c287 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ecparam.c,v 1.23 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: ecparam.c,v 1.24 2014/07/25 06:05:32 doug Exp $ */
 /*
  * Written by Nils Larsch for the OpenSSL project.
  */
@@ -578,8 +578,7 @@ end:
        if (ec_cofactor)
                BN_free(ec_cofactor);
        free(buffer);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (group != NULL)
index 3464290..d1943bf 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: enc.c,v 1.38 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: enc.c,v 1.39 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -600,17 +600,13 @@ end:
        ERR_print_errors(bio_err);
        free(strbuf);
        free(buff);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
-       if (benc != NULL)
-               BIO_free(benc);
-       if (b64 != NULL)
-               BIO_free(b64);
+       BIO_free(benc);
+       BIO_free(b64);
 #ifdef ZLIB
-       if (bzl != NULL)
-               BIO_free(bzl);
+       BIO_free(bzl);
 #endif
        free(pass);
 
index 919b40a..d45e237 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: gendsa.c,v 1.30 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: gendsa.c,v 1.31 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -219,8 +219,7 @@ bad:
 end:
        if (ret != 0)
                ERR_print_errors(bio_err);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        if (out != NULL)
                BIO_free_all(out);
        if (dsa != NULL)
index b9a74bb..6a16a47 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: passwd.c,v 1.23 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: passwd.c,v 1.24 2014/07/25 06:05:32 doug Exp $ */
 
 #if defined OPENSSL_NO_MD5
 #define NO_MD5CRYPT_1
@@ -242,8 +242,7 @@ err:
        ERR_print_errors(bio_err);
        free(salt_malloc);
        free(passwd_malloc);
-       if (in)
-               BIO_free(in);
+       BIO_free(in);
        if (out)
                BIO_free_all(out);
 
index b6e16f3..3deebcf 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: s_server.c,v 1.61 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: s_server.c,v 1.62 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -1773,8 +1773,7 @@ load_dh_param(const char *dhfile)
                goto err;
        ret = PEM_read_bio_DHparams(bio, NULL, NULL, NULL);
 err:
-       if (bio != NULL)
-               BIO_free(bio);
+       BIO_free(bio);
        return (ret);
 }
 #endif
index e841003..0022abc 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: sess_id.c,v 1.19 2014/07/14 00:35:10 deraadt Exp $ */
+/* $OpenBSD: sess_id.c,v 1.20 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -277,7 +277,6 @@ load_sess_id(char *infile, int format)
                goto end;
        }
 end:
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        return (x);
 }
index fd6b0e5..1a23c27 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bio_ndef.c,v 1.8 2014/06/12 15:49:27 deraadt Exp $ */
+/* $OpenBSD: bio_ndef.c,v 1.9 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project.
  */
@@ -144,8 +144,7 @@ BIO_new_NDEF(BIO *out, ASN1_VALUE *val, const ASN1_ITEM *it)
        return sarg.ndef_bio;
 
 err:
-       if (asn_bio)
-               BIO_free(asn_bio);
+       BIO_free(asn_bio);
        free(ndef_aux);
        return NULL;
 }
index a2b34a2..477208e 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bio_lib.c,v 1.20 2014/07/11 15:40:32 miod Exp $ */
+/* $OpenBSD: bio_lib.c,v 1.21 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -575,8 +575,7 @@ BIO_dup_chain(BIO *in)
        }
        return (ret);
 err:
-       if (ret != NULL)
-               BIO_free(ret);
+       BIO_free(ret);
        return (NULL);
 
 }
index 65d4bb2..b6131c4 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: bss_acpt.c,v 1.24 2014/07/13 16:03:09 beck Exp $ */
+/* $OpenBSD: bss_acpt.c,v 1.25 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -153,8 +153,7 @@ BIO_ACCEPT_free(BIO_ACCEPT *a)
 
        free(a->param_addr);
        free(a->addr);
-       if (a->bio_chain != NULL)
-               BIO_free(a->bio_chain);
+       BIO_free(a->bio_chain);
        free(a);
 }
 
@@ -358,8 +357,7 @@ acpt_ctrl(BIO *b, int cmd, long num, void *ptr)
                        } else if (num == 1) {
                                data->accept_nbio = (ptr != NULL);
                        } else if (num == 2) {
-                               if (data->bio_chain != NULL)
-                                       BIO_free(data->bio_chain);
+                               BIO_free(data->bio_chain);
                                data->bio_chain = (BIO *)ptr;
                        }
                }
index ee1f06c..4ca38e2 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: cms_lib.c,v 1.3 2014/06/12 15:49:28 deraadt Exp $ */
+/* $OpenBSD: cms_lib.c,v 1.4 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project.
  */
@@ -360,8 +360,7 @@ cms_DigestAlgorithm_init_bio(X509_ALGOR *digestAlgorithm)
        return mdbio;
 
 err:
-       if (mdbio)
-               BIO_free(mdbio);
+       BIO_free(mdbio);
        return NULL;
 }
 
index c895e9d..5d16271 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ocsp_ht.c,v 1.20 2014/07/12 14:58:32 miod Exp $ */
+/* $OpenBSD: ocsp_ht.c,v 1.21 2014/07/25 06:05:32 doug Exp $ */
 /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project 2006.
  */
@@ -111,8 +111,7 @@ OCSP_REQ_CTX_free(OCSP_REQ_CTX *rctx)
        if (rctx == NULL)
                return;
 
-       if (rctx->mem)
-               BIO_free(rctx->mem);
+       BIO_free(rctx->mem);
        free(rctx->iobuf);
        free(rctx);
 }
index cc6fd97..8f1e393 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: pk7_doit.c,v 1.28 2014/07/12 16:03:37 miod Exp $ */
+/* $OpenBSD: pk7_doit.c,v 1.29 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -133,8 +133,7 @@ PKCS7_bio_add_digest(BIO **pbio, X509_ALGOR *alg)
        return 1;
 
 err:
-       if (btmp)
-               BIO_free(btmp);
+       BIO_free(btmp);
        return 0;
 
 }
index 10adcfb..0436646 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: by_file.c,v 1.15 2014/07/11 08:44:49 jsing Exp $ */
+/* $OpenBSD: by_file.c,v 1.16 2014/07/25 06:05:32 doug Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -182,8 +182,7 @@ X509_load_cert_file(X509_LOOKUP *ctx, const char *file, int type)
 err:
        if (x != NULL)
                X509_free(x);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        return (ret);
 }
 
@@ -243,8 +242,7 @@ X509_load_crl_file(X509_LOOKUP *ctx, const char *file, int type)
 err:
        if (x != NULL)
                X509_CRL_free(x);
-       if (in != NULL)
-               BIO_free(in);
+       BIO_free(in);
        return (ret);
 }