diff options
| author |   bluhm <bluhm@openbsd.org> | 2018-04-06 12:16:06 +0000 |
|---|---|---|
| committer | bluhm <bluhm@openbsd.org> | 2018-04-06 12:16:06 +0000 |
| commit | 3119b8365f79c88c333ce067fa88894ca728d5a4 (patch) | |
| tree | c737861db03a799912c3a00fab7ec38b98a72bf1 /lib | |
| parent | Use the existing pf state to speed up UDP socket lookup. This was (diff) | |
| download | wireguard-openbsd-3119b8365f79c88c333ce067fa88894ca728d5a4.tar.xz wireguard-openbsd-3119b8365f79c88c333ce067fa88894ca728d5a4.zip | |
Revert revision 1.12 commit. Although *pval looks like a C pointer,
it may be something else. For primitive types it is possible that
a boolean int has been casted to an ASN1_VALUE pointer. Then the
64 bit read access to *pval may crash due to alignent or 32 bit
size.
bug report Anton Borowka; OK tedu@ jsing@ miod@
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/libcrypto/asn1/tasn_fre.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/libcrypto/asn1/tasn_fre.c b/lib/libcrypto/asn1/tasn_fre.c index 9276034808f..c05310ec285 100644 --- a/lib/libcrypto/asn1/tasn_fre.c +++ b/lib/libcrypto/asn1/tasn_fre.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tasn_fre.c,v 1.15 2016/12/30 16:04:34 jsing Exp $ */ +/* $OpenBSD: tasn_fre.c,v 1.16 2018/04/06 12:16:06 bluhm Exp $ */ /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL * project 2000. */ @@ -88,7 +88,10 @@ asn1_item_combine_free(ASN1_VALUE **pval, const ASN1_ITEM *it, int combine) ASN1_aux_cb *asn1_cb = NULL; int i; - if (pval == NULL || *pval == NULL) + if (pval == NULL) + return; + /* For primitive types *pval may be something other than C pointer. */ + if (it->itype != ASN1_ITYPE_PRIMITIVE && *pval == NULL) return; if (aux != NULL && aux->asn1_cb != NULL) |