diff options
author | Rich Salz <rsalz@akamai.com> | 2015-11-23 13:30:04 -0500 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2015-11-23 13:51:23 -0500 |
commit | cc99bfa76bd25e40672841c78db9cc171be5488b (patch) | |
tree | a6b7cfeb46abbd525e2597d70f820450b5de127e /crypto | |
parent | e44380a990a3edf1cd0c190c6459c8c026d53646 (diff) | |
download | openssl-cc99bfa76bd25e40672841c78db9cc171be5488b.tar.gz |
Fix a few missed "if (!ptr)" cleanups
And a scalar !x --> x==0 test
Reviewed-by: Kurt Roeckx <kurt@openssl.org>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/bio/bio_lib.c | 4 | ||||
-rw-r--r-- | crypto/x509/x_x509.c | 5 |
2 files changed, 4 insertions, 5 deletions
diff --git a/crypto/bio/bio_lib.c b/crypto/bio/bio_lib.c index 6ab471cbcb..0e3469d9cb 100644 --- a/crypto/bio/bio_lib.c +++ b/crypto/bio/bio_lib.c @@ -471,7 +471,7 @@ BIO *BIO_find_type(BIO *bio, int type) { int mt, mask; - if (!bio) + if (bio == NULL) return NULL; mask = type & 0xff; do { @@ -491,7 +491,7 @@ BIO *BIO_find_type(BIO *bio, int type) BIO *BIO_next(BIO *b) { - if (!b) + if (b == NULL) return NULL; return b->next_bio; } diff --git a/crypto/x509/x_x509.c b/crypto/x509/x_x509.c index ad2309cccf..cab17ddee6 100644 --- a/crypto/x509/x_x509.c +++ b/crypto/x509/x_x509.c @@ -175,12 +175,11 @@ X509 *d2i_X509_AUX(X509 **a, const unsigned char **pp, long length) /* Save start position */ q = *pp; - if (!a || *a == NULL) { + if (a == NULL || *a == NULL) freeret = 1; - } ret = d2i_X509(a, &q, length); /* If certificate unreadable then forget it */ - if (!ret) + if (ret == NULL) return NULL; /* update length */ length -= q - *pp; |