ecc: Store EdDSA session key in secure memory.
authorJo Van Bulck <jo.vanbulck@cs.kuleuven.be>
Thu, 19 Jan 2017 16:00:15 +0000 (17:00 +0100)
committerWerner Koch <wk@gnupg.org>
Thu, 1 Jun 2017 13:52:10 +0000 (15:52 +0200)
* cipher/ecc-eddsa.c (_gcry_ecc_eddsa_sign): use mpi_snew to allocate
session key.
--

An attacker who learns the EdDSA session key from side-channel
observation during the signing process, can easily revover the long-
term secret key. Storing the session key in secure memory ensures that
constant time point operations are used in the MPI library.

Signed-off-by: Jo Van Bulck <jo.vanbulck@cs.kuleuven.be>
cipher/ecc-eddsa.c

index f91f848..813e030 100644 (file)
@@ -603,7 +603,7 @@ _gcry_ecc_eddsa_sign (gcry_mpi_t input, ECC_secret_key *skey,
   a = mpi_snew (0);
   x = mpi_new (0);
   y = mpi_new (0);
-  r = mpi_new (0);
+  r = mpi_snew (0);
   ctx = _gcry_mpi_ec_p_internal_new (skey->E.model, skey->E.dialect, 0,
                                      skey->E.p, skey->E.a, skey->E.b);
   b = (ctx->nbits+7)/8;