[apparmor] [patch] aa.py: change_profile vs. changes_profile

[Christian Boltz]

Hello,

another patch from the "strange things noticed while reading aa.py" 
series:

aa.py uses profile_data[profile][hat]['change_profile'] at various 
places. However, there are also two places that use 'changes_profile' 
(note the additional 's').

I didn't test this patch because I don't have a profile with a 
change_profile rule here, but I'm quite sure it it needed to fix 
handling change_profile rules ;-)


[ aa.py-change_s_profile.diff ]

=== modified file 'utils/apparmor/aa.py'
--- utils/apparmor/aa.py        2014-11-27 17:34:45 +0000
+++ utils/apparmor/aa.py        2014-11-27 19:19:07 +0000
@@ -2769,7 +2769,7 @@
                 raise AppArmorException(_('Syntax Error: Unexpected change profile entry found in file: %(file)s line: %(line)s') % { 'file': file, 'line': lineno + 1 })
 
             cp = strip_quotes(matches[0])
-            profile_data[profile][hat]['changes_profile'][cp] = True
+            profile_data[profile][hat]['change_profile'][cp] = True
 
         elif RE_PROFILE_ALIAS.search(line):
             matches = RE_PROFILE_ALIAS.search(line).groups()
@@ -4005,7 +4005,7 @@
                 matches = RE_PROFILE_CHANGE_PROFILE.search(line).groups()
                 cp = strip_quotes(matches[0])
 
-                if not write_prof_data[hat]['changes_profile'][cp] is True:
+                if not write_prof_data[hat]['change_profile'][cp] is True:
                     correct = False
 
                 if correct:



Regards,

Christian Boltz
-- 
> vorweg: Hier war gerade Stromausfall. Da hat der Sturm wohl
> irgendwo einen Baum auf die Leitung geworfen...
> ext3 hat mich zwar anscheinend vor größeren Schäden bewahrt.
Mit reiserfs stände der Baum noch. :-)
[> Christian Boltz und Ratti in fontlinge-devel]