]> git.ipfire.org Git - thirdparty/postgresql.git/commit
pg_trgm: fix crash in 2-item picksplit
authorAndrew Gierth <rhodiumtoad@postgresql.org>
Thu, 12 Nov 2020 14:34:37 +0000 (14:34 +0000)
committerAndrew Gierth <rhodiumtoad@postgresql.org>
Thu, 12 Nov 2020 14:57:13 +0000 (14:57 +0000)
commit245a35f96e3a12b7fe19bad88671d75dcbec133c
tree07f84747a686b30e196448852a57139cf1dcba7b
parentcd39c23a21ac46009d21a51db7438bba9d5d1941
pg_trgm: fix crash in 2-item picksplit

Whether from size overflow in gistSplit or from secondary splits,
picksplit is (rarely) called with exactly two items to split.

Formerly, due to special-case handling of the last item, this would
lead to access to an uninitialized cache entry; prior to PG 13 this
might have been harmless or at worst led to an incorrect union datum,
but in 13 onwards it can cause a backend crash from using an
uninitialized pointer.

Repair by removing the special case, which was deemed not to have been
appropriate anyway. Backpatch all the way, because this bug has
existed since pg_trgm was added.

Per report on IRC from user "ftzdomino". Analysis and testing by me,
patch from Alexander Korotkov.

Discussion: https://postgr.es/m/87k0usfdxg.fsf@news-spur.riddles.org.uk
contrib/pg_trgm/trgm_gist.c