From 99c3426cbcb91a1b3509671c11ebffb1778057e6 Mon Sep 17 00:00:00 2001 From: redwire Date: Sun, 21 Jul 2013 21:13:04 -0230 Subject: Going with irungentoo's suggestion and using memcpy instead of sectioning with arithmetic --- core/net_crypto.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/core/net_crypto.c b/core/net_crypto.c index 26102a02..166adafb 100644 --- a/core/net_crypto.c +++ b/core/net_crypto.c @@ -141,16 +141,11 @@ void increment_nonce(uint8_t * nonce) /* fill the given nonce with random bytes. */ void random_nonce(uint8_t * nonce) { - uint32_t i, j, r, m = crypto_box_NONCEBYTES / 3, ind = 0; - for(i = 0; i < m; ++i) + uint32_t i, temp; + for (i = 0; i < crypto_box_NONCEBYTES / 4; ++i) { - r = random_int(); - for (j = 0; j < 3; j++) - { - nonce[ind] = r % 1000 % 256; - r /= 1000; - ++ind; - } + uint32_t temp = random_int(); + memcpy(nonce + 4 * i, &temp, 4); } } -- cgit v1.2.3