Mailing List Archive

[Bug 405] Logic error in pcre.c and pcre_compile.c
http://www.exim.org/bugzilla/show_bug.cgi?id=405


ph10@hermes.cam.ac.uk changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED




------- Comment #1 from ph10@hermes.cam.ac.uk 2006-11-07 16:35 -------
(In reply to comment #0)

> recently I came across the following bit of code in the source file
> exim4-4.63/src/pcre/pcre.c (function compile_branch, around line
> 3355):
>
> /* Condition to test for a numbered subpattern match. We know that
> if a digit follows ( then there will just be digits until ) because
> the syntax was checked in the first pass. */
>
> else if ((digitab[ptr[1]] && ctype_digit) != 0)

That's a nasty typo in PCRE which was noticed and fixed for PCRE release 6.7.
Exim is still using PCRE 6.2. I have imported 6.7 into the Exim source tree so
all should be well in the next Exim release.

(Long term: Exim should stop including its own PCRE library now that PCRE is so
widespread.)


--
Configure bugmail: http://www.exim.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##