[apparmor] [patch 08/12] parser: add test case for empty character class regex

Christian Boltz apparmor at cboltz.de
Tue Dec 3 23:27:23 UTC 2013


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 <steve at 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 <apparmor at 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]




More information about the AppArmor mailing list