[tor-commits] [tor/master] Use calloc, not malloc(a*b), in ed25519 batch signature check fn

nickm at torproject.org nickm at torproject.org
Wed May 25 13:00:25 UTC 2016


commit 0ef36626ea0b3735d06360fde27100d33f2f5462
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed May 25 08:59:08 2016 -0400

    Use calloc, not malloc(a*b), in ed25519 batch signature check fn
    
    [Not a triggerable bug unless somebody is going to go checking
    millions+ of signatures in a single go.]
---
 src/common/crypto_ed25519.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/common/crypto_ed25519.c b/src/common/crypto_ed25519.c
index c687a1b..84c3eec 100644
--- a/src/common/crypto_ed25519.c
+++ b/src/common/crypto_ed25519.c
@@ -259,11 +259,11 @@ ed25519_checksig_batch(int *okay_out,
     int *oks;
     int all_ok;
 
-    ms = tor_malloc(sizeof(uint8_t*)*n_checkable);
-    lens = tor_malloc(sizeof(size_t)*n_checkable);
-    pks = tor_malloc(sizeof(uint8_t*)*n_checkable);
-    sigs = tor_malloc(sizeof(uint8_t*)*n_checkable);
-    oks = okay_out ? okay_out : tor_malloc(sizeof(int)*n_checkable);
+    ms = tor_calloc(n_checkable, sizeof(uint8_t*));
+    lens = tor_calloc(n_checkable, sizeof(size_t));
+    pks = tor_calloc(n_checkable, sizeof(uint8_t*));
+    sigs = tor_calloc(n_checkable, sizeof(uint8_t*));
+    oks = okay_out ? okay_out : tor_calloc(n_checkable, sizeof(int));
 
     for (i = 0; i < n_checkable; ++i) {
       ms[i] = checkable[i].msg;



More information about the tor-commits mailing list