pgsql: Fix enforcement of restrictions inside regexp lookaround constra - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix enforcement of restrictions inside regexp lookaround constra
Date
Msg-id E1Zv7Wn-0000a5-Jq@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix enforcement of restrictions inside regexp lookaround constraints.

Lookahead and lookbehind constraints aren't allowed to contain backrefs,
and parentheses within them are always considered non-capturing.  Or so
says the manual.  But the regexp parser forgot about these rules once
inside a parenthesized subexpression, so that constructs like (\w)(?=(\1))
were accepted (but then not correctly executed --- a case like this acted
like (\w)(?=\w), without any enforcement that the two \w's match the same
text).  And in (?=((foo))) the innermost parentheses would be counted as
capturing parentheses, though no text would ever be captured for them.

To fix, properly pass down the "type" argument to the recursive invocation
of parse().

Back-patch to all supported branches; it was agreed that silent
misexecution of such patterns is worse than throwing an error, even though
new errors in minor releases are generally not desirable.

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/03ee6591dd305613a22902fb777a192fe1970adc

Modified Files
--------------
src/backend/regex/regcomp.c |    2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: pgsql: Try to convince gcc that TupleQueueRemap never falls off the end
Next
From: Tom Lane
Date:
Subject: pgsql: Fix enforcement of restrictions inside regexp lookaround constra