[git] GCRYPT - branch, master, updated. libgcrypt-1.6.0-232-g2a7aa3e

by Peter Wu cvs at cvs.gnupg.org
Fri Jul 10 03:30:37 CEST 2015


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The GNU crypto library".

The branch, master has been updated
       via  2a7aa3ea4d03a9c808d5888f5509c08cd27aa27c (commit)
      from  0a7547e487a8bc4e7ac9599c55579eb2e4a13f06 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 2a7aa3ea4d03a9c808d5888f5509c08cd27aa27c
Author: Peter Wu <peter at lekensteyn.nl>
Date:   Fri Jul 10 10:15:26 2015 +0900

    ecc: fix memory leak.
    
    * cipher/ecc.c (ecc_verify): Release memory which was allocated before
    by _gcry_pk_util_preparse_sigval.
    (ecc_decrypt_raw): Likewise.
    
    --
    
    Caught by LeakSanitizer (LSan). Now the test suite (make check) passes
    with no memleaks.
    
    Signed-off-by: Peter Wu <peter at lekensteyn.nl>
    
    The last commit (0a7547e487a8bc4e7ac9599c55579eb2e4a13f06) includes
    wrong fixes for sexp_release.
    
    ecc_decrypt_raw fix added by gniibe.

diff --git a/cipher/ecc.c b/cipher/ecc.c
index f5bc50a..e33f999 100644
--- a/cipher/ecc.c
+++ b/cipher/ecc.c
@@ -1041,6 +1041,7 @@ ecc_verify (gcry_sexp_t s_sig, gcry_sexp_t s_data, gcry_sexp_t s_keyparms)
         goto leave;
     }
   /* Add missing parameters using the optional curve parameter.  */
+  sexp_release (l1);
   l1 = sexp_find_token (s_keyparms, "curve", 5);
   if (l1)
     {
@@ -1417,6 +1418,7 @@ ecc_decrypt_raw (gcry_sexp_t *r_plain, gcry_sexp_t s_data, gcry_sexp_t keyparms)
         goto leave;
     }
   /* Add missing parameters using the optional curve parameter.  */
+  sexp_release (l1);
   l1 = sexp_find_token (keyparms, "curve", 5);
   if (l1)
     {

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

Summary of changes:
 cipher/ecc.c | 2 ++
 1 file changed, 2 insertions(+)


hooks/post-receive
-- 
The GNU crypto library
http://git.gnupg.org


_______________________________________________
Gnupg-commits mailing list
Gnupg-commits at gnupg.org
http://lists.gnupg.org/mailman/listinfo/gnupg-commits




More information about the Gcrypt-devel mailing list