<div dir="ltr">Hello,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 15, 2015 at 12:45 AM, 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>
load_variables() used a nearly-correct regex. Drop it and use<br>
re_match_include() instead.<br>
<br>
<br>
<br>
[ 50-severity-use-re_match_include.diff ]<br>
<br>
=== modified file utils/apparmor/severity.py<br>
--- utils/apparmor/severity.py 2015-06-06 14:53:16.868029000 +0200<br>
+++ utils/apparmor/severity.py 2015-06-14 21:09:57.462206062 +0200<br>
@@ -15,6 +15,7 @@<br>
import os<br>
import re<br>
from apparmor.common import AppArmorException, open_file_read, warn, convert_regexp # , msg, error, debug<br>
+from apparmor.regex import re_match_include<br>
<br>
class Severity(object):<br>
def __init__(self, dbname=None, default_rank=10):<br>
@@ -179,16 +180,14 @@<br>
<br>
def load_variables(self, prof_path):<br>
"""Loads the variables for the given profile"""<br>
- regex_include = re.compile('^#?include\s*<(\S*)>')<br>
if os.path.isfile(prof_path):<br>
with open_file_read(prof_path) as f_in:<br>
for line in f_in:<br>
line = line.strip()<br>
# If any includes, load variables from them first<br>
- match = regex_include.search(line)<br>
+ match = re_match_include(line)<br>
if match:<br>
- new_path = match.groups()[0]<br>
- new_path = self.PROF_DIR + '/' + new_path<br>
+ new_path = self.PROF_DIR + '/' + match<br>
self.load_variables(new_path)<br>
else:<br>
# Remove any comments<br>
<br>
<br></blockquote><div>Thanks for the patch.<br><br>Acked-by: Kshitij Gupta <<a href="mailto:kgupta8592@gmail.com" target="_blank">kgupta8592@gmail.com</a>>. <br><div><br></div>Regards, <br><br></div><div>Kshitij Gupta<br><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>
Jetzt bringt das KDE schon ein eigenes shutdown mit? Ist ja ein kHammer!<br>
(und morgen müssen wir kkernel (von klinux) und kinit installieren, was<br>
dann mit kbash und kperl das knetzwerk, kindernet, papache, kquid und<br>
kostfix knochfährt und dann das kX.org köffnet und kkde kanzeigt, kurz:<br>
Kunix! SKNR.) [Steffen Dettmer in suse-linux]<br>
<br>
<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>
</font></span></blockquote></div><br></div></div>