pgsql: Repair bug in regexp split performance improvements. - Mailing list pgsql-committers

From Andrew Gierth
Subject pgsql: Repair bug in regexp split performance improvements.
Date
Msg-id E1g0AJ0-0001AW-4H@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Repair bug in regexp split performance improvements.

Commit c8ea87e4b introduced a temporary conversion buffer for
substrings extracted during regexp splits. Unfortunately the code that
sized it was failing to ignore the effects of ignored degenerate
regexp matches, so for regexp_split_* calls it could under-size the
buffer in such cases.

Fix, and add some regression test cases (though those will only catch
the bug if run in a multibyte encoding).

Backpatch to 9.3 as the faulty code was.

Thanks to the PostGIS project, Regina Obe and Paul Ramsey for the
report (via IRC) and assistance in analysis. Patch by me.

Branch
------
REL_10_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/ab78c6e36635da96c5ef8c229942a16b5391c0b9

Modified Files
--------------
src/backend/utils/adt/regexp.c        | 16 ++++++++++------
src/test/regress/expected/strings.out | 18 ++++++++++++++++++
src/test/regress/sql/strings.sql      |  3 +++
3 files changed, 31 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: pgsql: On all Windows platforms, not just Cygwin,use _timezone and _tz
Next
From: Tom Lane
Date:
Subject: pgsql: Minor fixes for psql tab completion.