<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:lennart@poettering.net" title="Lennart Poettering <lennart@poettering.net>"> <span class="fn">Lennart Poettering</span></a>
</span> changed
<a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - Apparmor support in ConditionSecurity"
href="https://bugs.freedesktop.org/show_bug.cgi?id=63312">bug 63312</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>RESOLVED
</td>
<td>REOPENED
</td>
</tr>
<tr>
<td style="text-align:right;">Resolution</td>
<td>FIXED
</td>
<td>---
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - Apparmor support in ConditionSecurity"
href="https://bugs.freedesktop.org/show_bug.cgi?id=63312#c9">Comment # 9</a>
on <a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - Apparmor support in ConditionSecurity"
href="https://bugs.freedesktop.org/show_bug.cgi?id=63312">bug 63312</a>
from <span class="vcard"><a class="email" href="mailto:lennart@poettering.net" title="Lennart Poettering <lennart@poettering.net>"> <span class="fn">Lennart Poettering</span></a>
</span></b>
<pre>Hmm, I just noticed this patch:
<a href="https://code.launchpad.net/~mdeslaur/upstart/apparmor-support/+merge/164169">https://code.launchpad.net/~mdeslaur/upstart/apparmor-support/+merge/164169</a>
It contains a different check for AppArmor. Basically something like this:
/sys/module/apparmor/parameters/enabled == 'Y'
I'd prefer if we could change our code to do the same, given that the Ubuntu
guys are guys are upstream for apparmor.
Any chance you could rework the condition check? Most likely you can just use:
static bool apparmor_enabled(void) {
_cleanup_free_ char *p = NULL;
r = read_one_line_file("/sys/module/apparmor/parameters/enabled", &p);
if (r < 0)
return false;
return parse_boolean(p) > 0;
}
But I can't test this, so I am kinda counting on you to check if this works and
provide a patch?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>