From 892fb814d9b781840da8fbd959dc3d6d2513aabd Mon Sep 17 00:00:00 2001 From: jsing Date: Sat, 21 May 2022 13:16:19 +0000 Subject: [PATCH] Factor out ASN1_ITYPE_EXTERN handling. Factor out the ef->asn1_ex_d2i() callback handling - this allows us to pull out all of the related variables into a self-contained function. ok tb@ --- lib/libcrypto/asn1/tasn_dec.c | 47 ++++++++++++++++++++++++----------- 1 file changed, 32 insertions(+), 15 deletions(-) diff --git a/lib/libcrypto/asn1/tasn_dec.c b/lib/libcrypto/asn1/tasn_dec.c index a769ace9758..03a4d6313b9 100644 --- a/lib/libcrypto/asn1/tasn_dec.c +++ b/lib/libcrypto/asn1/tasn_dec.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tasn_dec.c,v 1.74 2022/05/21 11:21:31 jsing Exp $ */ +/* $OpenBSD: tasn_dec.c,v 1.75 2022/05/21 13:16:19 jsing Exp $ */ /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL * project 2000. */ @@ -903,6 +903,35 @@ asn1_item_d2i_sequence(ASN1_VALUE **pval, CBS *cbs, const ASN1_ITEM *it, return 0; } +static int +asn1_item_d2i_extern(ASN1_VALUE **pval, CBS *cbs, const ASN1_ITEM *it, + int tag_number, int tag_class, char optional) +{ + const ASN1_EXTERN_FUNCS *ef = it->funcs; + const unsigned char *p = NULL; + ASN1_TLC ctx = { 0 }; + int ret = 0; + + if (CBS_len(cbs) > LONG_MAX) + return 0; + + p = CBS_data(cbs); + + if ((ret = ef->asn1_ex_d2i(pval, &p, (long)CBS_len(cbs), it, + tag_number, tag_class, optional, &ctx)) == 1) { + if (!CBS_skip(cbs, p - CBS_data(cbs))) + goto err; + } + return ret; + + err: + ASN1_item_ex_free(pval, it); + + ERR_asprintf_error_data("Type=%s", it->sname); + + return 0; +} + /* * Decode an item, taking care of IMPLICIT tagging, if any. * If 'opt' set and tag mismatch return -1 to handle OPTIONAL @@ -911,11 +940,6 @@ static int asn1_item_d2i(ASN1_VALUE **pval, CBS *cbs, const ASN1_ITEM *it, int tag_number, int tag_class, char optional, int depth) { - const ASN1_EXTERN_FUNCS *ef = it->funcs; - const unsigned char *p = NULL; - ASN1_TLC ctx = { 0 }; - int ret = 0; - if (pval == NULL) return 0; @@ -949,15 +973,8 @@ asn1_item_d2i(ASN1_VALUE **pval, CBS *cbs, const ASN1_ITEM *it, optional); case ASN1_ITYPE_EXTERN: - if (CBS_len(cbs) > LONG_MAX) - return 0; - p = CBS_data(cbs); - if ((ret = ef->asn1_ex_d2i(pval, &p, (long)CBS_len(cbs), it, - tag_number, tag_class, optional, &ctx)) == 1) { - if (!CBS_skip(cbs, p - CBS_data(cbs))) - goto err; - } - return ret; + return asn1_item_d2i_extern(pval, cbs, it, tag_number, + tag_class, optional); case ASN1_ITYPE_CHOICE: return asn1_item_d2i_choice(pval, cbs, it, tag_number, -- 2.20.1