fix bug in MtmNoticeReceiver

kelvich · kelvich · commit 10385424184b · 2017-06-28T11:49:29.000+03:00
diff --git a/contrib/mmts/multimaster.c b/contrib/mmts/multimaster.c
@@ -4582,7 +4582,7 @@ MtmNoticeReceiver(void *i, const PGresult *res)
 	if ( (*(int *)i) != MtmNodeId - 1)
 		return;
 
-	stripped_notice = palloc0(len);
+	stripped_notice = palloc0(len + 1);
 
 	if (*notice == 'N')
 	{
@@ -4598,7 +4598,7 @@ MtmNoticeReceiver(void *i, const PGresult *res)
 	}
 	else
 	{
-		stripped_notice = notice;
+		strncpy(stripped_notice, notice, len + 1);
 		MTM_ELOG(WARNING, "%s", stripped_notice);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -4582,7 +4582,7 @@ MtmNoticeReceiver(void i, const PGresult res)`
`4582`	`4582`	`if ( ((int )i) != MtmNodeId - 1)`
`4583`	`4583`	`return;`
`4584`	`4584`
`4585`		`- stripped_notice = palloc0(len);`
	`4585`	`+ stripped_notice = palloc0(len + 1);`
`4586`	`4586`
`4587`	`4587`	`if (*notice == 'N')`
`4588`	`4588`	`{`
`@@ -4598,7 +4598,7 @@ MtmNoticeReceiver(void i, const PGresult res)`
`4598`	`4598`	`}`
`4599`	`4599`	`else`
`4600`	`4600`	`{`
`4601`		`- stripped_notice = notice;`
	`4601`	`+ strncpy(stripped_notice, notice, len + 1);`
`4602`	`4602`	`MTM_ELOG(WARNING, "%s", stripped_notice);`
`4603`	`4603`	`}`
`4604`	`4604`