Avoid the use of rand() in tests.

We control the random functions in crypto_core, so we can make them
deterministic more easily. This will help test reproducibility in the
future.

3 files changed, 17 insertions, 0 deletions

diff --git a/toxcore/crypto_core.api.h b/toxcore/crypto_core.api.h
index bb2c0a13..e9e8aeb2 100644
--- a/toxcore/crypto_core.api.h
+++ b/toxcore/crypto_core.api.h
@@ -117,6 +117,11 @@ static int32_t public_key_cmp(
 namespace random {
 
 /**
+ * Return a random 8 bit integer.
+ */
+static uint8_t u08();
+
+/**
  * Return a random 16 bit integer.
  */
 static uint16_t u16();
diff --git a/toxcore/crypto_core.c b/toxcore/crypto_core.c
index 26589219..b2f0e5f0 100644
--- a/toxcore/crypto_core.c
+++ b/toxcore/crypto_core.c
@@ -86,6 +86,13 @@ int32_t public_key_cmp(const uint8_t *pk1, const uint8_t *pk2)
     return crypto_verify_32(pk1, pk2);
 }
 
+uint8_t random_u08(void)
+{
+    uint8_t randnum;
+    randombytes(&randnum, 1);
+    return randnum;
+}
+
 uint16_t random_u16(void)
 {
     uint16_t randnum;
diff --git a/toxcore/crypto_core.h b/toxcore/crypto_core.h
index 2c83fd25..e7e913b6 100644
--- a/toxcore/crypto_core.h
+++ b/toxcore/crypto_core.h
@@ -127,6 +127,11 @@ void crypto_sha512(uint8_t *hash, const uint8_t *data, size_t length);
 int32_t public_key_cmp(const uint8_t *pk1, const uint8_t *pk2);
 
 /**
+ * Return a random 8 bit integer.
+ */
+uint8_t random_u08(void);
+
+/**
  * Return a random 16 bit integer.
  */
 uint16_t random_u16(void);