[apparmor] [patch] Reset aa and original_aa in read_profiles()

Christian Boltz apparmor at cboltz.de
Sat Oct 3 21:14:30 UTC 2015


Hallo Leute,

Am Sonntag, 13. September 2015 schrieb Christian Boltz:
> TL;DR: aa-genprof crashes with a wrong 'Conflicting profiles' error.
> 
> aa-genprof uses autodep() to create a basic profile, which is then
> stored in aa and original_aa. After that, read_profiles() is called,
> which reads all profiles (including the new one) from disk, causing a
> (wrong) 'Conflicting profiles' error in attach_profile_data() because
> the autodep()-generated profile is already there.
> 
> Therefore this patch resets aa and original_aa in read_profiles() to
> avoid that problem.
> 
> The conflict check is in trunk only, therefore I don't see a need to
> apply this patch to 2.9.
> 
> 
> 
> [ 87-reset-aa-in-read_profiles.diff ]

Any comments or reviews on this patch?

If nobody objects, I'll commit it on Wednesday as Acked-by <timeout>.


Regards,

Christian Boltz
-- 
Bist Du etwa auch dafür so wie die Araber aus allen Wörtern die
Vokale wegzulassen? Das schreibt sich sicherlich extrem effizient.
dr bst D d ndrr nscht? Knnst D nch vrsthn, ws ch schrb? Mn sllt lls
br dn Hfn wrfn nd sch wdr w d Stnztmnschn ntrhltn: AAArgh Ugh
*Keule schwing* *platthau*.
Na, fandest Du das angenehm zu lesen? War auf jeden Fall sehr
effizient zu schreiben!     [Philipp Zacharias in suse-linux]




More information about the AppArmor mailing list