Browse Source

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.]
Nick Mathewson 8 years ago
parent
commit
0ef36626ea
1 changed files with 5 additions and 5 deletions
  1. 5 5
      src/common/crypto_ed25519.c

+ 5 - 5
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;