[tor-commits] [tor/master] crypt_ops: Stop using a separate buffer in ed25519_signature_from_base64()

asn at torproject.org asn at torproject.org
Fri Apr 12 10:48:08 UTC 2019


commit ebbc2c3d8f8268c09b6df8270907a1123f926caa
Author: teor <teor at torproject.org>
Date:   Fri Apr 12 13:00:02 2019 +1000

    crypt_ops: Stop using a separate buffer in ed25519_signature_from_base64()
    
    Part of 29960.
---
 src/lib/crypt_ops/crypto_format.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/lib/crypt_ops/crypto_format.c b/src/lib/crypt_ops/crypto_format.c
index 1827168c7..e11b39119 100644
--- a/src/lib/crypt_ops/crypto_format.c
+++ b/src/lib/crypt_ops/crypto_format.c
@@ -248,11 +248,9 @@ ed25519_signature_from_base64(ed25519_signature_t *sig,
 {
   if (strlen(input) != ED25519_SIG_BASE64_LEN)
     return -1;
-  char buf[ED25519_SIG_BASE64_LEN+1];
-  memcpy(buf, input, ED25519_SIG_BASE64_LEN);
-  buf[ED25519_SIG_BASE64_LEN] = 0;
   char decoded[128];
-  int n = base64_decode(decoded, sizeof(decoded), buf, strlen(buf));
+  int n = base64_decode(decoded, sizeof(decoded), input,
+                        ED25519_SIG_BASE64_LEN);
   if (n < 0 || n != ED25519_SIG_LEN)
     return -1;
   memcpy(sig->sig, decoded, ED25519_SIG_LEN);





More information about the tor-commits mailing list