Fix use of free in walsender error handling after a sysid mismatch.

hlinnaka · hlinnaka · commit 3a8e9e977fa6 · 2014-05-06T15:17:41.000+03:00
Found via valgrind. The bug exists since the introduction of the walsender,
so backpatch to 9.0.

Andres Freund
diff --git a/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c b/src/backend/replication/libpqwalreceiver/libpqwalreceiver.c
@@ -152,6 +152,7 @@ libpqrcv_identify_system(TimeLineID *primary_tli)
 			 GetSystemIdentifier());
 	if (strcmp(primary_sysid, standby_sysid) != 0)
 	{
+		primary_sysid = pstrdup(primary_sysid);
 		PQclear(res);
 		ereport(ERROR,
 				(errmsg("database system identifier differs between the primary and standby"),

Original file line number	Diff line number	Diff line change
`@@ -152,6 +152,7 @@ libpqrcv_identify_system(TimeLineID *primary_tli)`
`152`	`152`	`GetSystemIdentifier());`
`153`	`153`	`if (strcmp(primary_sysid, standby_sysid) != 0)`
`154`	`154`	`{`
	`155`	`+ primary_sysid = pstrdup(primary_sysid);`
`155`	`156`	`PQclear(res);`
`156`	`157`	`ereport(ERROR,`
`157`	`158`	`(errmsg("database system identifier differs between the primary and standby"),`