Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 329cacb

Browse files
committed
Initialize random() in bootstrap/stand-alone postgres and in initdb.
This removes a difference between the standard IsUnderPostmaster execution environment and that of --boot and --single. In a stand-alone backend, "SELECT random()" always started at the same seed. On a system capable of using posix shared memory, initdb could still conclude "selecting dynamic shared memory implementation ... sysv". Crashed --boot or --single postgres processes orphaned shared memory objects having names that collided with the not-actually-random names that initdb probed. The sysv fallback appeared after ten crashes of --boot or --single postgres. Since --boot and --single are rare in production use, systems used for PostgreSQL development are the principal candidate to notice this symptom. Back-patch to 9.3 (all supported versions). PostgreSQL 9.4 introduced dynamic shared memory, but 9.3 does share the "SELECT random()" problem. Reviewed by Tom Lane and Kyotaro HORIGUCHI. Discussion: https://postgr.es/m/20180915221546.GA3159382@rfd.leadboat.com
1 parent 77d2a48 commit 329cacb

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

src/backend/utils/init/miscinit.c

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,14 @@ InitStandaloneProcess(const char *argv0)
223223

224224
MyStartTime = time(NULL); /* set our start time in case we call elog */
225225

226+
/*
227+
* Initialize random() for the first time, like PostmasterMain() would.
228+
* In a regular IsUnderPostmaster backend, BackendRun() computes a
229+
* high-entropy seed before any user query. Fewer distinct initial seeds
230+
* can occur here.
231+
*/
232+
srandom((unsigned int) (MyProcPid ^ MyStartTime));
233+
226234
/* Initialize process-local latch support */
227235
InitializeLatchSupport();
228236
MyLatch = &LocalLatchData;

src/bin/initdb/initdb.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1058,6 +1058,9 @@ choose_dsm_implementation(void)
10581058
#ifdef HAVE_SHM_OPEN
10591059
int ntries = 10;
10601060

1061+
/* Initialize random(); this function is its only user in this program. */
1062+
srandom((unsigned int) (getpid() ^ time(NULL)));
1063+
10611064
while (ntries > 0)
10621065
{
10631066
uint32 handle;

0 commit comments

Comments
 (0)