Commitb1aed95

committed

Use PostmasterRandom(), not random(), for DSM control segment ID.

Otherwise, every startup gets the same "random" value, which isdefinitely not what was intended.

1 parentc124e36 commitb1aed95Copy full SHA for b1aed95

File tree

-3

lines changed

-3

lines changed

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -399,7 +399,6 @@ static void processCancelRequest(Port port, void pkt);`
`399`	`399`	`staticintinitMasks(fd_set*rmask);`
`400`	`400`	`staticvoidreport_fork_failure_to_client(Port*port,interrnum);`
`401`	`401`	`staticCAC_statecanAcceptConnections(void);`
`402`		`-staticlongPostmasterRandom(void);`
`403`	`402`	`staticvoidRandomSalt(char*md5Salt);`
`404`	`403`	`staticvoidsignal_child(pid_tpid,intsignal);`
`405`	`404`	`staticboolSignalSomeChildren(intsignal,inttargets);`
`@@ -5077,7 +5076,7 @@ RandomSalt(char *md5Salt)`
`5077`	`5076`	`/*`
`5078`	`5077`	`* PostmasterRandom`
`5079`	`5078`	`*/`
`5080`		`-staticlong`
	`5079`	`+long`
`5081`	`5080`	`PostmasterRandom(void)`
`5082`	`5081`	`{`
`5083`	`5082`	`/*`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@`
`36`	`36`
`37`	`37`	`#include"lib/ilist.h"`
`38`	`38`	`#include"miscadmin.h"`
	`39`	`+#include"postmaster/postmaster.h"`
`39`	`40`	`#include"storage/dsm.h"`
`40`	`41`	`#include"storage/ipc.h"`
`41`	`42`	`#include"storage/lwlock.h"`
`@@ -179,7 +180,7 @@ dsm_postmaster_startup(PGShmemHeader *shim)`
`179`	`180`	`{`
`180`	`181`	`Assert(dsm_control_address==NULL);`
`181`	`182`	`Assert(dsm_control_mapped_size==0);`
`182`		`-dsm_control_handle=random();`
	`183`	`+dsm_control_handle=(dsm_handle)PostmasterRandom();`
`183`	`184`	`if (dsm_control_handle==0)`
`184`	`185`	`continue;`
`185`	`186`	`if (dsm_impl_op(DSM_OP_CREATE,dsm_control_handle,segsize,`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@ extern const char *progname;`
`48`	`48`
`49`	`49`	`externvoidPostmasterMain(intargc,char*argv[])pg_attribute_noreturn();`
`50`	`50`	`externvoidClosePostmasterPorts(boolam_syslogger);`
	`51`	`+externlongPostmasterRandom(void);`
`51`	`52`
`52`	`53`	`externintMaxLivePostmasterChildren(void);`
`53`	`54`

Comments

(0)