aboutsummaryrefslogtreecommitdiff
path: root/src/random.h
diff options
context:
space:
mode:
authorWladimir J. van der Laan <laanwj@gmail.com>2017-02-21 17:36:37 +0100
committerWladimir J. van der Laan <laanwj@gmail.com>2017-02-21 20:57:34 +0100
commit224e6eb089a0f4977d22f3803fc27e44b5e7eea5 (patch)
tree4bbac190c5858510347bc40f825d514a594c1d35 /src/random.h
parent5f0556d0326bf6f4d34f7e8b9ada7bbb09cb1df2 (diff)
downloadbitcoin-224e6eb089a0f4977d22f3803fc27e44b5e7eea5.tar.xz
util: Specific GetOSRandom for Linux/FreeBSD/OpenBSD
These are available in sandboxes without access to files or devices. Also [they are safer and more straightforward](https://en.wikipedia.org/wiki/Entropy-supplying_system_calls) to use than `/dev/urandom` as reading from a file has quite a few edge cases: - Linux: `getrandom(buf, buflen, 0)`. [getrandom(2)](http://man7.org/linux/man-pages/man2/getrandom.2.html) was introduced in version 3.17 of the Linux kernel. - OpenBSD: `getentropy(buf, buflen)`. The [getentropy(2)](http://man.openbsd.org/cgi-bin/man.cgi/OpenBSD-current/man2/getentropy.2) function appeared in OpenBSD 5.6. - FreeBSD and NetBSD: `sysctl(KERN_ARND)`. Not sure when this was added but it has existed for quite a while. Alternatives: - Linux has sysctl `CTL_KERN` / `KERN_RANDOM` / `RANDOM_UUID` which gives 16 bytes of randomness. This may be available on older kernels, however [sysctl is deprecated on Linux](https://lwn.net/Articles/605392/) and even removed in some distros so we shouldn't use it. Add tests for `GetOSRand()`: - Test that no error happens (otherwise `RandFailure()` which aborts) - Test that all 32 bytes are overwritten (initialize with zeros, try multiple times) Discussion: - When to use these? Currently they are always used when available. Another option would be to use them only when `/dev/urandom` is not available. But this would mean these code paths receive less testing, and I'm not sure there is any reason to prefer `/dev/urandom`. Closes: #9676
Diffstat (limited to 'src/random.h')
-rw-r--r--src/random.h8
1 files changed, 8 insertions, 0 deletions
diff --git a/src/random.h b/src/random.h
index 664f030eba..0e8bb27176 100644
--- a/src/random.h
+++ b/src/random.h
@@ -46,4 +46,12 @@ public:
uint32_t Rw;
};
+/* Number of random bytes returned by GetOSRand */
+static const ssize_t NUM_OS_RANDOM_BYTES = 32;
+
+/** Get 32 bytes of system entropy. Do not use this in application code: use
+ * GetStrongRandBytes instead.
+ */
+void GetOSRand(unsigned char *ent32);
+
#endif // BITCOIN_RANDOM_H