diff options
author | teor <teor@torproject.org> | 2019-04-05 15:08:54 +1000 |
---|---|---|
committer | teor <teor@torproject.org> | 2019-04-05 15:17:19 +1000 |
commit | ce5e38642d1f5e48a7e5c98422e0fa23145f0363 (patch) | |
tree | ae91e9e569da5507cf407acb9c43306240881ce7 /src/feature/relay/router.c | |
parent | e3124fef54f90828f7b06c41fd4e39ef7778f2e3 (diff) | |
download | tor-ce5e38642d1f5e48a7e5c98422e0fa23145f0363.tar.gz tor-ce5e38642d1f5e48a7e5c98422e0fa23145f0363.zip |
crypto_format: Remove the return value from ed25519_signature_to_base64()
Also remove all checks for the return value, which were redundant anyway,
because the function never failed.
Part of 29660.
Diffstat (limited to 'src/feature/relay/router.c')
-rw-r--r-- | src/feature/relay/router.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/feature/relay/router.c b/src/feature/relay/router.c index 837465cfe9..ac4b3b7a02 100644 --- a/src/feature/relay/router.c +++ b/src/feature/relay/router.c @@ -2974,8 +2974,7 @@ router_dump_router_to_string(routerinfo_t *router, if (ed25519_sign(&sig, (const uint8_t*)digest, DIGEST256_LEN, signing_keypair) < 0) goto err; - if (ed25519_signature_to_base64(buf, &sig) < 0) - goto err; + ed25519_signature_to_base64(buf, &sig); smartlist_add_asprintf(chunks, "%s\n", buf); } @@ -3249,8 +3248,7 @@ extrainfo_dump_to_string(char **s_out, extrainfo_t *extrainfo, if (ed25519_sign(&ed_sig, (const uint8_t*)sha256_digest, DIGEST256_LEN, signing_keypair) < 0) goto err; - if (ed25519_signature_to_base64(buf, &ed_sig) < 0) - goto err; + ed25519_signature_to_base64(buf, &ed_sig); smartlist_add_asprintf(chunks, "%s\n", buf); } |