diff options
| author | miod <> | 2015-02-14 15:19:04 +0000 |
|---|---|---|
| committer | miod <> | 2015-02-14 15:19:04 +0000 |
| commit | bac381ab60e124223ceb8ee451aceca22f9827b7 (patch) | |
| tree | e2ff712c9e451c30c4493b58c3775874b6b896d3 | |
| parent | 1f4fbcbb948d21d3355681a520712b652ae10c1d (diff) | |
| download | openbsd-bac381ab60e124223ceb8ee451aceca22f9827b7.tar.gz openbsd-bac381ab60e124223ceb8ee451aceca22f9827b7.tar.bz2 openbsd-bac381ab60e124223ceb8ee451aceca22f9827b7.zip | |
Memory leak in `should not happen' condition; Coverity CID 78889.
ok doug@ jsing@
Diffstat (limited to '')
| -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; |
