From 45650e0e51a410d1f54301ae6a26f175869dba56 Mon Sep 17 00:00:00 2001 From: tb <> Date: Thu, 18 Apr 2024 11:51:01 +0000 Subject: Turn ecdh_cms_set_shared_info() into single exit ok jsing --- src/lib/libcrypto/ec/ec_ameth.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/lib/libcrypto/ec/ec_ameth.c b/src/lib/libcrypto/ec/ec_ameth.c index 155c1855b4..cbc8fa929e 100644 --- a/src/lib/libcrypto/ec/ec_ameth.c +++ b/src/lib/libcrypto/ec/ec_ameth.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ec_ameth.c,v 1.63 2024/04/17 14:01:33 tb Exp $ */ +/* $OpenBSD: ec_ameth.c,v 1.64 2024/04/18 11:51:01 tb Exp $ */ /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL * project 2006. */ @@ -826,18 +826,18 @@ ecdh_cms_set_shared_info(EVP_PKEY_CTX *pctx, CMS_RecipientInfo *ri) int plen, keylen; const EVP_CIPHER *kekcipher; EVP_CIPHER_CTX *kekctx; - int rv = 0; + int ret = 0; if (!CMS_RecipientInfo_kari_get0_alg(ri, &alg, &ukm)) - return 0; + goto err; if (!ecdh_cms_set_kdf_param(pctx, OBJ_obj2nid(alg->algorithm))) { ECerror(EC_R_KDF_PARAMETER_ERROR); - return 0; + goto err; } if (alg->parameter->type != V_ASN1_SEQUENCE) - return 0; + goto err; p = alg->parameter->value.sequence->data; plen = alg->parameter->value.sequence->length; @@ -867,11 +867,13 @@ ecdh_cms_set_shared_info(EVP_PKEY_CTX *pctx, CMS_RecipientInfo *ri) goto err; der = NULL; - rv = 1; + ret = 1; + err: X509_ALGOR_free(kekalg); free(der); - return rv; + + return ret; } static int -- cgit v1.2.3-55-g6feb