[openssl] master update

tomas at openssl.org tomas at openssl.org
Fri Dec 17 07:58:38 UTC 2021


The branch master has been updated
       via  ec9135a62320c861ab17f7179ebe470686360c64 (commit)
      from  869b7dd00046951efb06dbb13c052ff9d7c87113 (commit)


- Log -----------------------------------------------------------------
commit ec9135a62320c861ab17f7179ebe470686360c64
Author: Peiwei Hu <jlu.hpw at foxmail.com>
Date:   Wed Dec 15 17:46:04 2021 +0800

    get_ecdsa_sig_rs_bytes: free value of d2i_ECDSA_SIG() before return
    
    Reviewed-by: Paul Dale <pauli at openssl.org>
    Reviewed-by: Matt Caswell <matt at openssl.org>
    Reviewed-by: Tomas Mraz <tomas at openssl.org>
    (Merged from https://github.com/openssl/openssl/pull/17280)

-----------------------------------------------------------------------

Summary of changes:
 test/acvp_test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/acvp_test.c b/test/acvp_test.c
index d8425f0d20..bbc77d0ae6 100644
--- a/test/acvp_test.c
+++ b/test/acvp_test.c
@@ -218,7 +218,7 @@ static int get_ecdsa_sig_rs_bytes(const unsigned char *sig, size_t sig_len,
     r1 = ECDSA_SIG_get0_r(sign);
     s1 = ECDSA_SIG_get0_s(sign);
     if (r1 == NULL || s1 == NULL)
-        return 0;
+        goto err;
 
     r1_len = BN_num_bytes(r1);
     s1_len = BN_num_bytes(s1);
@@ -560,7 +560,7 @@ static int get_dsa_sig_rs_bytes(const unsigned char *sig, size_t sig_len,
         return 0;
     DSA_SIG_get0(sign, &r1, &s1);
     if (r1 == NULL || s1 == NULL)
-        return 0;
+        goto err;
 
     r1_len = BN_num_bytes(r1);
     s1_len = BN_num_bytes(s1);


More information about the openssl-commits mailing list