Merge pull request #838 from maraoz/improve/README

return comment to deterministicK
bitpay · Dec 22, 2014 · 7473ab8 · 7473ab8
2 parents bf09710 + 225ade6
commit 7473ab8
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/lib/crypto/ecdsa.js b/lib/crypto/ecdsa.js
@@ -75,6 +75,9 @@ ECDSA.prototype.randomK = function() {
 // https://tools.ietf.org/html/rfc6979#section-3.2
 ECDSA.prototype.deterministicK = function(badrs) {
   /* jshint maxstatements: 25 */
+  // if r or s were invalid when this function was used in signing,
+  // we do not want to actually compute r, s here for efficiency, so,
+  // we can increment badrs. explained at end of RFC 6979 section 3.2
   if (_.isUndefined(badrs)) {
     badrs = 0;
   }