Merge pull request #482 from ryanxcharles/bug/k-too-small

k should be 32 bytes, not 8 bytes
2014-08-11 17:07:11 -03:00 · 2014-08-11 17:07:11 -03:00 · 3b60170e9f
parent d9ffe75dc5 9f9e2f1d41
commit 3b60170e9f
1 changed files with 2 additions and 1 deletions
--- a/lib/common/Key.js
+++ b/lib/common/Key.js
@ -159,7 +159,8 @@ Key.calcPubKeyRecoveryParam = function(e, r, s, Q) {

 Key.genk = function() {
  //TODO: account for when >= n
-  return new bignum(SecureRandom.getRandomBuffer(8));
+  var k = new bignum(SecureRandom.getRandomBuffer(32))
+  return k;
 };

 module.exports = Key;