pgsql: Fix misbehavior in contrib/pg_trgm with an unsatisfiable regex. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix misbehavior in contrib/pg_trgm with an unsatisfiable regex.
Date
Msg-id E1pb2p0-00330t-N7@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix misbehavior in contrib/pg_trgm with an unsatisfiable regex.

If the regex compiler can see that a regex is unsatisfiable
(for example, '$foo') then it may emit an NFA having no arcs.
pg_trgm's packGraph function did the wrong thing in this case;
it would access off the end of a work array, and with bad luck
could produce a corrupted output data structure causing more
problems later.  This could end with wrong answers or crashes
in queries using a pg_trgm GIN or GiST index with such a regex.

Fix by not trying to de-duplicate if there aren't at least 2 arcs.

Per bug #17830 from Alexander Lakhin.  Back-patch to all supported
branches.

Discussion: https://postgr.es/m/17830-57ff5f89bdb02b09@postgresql.org

Branch
------
REL_15_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/6170386c7fc1c1cfd7c2a655b8107872d18a0193

Modified Files
--------------
contrib/pg_trgm/expected/pg_word_trgm.out |  6 ++++++
contrib/pg_trgm/sql/pg_word_trgm.sql      |  3 +++
contrib/pg_trgm/trgm_regexp.c             | 26 ++++++++++++++++----------
3 files changed, 25 insertions(+), 10 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: pgsql: Add standard collation UNICODE
Next
From: Jeff Davis
Date:
Subject: Re: pgsql: Add standard collation UNICODE