Add MtmNodeId range check.

kvap · kelvich · commit b3a0c79b11ec · 2017-11-13T02:17:30.000+03:00
diff --git a/multimaster.c b/multimaster.c
@@ -1869,7 +1869,10 @@ static void MtmSplitConnStrs(void)
 	}
 	if (i > MAX_NODES) { 
 		elog(ERROR, "Multimaster with more than %d nodes is not currently supported", MAX_NODES);
-	}	
+	}
+	if (MtmNodeId > i) {
+		elog(ERROR, "Multimaster node id %d is out of range [%d..%d]", MtmNodeId, 1, i);
+	}
 	if (i < 2) { 
         elog(ERROR, "Multimaster should have at least two nodes");
 	}	

Original file line number	Diff line number	Diff line change
`@@ -1869,7 +1869,10 @@ static void MtmSplitConnStrs(void)`
`1869`	`1869`	`}`
`1870`	`1870`	`if (i > MAX_NODES) {`
`1871`	`1871`	`elog(ERROR, "Multimaster with more than %d nodes is not currently supported", MAX_NODES);`
`1872`		`- }`
	`1872`	`+ }`
	`1873`	`+ if (MtmNodeId > i) {`
	`1874`	`+ elog(ERROR, "Multimaster node id %d is out of range [%d..%d]", MtmNodeId, 1, i);`
	`1875`	`+ }`
`1873`	`1876`	`if (i < 2) {`
`1874`	`1877`	`elog(ERROR, "Multimaster should have at least two nodes");`
`1875`	`1878`	`}`