Regex fixes (latest CVS this time)

Jakub Jelinek jakub@redhat.com
Sat Sep 20 20:54:00 GMT 2003


On Sat, Sep 20, 2003 at 04:02:50PM +0200, Paolo Bonzini wrote:
> I downloaded the latest CVS; it fixes one of the XFAILs in sed; five remain,
> I'll send details soon.
> 
> I found a possible out of bounds left shift.  The first of these two patches
> fixes it, together with a bug in which \(^ was wrongly parsed as an anchor.
> Doing lookbehind in this case is wrong, instead it is ok for treating ^ as
> special if coming after a new-line character, and lookahead is ok for
> checking if $ is special.  You should pass state down from the parser: I
> used the syntax bits for this, it may not be nice but it is simple and does
> not waste an entire parameter.

Can you please also include tests for the bugs for glibc testsuite?
(if they can be added just as another subtest for bug-regex{11,12}.c, then
that way, otherwise new tests).

	Jakub



More information about the Libc-alpha mailing list