aboutsummaryrefslogtreecommitdiffstats
path: root/src/lib/rng/rng.h
diff options
context:
space:
mode:
authorJack Lloyd <[email protected]>2016-01-01 10:14:42 -0500
committerJack Lloyd <[email protected]>2016-01-01 10:14:42 -0500
commit7583a2a882f4c9d71d947db853c266394f3ca760 (patch)
treeda42920e915cd94249de1bf93810df0bb2a9d1f9 /src/lib/rng/rng.h
parent948eb5448c9ffbd9217780b746f5cbfc8b5b11e4 (diff)
Remove RNG::gen_mask, which had undefined behavior when bits >= 32
Bug found by Daniel Neus The function wasn't being used anywhere in the library (and was only added in 1.11.20) so it seems easier to remove than fix. And removing it serves to put any user on notice that something bad happened; Daniel tested this as returning just 0 when bits >= 32 with his system's compiler.
Diffstat (limited to 'src/lib/rng/rng.h')
-rw-r--r--src/lib/rng/rng.h12
1 files changed, 0 insertions, 12 deletions
diff --git a/src/lib/rng/rng.h b/src/lib/rng/rng.h
index 23e974b92..a01212359 100644
--- a/src/lib/rng/rng.h
+++ b/src/lib/rng/rng.h
@@ -61,18 +61,6 @@ class BOTAN_DLL RandomNumberGenerator
}
/**
- * Return a value in range [0,2^bits)
- */
- u64bit gen_mask(size_t bits)
- {
- if(bits == 0 || bits > 64)
- throw Invalid_Argument("RandomNumberGenerator::gen_mask invalid argument");
-
- const u64bit mask = ((1 << bits) - 1);
- return this->get_random<u64bit>() & mask;
- }
-
- /**
* Return a random byte
* @return random byte
*/