<div dir="ltr">Hello,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 21, 2016 at 10:03 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>
$subject.<br>
<br>
This should happen rarely, but nevertheless it can happen - and since<br>
AppArmor needs the symlink target in the profile, we have to resolve any<br>
symlink.<br>
<br>
<br>
[ 76-handle_binfmt-resolve-symlinks.diff ]<br>
<br>
=== modified file ./utils/apparmor/aa.py<br>
--- utils/apparmor/aa.py        2016-02-21 17:14:28.444520585 +0100<br>
+++ utils/apparmor/aa.py        2016-02-21 16:06:41.744595751 +0100<br>
@@ -386,6 +388,7 @@<br>
     reqs = get_reqs(path)<br>
     while reqs:<br>
         library = reqs.pop()<br>
+        library = get_full_path(library)  # resolve symlinks<br></blockquote><div>How about inlining the get_full_path with the pop?<br></div><div>Also, is the comment above adding any value and worth it?<br><br>Acked-by: Kshitij Gupta <<a href="mailto:kgupta8592@gmail.com" target="_blank">kgupta8592@gmail.com</a>><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">
         if not reqs_processed.get(library, False):<br>
             if get_reqs(library):<br>
                 reqs += get_reqs(library)<br>
<br>
<br>
<br>
Regards,<br>
<br>
Christian Boltz<br>
<span class=""><font color="#888888">--<br>
The "Well" was referring to my role: as a product<br>
manager, I am not even entitled to review the "validity<br>
of implementation details", ... I sometime cannot resist<br>
and do nevertheless, ...;-)<br>
[Matthias G. Eckermann 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></div>