On Wed, Nov 06, 2013 at 04:38:09PM -0800, John Johansen wrote:
> On 11/06/2013 04:19 PM, Steve Beattie wrote:
> > On Mon, Nov 04, 2013 at 05:42:29PM -0800, Seth Arnold wrote:
> >> I suspect there is more work to be done in this block of code; '[' may
> >> need a corresponding if (incharclass == 1) test, unless this is supposed
> >> to work: [[]  /* character class with a [ inside the class */
> > 
> err that is perfectly valid pcre, [ in a character class is only special if
> its part of a posix character class.
> 
> http://perldoc.perl.org/perlrecharclass.html#Bracketed-Character-Classes
> Special Characters Inside a Bracketed Character Class subsection of
> Bracketed Character Classes
> 
> and ] has the exception that it isn't special if its the first character
> within the character set to match (where a leading ^ isn't the first character
> to match because it indicates inverting the class
>   ie.
>    []]
>    [^]]
> are valid

Okay, patch withdrawn for consideration. How much of pcre's character
classes are you hoping to support?  Because there's a bunch of it
that we don't (POSIX character classes, etc.)...

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to