diff options
| author |   tedu <tedu@openbsd.org> | 2014-04-19 17:40:49 +0000 |
|---|---|---|
| committer | tedu <tedu@openbsd.org> | 2014-04-19 17:40:49 +0000 |
| commit | 004dd34e5c93a1d96abb5221f54f4c3267e3ec3a (patch) | |
| tree | 33ee72e76108107c86ab3d8c58750a832bc57838 /lib/libssl/src | |
| parent | (void) cast snprintf() calls that cannot truncate (and would be harmless (diff) | |
| download | wireguard-openbsd-004dd34e5c93a1d96abb5221f54f4c3267e3ec3a.tar.xz wireguard-openbsd-004dd34e5c93a1d96abb5221f54f4c3267e3ec3a.zip | |
another attempt at fixing stale x509 data. since we don't know where the
initial storage came from, we can't free it. just memset in the sequence
case. probably ok beck
Diffstat (limited to 'lib/libssl/src')
| -rw-r--r-- | lib/libssl/src/crypto/asn1/tasn_dec.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/lib/libssl/src/crypto/asn1/tasn_dec.c b/lib/libssl/src/crypto/asn1/tasn_dec.c index 66f039806e9..560b6f3d5fc 100644 --- a/lib/libssl/src/crypto/asn1/tasn_dec.c +++ b/lib/libssl/src/crypto/asn1/tasn_dec.c @@ -172,12 +172,6 @@ ASN1_item_ex_d2i(ASN1_VALUE **pval, const unsigned char **in, long len, if (!pval) return 0; - /* always start fresh */ - if (0 && *pval) { - ASN1_item_ex_free(pval, it); - *pval = NULL; - } - if (aux && aux->asn1_cb) asn1_cb = aux->asn1_cb; else @@ -388,10 +382,14 @@ ASN1_item_ex_d2i(ASN1_VALUE **pval, const unsigned char **in, long len, goto err; } - if (!*pval && !ASN1_item_ex_new(pval, it)) { - ASN1err(ASN1_F_ASN1_ITEM_EX_D2I, - ERR_R_NESTED_ASN1_ERROR); - goto err; + if (!*pval) { + if (!ASN1_item_ex_new(pval, it)) { + ASN1err(ASN1_F_ASN1_ITEM_EX_D2I, + ERR_R_NESTED_ASN1_ERROR); + goto err; + } + } else { + memset(*pval, 0, it->size); } if (asn1_cb && !asn1_cb(ASN1_OP_D2I_PRE, pval, it, NULL)) |