On Fri, Aug 15, 2014 at 12:20:47PM -0700, [email protected] wrote:
> Signed-off-by: John Johansen <[email protected]>

Acked-by: Steve Beattie <[email protected]>

> +     /*
> +      * Check if we have an expression with or without wildcards. This
> +      * determines how exec modifiers are merged in accept_perms() based
> +      * on how we split permission bitmasks here.
> +      */
> +     exact_match = 1;
> +     for (depth_first_traversal i(tree); i && exact_match; i++) {
> +             if (dynamic_cast<StarNode *>(*i) ||
> +                 dynamic_cast<PlusNode *>(*i) ||
> +                 dynamic_cast<AnyCharNode *>(*i) ||
> +                 dynamic_cast<CharSetNode *>(*i) ||
> +                 dynamic_cast<NotCharSetNode *>(*i))
> +                     exact_match = 0;
> +     }

One of these days I'll get around to pushing an is_exact_match()
method into each of the Node classes so that all the dynamic_casts
don't need to occur while walking the tree. Or maybe someone else will,
which would be great, too.

-- 
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