Thread: pgsql: Fix locking when fixing an incomplete split of a GIN internal pa
pgsql: Fix locking when fixing an incomplete split of a GIN internal pa
From
Heikki Linnakangas
Date:
Fix locking when fixing an incomplete split of a GIN internal page ginFinishSplit() expects the caller to hold an exclusive lock on the buffer, but when finishing an earlier "leftover" incomplete split of an internal page, the caller held a shared lock. That caused an assertion failure in MarkBufferDirty(). Without assertions, it could lead to corruption if two backends tried to complete the split at the same time. On master, add a test case using the new injection point facility. Report and analysis by Fei Changhong. Backpatch the fix to all supported versions. Reviewed-by: Fei Changhong, Michael Paquier Discussion: https://www.postgresql.org/message-id/tencent_A3CE810F59132D8E230475A5F0F7A08C8307@qq.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/6a1ea02c491d16474a6214603dce40b5b122d4d1 Modified Files -------------- src/backend/access/gin/ginbtree.c | 77 ++++++--- src/test/modules/Makefile | 4 +- src/test/modules/gin/Makefile | 16 ++ .../modules/gin/expected/gin_incomplete_splits.out | 180 +++++++++++++++++++++ src/test/modules/gin/meson.build | 16 ++ src/test/modules/gin/sql/gin_incomplete_splits.sql | 144 +++++++++++++++++ src/test/modules/meson.build | 1 + 7 files changed, 418 insertions(+), 20 deletions(-)