On Sun, Aug 14, 2016 at 10:02:45PM +0200, Christian Boltz wrote:
> This turned out to be a simple case of misinterpreting the promptUser()
> result - it returns the answer and the selected option, and
> "surprisingly" something like
>     ('CMD_ADDHAT', 0)
> never matched
>     'CMD_ADDHAT'
> ;-)
> 
> I also noticed that the new hat doesn't get initialized as
> profile_storage(), and that the changed profile doesn't get marked as
> changed. This is also fixed by this patch.
> 
> 
> References: https://bugs.launchpad.net/apparmor/+bug/1538306
> 
> 
> I propose this patch for trunk, 2.10 and 2.9.
> Note that 2.9 doesn't have profile_storage(), therefore I won't add
> that line there.

Acked-by: Steve Beattie <[email protected]> for all three branches. Thanks.

> [ fix-add-hat.diff ]

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: PGP signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to