diff options
author | Dr. David von Oheimb <David.von.Oheimb@siemens.com> | 2020-08-19 20:16:53 +0200 |
---|---|---|
committer | Dr. David von Oheimb <David.von.Oheimb@siemens.com> | 2020-11-27 16:53:32 +0100 |
commit | ee46dfbf2c117a9532f887b478c9c65d8f30d50c (patch) | |
tree | 1106cecd9a6cd196961caa1ccf7730508ee48d37 /crypto/asn1 | |
parent | asn1t.h: Improve comments documenting ASN1_ITYPE_... and the 'funcs' field (diff) | |
download | openssl-ee46dfbf2c117a9532f887b478c9c65d8f30d50c.tar.xz openssl-ee46dfbf2c117a9532f887b478c9c65d8f30d50c.zip |
X509_dup: fix copying of libctx and propq using new ASN1_OP_DUP_POST cb operation
Fixes #12680
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/12687)
Diffstat (limited to 'crypto/asn1')
-rw-r--r-- | crypto/asn1/a_dup.c | 25 |
1 files changed, 23 insertions, 2 deletions
diff --git a/crypto/asn1/a_dup.c b/crypto/asn1/a_dup.c index 624fef9e5c..bdefa448ec 100644 --- a/crypto/asn1/a_dup.c +++ b/crypto/asn1/a_dup.c @@ -9,7 +9,7 @@ #include <stdio.h> #include "internal/cryptlib.h" -#include <openssl/asn1.h> +#include <openssl/asn1t.h> #ifndef NO_OLD_ASN1 @@ -48,14 +48,26 @@ void *ASN1_dup(i2d_of_void *i2d, d2i_of_void *d2i, const void *x) void *ASN1_item_dup(const ASN1_ITEM *it, const void *x) { + ASN1_aux_cb *asn1_cb = NULL; unsigned char *b = NULL; const unsigned char *p; long i; - void *ret; + ASN1_VALUE *ret; if (x == NULL) return NULL; + if (it->itype == ASN1_ITYPE_SEQUENCE || it->itype == ASN1_ITYPE_CHOICE + || it->itype == ASN1_ITYPE_NDEF_SEQUENCE) { + const ASN1_AUX *aux = it->funcs; + + asn1_cb = aux != NULL ? aux->asn1_cb : NULL; + } + + if (asn1_cb != NULL + && !asn1_cb(ASN1_OP_DUP_PRE, (ASN1_VALUE **)&x, it, NULL)) + goto auxerr; + i = ASN1_item_i2d(x, &b, it); if (b == NULL) { ERR_raise(ERR_LIB_ASN1, ERR_R_MALLOC_FAILURE); @@ -64,5 +76,14 @@ void *ASN1_item_dup(const ASN1_ITEM *it, const void *x) p = b; ret = ASN1_item_d2i(NULL, &p, i, it); OPENSSL_free(b); + + if (asn1_cb != NULL + && !asn1_cb(ASN1_OP_DUP_POST, &ret, it, (void *)x)) + goto auxerr; + return ret; + + auxerr: + ERR_raise_data(ERR_LIB_ASN1, ASN1_R_AUX_ERROR, "Type=%s", it->sname); + return NULL; } |