From: | tgl(at)postgresql(dot)org (Tom Lane) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Fix similar_escape() so that SIMILAR TO works properly for |
Date: | 2006-04-13 18:01:45 |
Message-ID: | 20060413180145.59A3311F6613@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Log Message:
-----------
Fix similar_escape() so that SIMILAR TO works properly for patterns involving
alternatives ("|" symbol). The original coding allowed the added ^ and $
constraints to be absorbed into the first and last alternatives, producing
a pattern that would match more than it should. Per report from Eric Noriega.
I also changed the pattern to add an ARE director ("***:"), ensuring that
SIMILAR TO patterns do not change behavior if regex_flavor is changed. This
is necessary to make the non-capturing parentheses work, and seems like a
good idea on general principles.
Back-patched as far as 7.4. 7.3 also has the bug, but a fix seems impractical
because that version's regex engine doesn't have non-capturing parens.
Tags:
----
REL8_0_STABLE
Modified Files:
--------------
pgsql/src/backend/utils/adt:
regexp.c (r1.56 -> r1.56.4.1)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/regexp.c.diff?r1=1.56&r2=1.56.4.1)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2006-04-13 18:01:53 | pgsql: Fix similar_escape() so that SIMILAR TO works properly for |
Previous Message | Tom Lane | 2006-04-13 18:01:38 | pgsql: Fix similar_escape() so that SIMILAR TO works properly for |