Fix incorrect initialization of BackendActivityBuffer.

tglsfdc · tglsfdc · commit 753051cc721e · 2018-08-07T16:01:14.000-04:00
Since commit c8e8b5a, this has been zeroed out using the wrong length. In practice the length would always be too small, leading to not zeroing the whole buffer rather than clobbering additional memory; and that's pretty harmless, both because shmem would likely start out as zeroes and because we'd reinitialize any given entry before use. Still, it's bogus, so fix it. Reported by Petru-Florin Mihancea (bug #15312) Discussion: https://postgr.es/m/153363913073.1303.6518849192351268091@wrigleys.postgresql.org
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
@@ -2516,7 +2516,7 @@ CreateSharedBackendStatus(void)
 
 	if (!found)
 	{
-		MemSet(BackendActivityBuffer, 0, size);
+		MemSet(BackendActivityBuffer, 0, BackendActivityBufferSize);
 
 		/* Initialize st_activity pointers. */
 		buffer = BackendActivityBuffer;

Original file line number	Diff line number	Diff line change
`@@ -2516,7 +2516,7 @@ CreateSharedBackendStatus(void)`
`2516`	`2516`
`2517`	`2517`	`if (!found)`
`2518`	`2518`	`{`
`2519`		`- MemSet(BackendActivityBuffer, 0, size);`
	`2519`	`+ MemSet(BackendActivityBuffer, 0, BackendActivityBufferSize);`
`2520`	`2520`
`2521`	`2521`	`/* Initialize st_activity pointers. */`
`2522`	`2522`	`buffer = BackendActivityBuffer;`