Don't release index root page pin in ginFindParents().

tglsfdc · tglsfdc · commit f76b975d5887 · 2023-11-13T11:44:48.000-05:00
It's clearly stated in the comments that ginFindParents() must keep the pin on the index's root page that's associated with the topmost GinBtreeStack item. However, the code path for the case that the desired downlink has been pushed down to the next index level ignored this proviso, and would release the pin anyway if we were still examining the root level. That led to an assertion failure or "buffer NNNN is not owned by resource owner" error later, when we try to release the pin again at the end of the insertion. This is quite hard to reproduce, since it can only happen if an index root page split occurs concurrently with our own insertion. Thanks to Jeff Janes for finding a test case that triggers it often enough to allow investigation. This has been there since the beginning of GIN, so back-patch to all supported branches. Discussion: https://postgr.es/m/CAMkU=1yCAKtv86dMrD__Ja-7KzjE=uMeKX8y__cx5W-OEWy2ow@mail.gmail.com
diff --git a/src/backend/access/gin/ginbtree.c b/src/backend/access/gin/ginbtree.c
@@ -274,7 +274,11 @@ ginFindParents(GinBtree btree, GinBtreeStack *stack)
 			blkno = GinPageGetOpaque(page)->rightlink;
 			if (blkno == InvalidBlockNumber)
 			{
-				UnlockReleaseBuffer(buffer);
+				/* Link not present in this level */
+				LockBuffer(buffer, GIN_UNLOCK);
+				/* Do not release pin on the root buffer */
+				if (buffer != root->buffer)
+					ReleaseBuffer(buffer);
 				break;
 			}
 			buffer = ginStepRight(buffer, btree->index, GIN_EXCLUSIVE);

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,11 @@ ginFindParents(GinBtree btree, GinBtreeStack *stack)`
`274`	`274`	`blkno = GinPageGetOpaque(page)->rightlink;`
`275`	`275`	`if (blkno == InvalidBlockNumber)`
`276`	`276`	`{`
`277`		`- UnlockReleaseBuffer(buffer);`
	`277`	`+ /* Link not present in this level */`
	`278`	`+ LockBuffer(buffer, GIN_UNLOCK);`
	`279`	`+ /* Do not release pin on the root buffer */`
	`280`	`+ if (buffer != root->buffer)`
	`281`	`+ ReleaseBuffer(buffer);`
`278`	`282`	`break;`
`279`	`283`	`}`
`280`	`284`	`buffer = ginStepRight(buffer, btree->index, GIN_EXCLUSIVE);`