Fix order of arguments to SubTransSetParent().

tglsfdc · tglsfdc · commit a66e01bbcc88 · 2017-04-23T13:10:57.000-04:00
ProcessTwoPhaseBuffer (formerly StandbyRecoverPreparedTransactions) mixed up the parent and child XIDs when calling SubTransSetParent to record the transactions' relationship in pg_subtrans. Remarkably, analysis by Simon Riggs suggests that this doesn't lead to visible problems (at least, not in non-Assert builds). That might explain why we'd not noticed it before. Nonetheless, it's surely wrong. This code was born broken, so back-patch to all supported branches. Discussion: https://postgr.es/m/20110.1492905318@sss.pgh.pa.us
diff --git a/src/backend/access/transam/twophase.c b/src/backend/access/transam/twophase.c
@@ -1934,7 +1934,7 @@ StandbyRecoverPreparedTransactions(bool overwriteOK)
 				TransactionId subxid = subxids[i];
 
 				Assert(TransactionIdFollows(subxid, xid));
-				SubTransSetParent(xid, subxid, overwriteOK);
+				SubTransSetParent(subxid, xid, overwriteOK);
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1934,7 +1934,7 @@ StandbyRecoverPreparedTransactions(bool overwriteOK)`
`1934`	`1934`	`TransactionId subxid = subxids[i];`
`1935`	`1935`
`1936`	`1936`	`Assert(TransactionIdFollows(subxid, xid));`
`1937`		`- SubTransSetParent(xid, subxid, overwriteOK);`
	`1937`	`+ SubTransSetParent(subxid, xid, overwriteOK);`
`1938`	`1938`	`}`
`1939`	`1939`	`}`
`1940`	`1940`	`}`