<div dir="ltr">Hello<br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 27, 2015 at 8:33 PM, Christian Boltz <span dir="ltr"><<a href="mailto:apparmor@cboltz.de" target="_blank">apparmor@cboltz.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
as a preparation for the DbusRule class, add a <details> match group<br>
to RE_PROFILE_DBUS.<br>
<br>
Also adjust test-regex_matches.py for the added group.<br>
<br>
Note: RE_PROFILE_DBUS is only used in aa.py, and only matches[0..2]<br>
are used. 0 and 1 are audit and allow/deny and 2 is and stays the whole<br>
rule (except audit and allow/deny). Therefore no aa.py changes are<br>
needed.<br>
<br>
<br>
<br>
[ 52-add-match-group-to-RE_PROFILE_DBUS.diff ]<br>
<br>
=== modified file ./utils/apparmor/regex.py<br>
--- utils/apparmor/regex.py     2015-12-12 13:34:40.549997194 +0100<br>
+++ utils/apparmor/regex.py     2015-12-20 17:22:56.051619606 +0100<br>
@@ -46,7 +46,7 @@<br>
 RE_PROFILE_NETWORK      = re.compile(RE_AUDIT_DENY + 'network(?P<details>\s+.*)?' + RE_COMMA_EOL)<br>
 RE_PROFILE_CHANGE_HAT   = re.compile('^\s*\^(\"??.+?\"??)' + RE_COMMA_EOL)<br>
 RE_PROFILE_HAT_DEF      = re.compile('^(?P<leadingspace>\s*)(?P<hat_keyword>\^|hat\s+)(?P<hat>\"??.+?\"??)\s+((flags=)?\((?P<flags>.+)\)\s+)*\{' + RE_EOL)<br>
-RE_PROFILE_DBUS         = re.compile(RE_AUDIT_DENY + '(dbus\s*,|dbus\s+[^#]*\s*,)' + RE_EOL)<br>
+RE_PROFILE_DBUS         = re.compile(RE_AUDIT_DENY + '(dbus\s*,|dbus(?P<details>\s+[^#]*)\s*,)' + RE_EOL)<br>
 RE_PROFILE_MOUNT        = re.compile(RE_AUDIT_DENY + '((mount|remount|umount|unmount)(\s+[^#]*)?\s*,)' + RE_EOL)<br>
 RE_PROFILE_SIGNAL       = re.compile(RE_AUDIT_DENY + '(signal\s*,|signal(?P<details>\s+[^#]*)\s*,)' + RE_EOL)<br>
 RE_PROFILE_PTRACE       = re.compile(RE_AUDIT_DENY + '(ptrace\s*,|ptrace(?P<details>\s+[^#]*)\s*,)' + RE_EOL)<br>
=== modified file ./utils/test/test-regex_matches.py<br>
--- utils/test/test-regex_matches.py    2015-12-12 13:34:40.553997166 +0100<br>
+++ utils/test/test-regex_matches.py    2015-12-20 17:26:03.002435746 +0100<br>
@@ -259,10 +259,10 @@<br>
         self.regex = aa.RE_PROFILE_DBUS<br>
<br>
     tests = [<br>
-        ('   dbus,', (None, None, 'dbus,', None)),<br>
-        ('   audit dbus,', ('audit', None, 'dbus,', None)),<br>
-        ('   dbus send member=no_comment,', (None, None, 'dbus send member=no_comment,', None)),<br>
-        ('   dbus send member=no_comment, # comment', (None, None, 'dbus send member=no_comment,', '# comment')),<br>
+        ('   dbus,',                                    (None,      None,   'dbus,',                            None,                       None)),<br>
+        ('   audit dbus,',                              ('audit',   None,   'dbus,',                            None,                       None)),<br>
+        ('   dbus send member=no_comment,',             (None,      None,   'dbus send member=no_comment,',     'send member=no_comment',   None)),<br>
+        ('   dbus send member=no_comment, # comment',   (None,      None,   'dbus send member=no_comment,',     'send member=no_comment',   '# comment')),<br>
<br>
         ('   dbusdriver,', False),<br>
         ('   audit dbusdriver,', False),<br>
<br>
<br></blockquote><div><br>Acked-by: Kshitij Gupta <<a href="mailto:kgupta8592@gmail.com" target="_blank">kgupta8592@gmail.com</a>>   <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
<br>
Christian Boltz<br>
<span class=""><font color="#888888">--<br>
I built version 1.7.1 in my branch.. a bit messy, as most work seems<br>
to have gone into 'replacing autofoo with cmake' (aka one broken build<br>
system for another one). [Dominique Leuenberger in opensuse-factory]<br>
</font></span><br>--<br>
AppArmor mailing list<br>
<a href="mailto:AppArmor@lists.ubuntu.com">AppArmor@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/apparmor" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/apparmor</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards,<br><br></div>Kshitij Gupta<br></div></div>
</div></div>