diff options
author | 卜部昌平 <[email protected]> | 2023-04-13 15:36:24 +0900 |
---|---|---|
committer | 卜部昌平 <[email protected]> | 2023-04-13 19:50:00 +0900 |
commit | 64e503eb62aff0952b655e9a86217e355f786146 () | |
tree | 22d972813dad02a152e8320cb934b2250a7f41f1 /lib/securerandom.rb | |
parent | 0a092d00ccfa6bab2e6b09d118ce561ed0ea2908 (diff) |
avoid seeding
OpenSSL's man page previously stated that "the application is responsible for seeding the PRNG by calling RAND_add" (see [1]). So we had this code. However things changed. They no longer say so, instead "manual (re-)seeding of the default OpenSSL random generator is not necessary" now (see [2]). It seems all OpenSSL versions that we support now already behaves like this. Let's follow that. [1]: https://www.openssl.org/docs/man1.0.2/man3/RAND_add.html [2]: https://www.openssl.org/docs/manmaster/man3/RAND_add.html
Notes: Merged: https://.com/ruby/ruby/pull/7705
-rw-r--r-- | lib/securerandom.rb | 11 |
1 files changed, 0 insertions, 11 deletions
@@ -47,17 +47,6 @@ module SecureRandom private def gen_random_openssl(n) - @pid = 0 unless defined?(@pid) - pid = $$ - unless @pid == pid - now = Process.clock_gettime(Process::CLOCK_REALTIME, :nanosecond) - OpenSSL::Random.random_add([now, @pid, pid].join(""), 0.0) - seed = Random.urandom(16) - if (seed) - OpenSSL::Random.random_add(seed, 16) - end - @pid = pid - end return OpenSSL::Random.random_bytes(n) end |