On Thu, Feb 09, 2017 at 10:18:01AM -0600, Tyler Hicks wrote:
> > This appears to suffer a similar problem:
> > 
> >> +        # Check __AA_PARSER, which may be set by the Makefile, to see if
> >> +        # we should use a non-default apparmor_parser path to verify
> >> +        # policy
> >> +        parser = os.getenv('__AA_PARSER')
> >> +        if parser:
> >> +            self.full_args.append('--parser=%s' % parser)
> >> +
> 
> I feel like this one behaves as intended. If __AA_PARSER is an empty
> string, the "if parser:" conditional is tests out to be false and
> self.full_args remains unchanged.

Ah, you're right. I should have tested first. :)

Acked-by: Seth Arnold <seth.arn...@canonical.com>

Thanks

Attachment: signature.asc
Description: PGP signature

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

Reply via email to