diff options
author | miod <> | 2015-02-14 15:19:04 +0000 |
---|---|---|
committer | miod <> | 2015-02-14 15:19:04 +0000 |
commit | 7e1e1e77ce01ee4d256b26154c220a015fa615ae (patch) | |
tree | e2ff712c9e451c30c4493b58c3775874b6b896d3 /src | |
parent | 2b45e2949758075c07055aea2ee414357138fefc (diff) | |
download | openbsd-7e1e1e77ce01ee4d256b26154c220a015fa615ae.tar.gz openbsd-7e1e1e77ce01ee4d256b26154c220a015fa615ae.tar.bz2 openbsd-7e1e1e77ce01ee4d256b26154c220a015fa615ae.zip |
Memory leak in `should not happen' condition; Coverity CID 78889.
ok doug@ jsing@
Diffstat (limited to 'src')
-rw-r--r-- | src/lib/libcrypto/x509v3/v3_crld.c | 8 | ||||
-rw-r--r-- | src/lib/libssl/src/crypto/x509v3/v3_crld.c | 8 |
2 files changed, 8 insertions, 8 deletions
diff --git a/src/lib/libcrypto/x509v3/v3_crld.c b/src/lib/libcrypto/x509v3/v3_crld.c index 207e0944aa..b2e4370658 100644 --- a/src/lib/libcrypto/x509v3/v3_crld.c +++ b/src/lib/libcrypto/x509v3/v3_crld.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: v3_crld.c,v 1.15 2015/02/10 08:33:10 jsing Exp $ */ | 1 | /* $OpenBSD: v3_crld.c,v 1.16 2015/02/14 15:19:04 miod Exp $ */ |
2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL | 2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL |
3 | * project 1999. | 3 | * project 1999. |
4 | */ | 4 | */ |
@@ -204,10 +204,10 @@ set_reasons(ASN1_BIT_STRING **preas, char *value) | |||
204 | const char *bnam; | 204 | const char *bnam; |
205 | int i, ret = 0; | 205 | int i, ret = 0; |
206 | 206 | ||
207 | rsk = X509V3_parse_list(value); | 207 | if (*preas != NULL) |
208 | if (!rsk) | ||
209 | return 0; | 208 | return 0; |
210 | if (*preas) | 209 | rsk = X509V3_parse_list(value); |
210 | if (rsk == NULL) | ||
211 | return 0; | 211 | return 0; |
212 | for (i = 0; i < sk_CONF_VALUE_num(rsk); i++) { | 212 | for (i = 0; i < sk_CONF_VALUE_num(rsk); i++) { |
213 | bnam = sk_CONF_VALUE_value(rsk, i)->name; | 213 | bnam = sk_CONF_VALUE_value(rsk, i)->name; |
diff --git a/src/lib/libssl/src/crypto/x509v3/v3_crld.c b/src/lib/libssl/src/crypto/x509v3/v3_crld.c index 207e0944aa..b2e4370658 100644 --- a/src/lib/libssl/src/crypto/x509v3/v3_crld.c +++ b/src/lib/libssl/src/crypto/x509v3/v3_crld.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: v3_crld.c,v 1.15 2015/02/10 08:33:10 jsing Exp $ */ | 1 | /* $OpenBSD: v3_crld.c,v 1.16 2015/02/14 15:19:04 miod Exp $ */ |
2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL | 2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL |
3 | * project 1999. | 3 | * project 1999. |
4 | */ | 4 | */ |
@@ -204,10 +204,10 @@ set_reasons(ASN1_BIT_STRING **preas, char *value) | |||
204 | const char *bnam; | 204 | const char *bnam; |
205 | int i, ret = 0; | 205 | int i, ret = 0; |
206 | 206 | ||
207 | rsk = X509V3_parse_list(value); | 207 | if (*preas != NULL) |
208 | if (!rsk) | ||
209 | return 0; | 208 | return 0; |
210 | if (*preas) | 209 | rsk = X509V3_parse_list(value); |
210 | if (rsk == NULL) | ||
211 | return 0; | 211 | return 0; |
212 | for (i = 0; i < sk_CONF_VALUE_num(rsk); i++) { | 212 | for (i = 0; i < sk_CONF_VALUE_num(rsk); i++) { |
213 | bnam = sk_CONF_VALUE_value(rsk, i)->name; | 213 | bnam = sk_CONF_VALUE_value(rsk, i)->name; |