[apparmor] [patch] [17/38] Implement globbing in FileRule

Christian Boltz apparmor at cboltz.de
Thu Sep 22 19:47:01 UTC 2016


Hello,

Am Donnerstag, 22. September 2016, 12:37:22 CEST schrieb Steve Beattie:
> On Fri, Aug 12, 2016 at 10:57:08PM +0200, Christian Boltz wrote:
> > this patch adds the glob() and glob_ext() functions to FileRule, and
> > sets self.can_glob and self.can_glob_ext. Also add some tests (just
> > enough to make sure the FileRule integration works - the globbing
> > is handled inside AARE,and the AARE tests contain more testcases).
> 
> Can you add a comment to this effect in the FileGlobTest class, that
> it's purpose is to ensure the FileRule integration works, and a
> pointer to the more comprehensive regex tests in test-aare.py?

Good idea, I'll add that comment before commiting.

> > Note that the implementation differs from the original plan (which
> > was to have globbing in *Ruleset). Therefore add can_glob and
> > can_glob_ext to BaseRule (both default to False), and add a comment
> > to BaseRuleset that globbing needs to be removed from all *Ruleset
> > classes.
> Okay.
> 
> > [ 17-FileRule-implement-globbing.diff ]
> 
> Otherwise, looks fine. Acked-by: Steve Beattie <steve at nxnw.org>.
> Thanks!
Regards,

Christian Boltz
-- 
> copy+paste zusammen gesetzt. Naja, er hat Stein und Bein geschworen
> das schon öfter so gemacht zu haben, erfolgreich versteht sich, drum
Logisch. Selbstverständlich.
Und Du bist der ALLEROBEREINZIGSTE Mailserver auf der ganzen Welt wo es
nicht geht -- und das ist ja nun wirklich unverschämt von Dir.
[Robert Stark und Peer Heinlein in postfixbuch-users]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20160922/f3ac4374/attachment.pgp>


More information about the AppArmor mailing list