diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-05-30 21:32:27 +0200 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-05-31 01:24:51 +0200 |
commit | fc07df7d4c5d87a4789b13c2df6a482ebf9e924d (patch) | |
tree | 665074a29a61b754c0ce1990ebb34a09a21606c1 /contrib | |
parent | device: do not assume dst is always valid (diff) | |
download | wireguard-monolithic-historical-fc07df7d4c5d87a4789b13c2df6a482ebf9e924d.tar.xz wireguard-monolithic-historical-fc07df7d4c5d87a4789b13c2df6a482ebf9e924d.zip |
tools: constanter time encoding
Diffstat (limited to 'contrib')
-rw-r--r-- | contrib/examples/embeddable-wg-library/wireguard.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/contrib/examples/embeddable-wg-library/wireguard.c b/contrib/examples/embeddable-wg-library/wireguard.c index 16ddd9a..560d7ab 100644 --- a/contrib/examples/embeddable-wg-library/wireguard.c +++ b/contrib/examples/embeddable-wg-library/wireguard.c @@ -1559,25 +1559,25 @@ int wg_key_from_base64(wg_key key, const wg_key_b64_string base64) { unsigned int i; int val; + volatile uint8_t ret = 0; - errno = EINVAL; - if (strlen(base64) != sizeof(wg_key_b64_string) - 1 || base64[sizeof(wg_key_b64_string) - 2] != '=') + if (strlen(base64) != sizeof(wg_key_b64_string) - 1 || base64[sizeof(wg_key_b64_string) - 2] != '=') { + errno = EINVAL; goto out; + } for (i = 0; i < 32 / 3; ++i) { val = decode_base64(&base64[i * 4]); - if (val < 0) - goto out; + ret |= (uint32_t)val >> 31; key[i * 3 + 0] = (val >> 16) & 0xff; key[i * 3 + 1] = (val >> 8) & 0xff; key[i * 3 + 2] = val & 0xff; } val = decode_base64((const char[]){ base64[i * 4 + 0], base64[i * 4 + 1], base64[i * 4 + 2], 'A' }); - if (val < 0 || val & 0xff) - goto out; + ret |= ((uint32_t)val >> 31) | (val & 0xff); key[i * 3 + 0] = (val >> 16) & 0xff; key[i * 3 + 1] = (val >> 8) & 0xff; - errno = 0; + errno = EINVAL & ~((ret - 1) >> 8); out: return -errno; } |