summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNikos Mavrogiannopoulos <nmav@gnutls.org>2011-05-21 12:51:32 +0200
committerNiels Möller <nisse@lysator.liu.se>2011-05-21 12:51:32 +0200
commit037d694ba0e9d9e84f30b2176231fa1e06680c03 (patch)
tree2ddc043e133d45541946ddde0c181cd45218aa44
parent7c6aadc71e340a4309922335816d4cb754557d74 (diff)
downloadnettle-037d694ba0e9d9e84f30b2176231fa1e06680c03.tar.gz
(dsa_generate_keypair): Avoid double init of mpz
variable. Spotted by Nikos Mavrogiannopoulos. Rev: nettle/ChangeLog:1.167 Rev: nettle/dsa-keygen.c:1.10
-rw-r--r--ChangeLog5
-rw-r--r--dsa-keygen.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 2a61f702..2a4e2c52 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-05-21 Niels Möller <nisse@lysator.liu.se>
+
+ * dsa-keygen.c (dsa_generate_keypair): Avoid double init of mpz
+ variable. Spotted by Nikos Mavrogiannopoulos.
+
2011-05-06 Niels Möller <nisse@lysator.liu.se>
* configure.ac: Fix link flags for shared libraries on Solaris,
diff --git a/dsa-keygen.c b/dsa-keygen.c
index be177712..00bd35f6 100644
--- a/dsa-keygen.c
+++ b/dsa-keygen.c
@@ -106,7 +106,7 @@ dsa_generate_keypair(struct dsa_public_key *pub,
if (progress)
progress (progress_ctx, 'g');
- mpz_init_set(r, pub->q);
+ mpz_set(r, pub->q);
mpz_sub_ui(r, r, 2);
nettle_mpz_random(key->x, random_ctx, random, r);