Avoid using PostmasterRandom() for DSM control segment ID.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 23 Sep 2016 13:54:11 +0000 (09:54 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 23 Sep 2016 13:54:11 +0000 (09:54 -0400)
commitb251379fb44b4893624999f1ba00986c029f11cf
tree15f1c0f1a6383add79fe76487a9c75b5b8402bb3
parenta88fe25f50ae9ce491fe10a90aeef2cbed6f2b9a
Avoid using PostmasterRandom() for DSM control segment ID.

Commits 470d886c3 et al intended to fix the problem that the postmaster
selected the same "random" DSM control segment ID on every start.  But
using PostmasterRandom() for that destroys the intended property that the
delay between random_start_time and random_stop_time will be unpredictable.
(Said delay is probably already more predictable than we could wish, but
that doesn't mean that reducing it by a couple orders of magnitude is OK.)
Revert the previous patch and add a comment warning against misuse of
PostmasterRandom.  Fix the original problem by calling srandom() early in
PostmasterMain, using a low-security seed that will later be overwritten
by PostmasterRandom.

Discussion: <20789.1474390434@sss.pgh.pa.us>
src/backend/postmaster/postmaster.c
src/backend/storage/ipc/dsm.c
src/include/postmaster/postmaster.h