[apparmor] [PATCH] utils: Handle unmount rules

Christian Boltz apparmor at cboltz.de
Tue Jun 3 17:19:28 UTC 2014


Hello,

Am Montag, 2. Juni 2014 schrieb Tyler Hicks:
> Bug: https://bugs.launchpad.net/bugs/1325109
> 
> The parser will accept rules with either umount or unmount rule types.
> The utils should follow suite.
> 
> Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> ---
>  utils/apparmor/aa.py             |  2 +-
>  utils/test/test-mount_parse.py   | 12 ++++++++++++
>  utils/test/test-regex_matches.py |  2 ++
>  3 files changed, 15 insertions(+), 1 deletion(-)
[...]
> --- a/utils/test/test-mount_parse.py
> +++ b/utils/test/test-mount_parse.py

> +    def test_parse_plain_unmount_rule(self):
> +        rule = 'unmount,'
> +        mount = aa.parse_mount_rule(rule)
> +        self.assertEqual(rule, mount.serialize(),
> +                'mount object returned "%s", expected "%s"' %
> (mount.serialize(), rule)) +
> +    def test_parse_unmount_with_mount_point(self):
> +        rule = 'unmount /mnt/external,'
> +        mount = aa.parse_mount_rule(rule)
> +        self.assertEqual(rule, mount.serialize(),
> +                'mount object returned "%s", expected "%s"' %

Doing this with an array and a loop is still on your TODO list, right? 
;-)


That said,
Acked-by: Christian Boltz <apparmor at cboltz.de>


Regards,

Christian Boltz
-- 
>Jetzt, wo du es sagst. Ich benutze aber immer cat. Auch wenn ich mit
>grep eine Datei durchsuchen will. Keine Ahnung warum.
>Ich glaube, dass ich das mal am Anfang irgendwo gelesen habe, und dann
>hat es sich ins Hirn eingebrannt.
*tsk* Ich glaub, wir muessen dich mal einer Gehirnwaesche unterziehen...
[> Marcus Habermehl und David Haller in suse-linux]




More information about the AppArmor mailing list