Hello,

Am Dienstag, 3. Dezember 2013 schrieb Steve Beattie:
> This patch adds a test that verifies the parser considers an emty
> character class regex as a parse arror.
> 
> Signed-off-by: Steve Beattie <st...@nxnw.org>
> ---
>  parser/tst/simple_tests/file/bad_re_brace_1.sd |    8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> Index: b/parser/tst/simple_tests/file/bad_re_brace_1.sd
> ===================================================================
> --- /dev/null
> +++ b/parser/tst/simple_tests/file/bad_re_brace_1.sd
> @@ -0,0 +1,8 @@
> +#
> +#=DESCRIPTION regex with empty character class (brace)
> +#=EXRESULT FAIL
> +#
> +/usr/bin/foo {
> +  /alpha/[]beta rw,
> +}
> +

Good idea!

Acked-By: Christian Boltz <appar...@cboltz.de>

BTW: Do we already have a similar test for empty alternations, like
    /foo{}/bar rw,
?


Regards,

Christian Boltz
-- 
> The kernel will stay the same between SUSE Linux 10.1 and SLE10 -
> it just might be that we release them at different days,
Good. Let the SLED customers test it for us first ;)
[> Andreas Jaeger and Martin Schlander in opensuse]


-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to