mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-08-23 02:40:41 +02:00
Guard against rare RAND_bytes() failures in pg_strong_random().
When built using OpenSSL, pg_strong_random() uses RAND_bytes() to generate the random number. On very rare occasions that can fail, if its PRNG has not been seeded with enough data. Additionally, once it does fail, all subsequent calls will also fail until more seed data is added. Since this is required during backend startup, this can result in all new backends failing to start until a postmaster restart. Guard against that by checking the state of OpenSSL's PRNG using RAND_status(), and if necessary (very rarely), seeding it using RAND_poll(). Back-patch to v10, where pg_strong_random() was introduced. Dean Rasheed and Michael Paquier. Discussion: https://postgr.es/m/CAEZATCXMtxbzSAvyKKk5uCRf9pNt4UV%2BF_5v%3DgLfJUuPxU4Ytg%40mail.gmail.com
This commit is contained in:
parent
608793266d
commit
1f919e663e
@ -103,6 +103,35 @@ pg_strong_random(void *buf, size_t len)
|
||||
* When built with OpenSSL, use OpenSSL's RAND_bytes function.
|
||||
*/
|
||||
#if defined(USE_OPENSSL_RANDOM)
|
||||
int i;
|
||||
|
||||
/*
|
||||
* Check that OpenSSL's CSPRNG has been sufficiently seeded, and if not
|
||||
* add more seed data using RAND_poll(). With some older versions of
|
||||
* OpenSSL, it may be necessary to call RAND_poll() a number of times.
|
||||
*/
|
||||
#define NUM_RAND_POLL_RETRIES 8
|
||||
|
||||
for (i = 0; i < NUM_RAND_POLL_RETRIES; i++)
|
||||
{
|
||||
if (RAND_status() == 1)
|
||||
{
|
||||
/* The CSPRNG is sufficiently seeded */
|
||||
break;
|
||||
}
|
||||
|
||||
if (RAND_poll() == 0)
|
||||
{
|
||||
/*
|
||||
* RAND_poll() failed to generate any seed data, which means that
|
||||
* RAND_bytes() will probably fail. For now, just fall through
|
||||
* and let that happen. XXX: maybe we could seed it some other
|
||||
* way.
|
||||
*/
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
if (RAND_bytes(buf, len) == 1)
|
||||
return true;
|
||||
return false;
|
||||
|
Loading…
Reference in New Issue
Block a user