[apparmor] [patch] [40/38] Load all includes in aa-mergeprof ask_the_questions()

Steve Beattie steve at nxnw.org
Tue Sep 27 21:32:48 UTC 2016


On Tue, Sep 27, 2016 at 11:17:05PM +0200, Christian Boltz wrote:
> aa-mergeprof empties 'includes' when running reset_aa(). The result is
>     KeyError: 'abstractions/newly_added_abstraction'
> if an include file gets added because it isn't part of 'includes' at
> this time. Note that you'll need to add another rule after adding the
> include to trigger checking the includes for superfluous rules.
> 
> This fixes the regression found by Steve - which isn't really a
> regression, "just" one more thing that got more visible with the new
> code. Before, it was just an ill-addressed hasher that didn't complain ;-)
> 
> [ 40-mergeprof-load-includes.diff ]

Acked-by: Steve Beattie <steve at nxnw.org>. Thanks for addressing it
quickly!

-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20160927/c27bbc16/attachment.pgp>


More information about the AppArmor mailing list