[apparmor] [PATCH 4/4] libapparmor: Provide privately exported aa_is_blacklisted() through swig
John Johansen
john.johansen at canonical.com
Tue Jun 16 20:42:02 UTC 2015
On 06/16/2015 01:25 PM, Christian Boltz wrote:
> Hello,
>
> Am Dienstag, 16. Juni 2015 schrieb Tyler Hicks:
>> On 2015-06-16 13:14:15, Christian Boltz wrote:
>
>>> BTW: The function definition is
>>>
>>> int _aa_is_blacklisted(const char *name, const char *path)
>>>
>>> but "path" is never used inside the function. Is this intentional?
>>
>> Nice catch. No, it isn't intentional. I'll fix it in a follow up patch
>> and will include the rational in the commit message.
>
> Thanks!
>
>> There's an additional follow up patch that I'll also be sending that
>> prevents crazy exceptions from being raised by
>> LibAppArmor._aa_is_blacklisted() in some situations.
>>
>> Do I have your ACK on this patch?
>
> With my limited C knownledge, I hesitate to ack any C or C++ patches
> that do more than updating a comment or fixing a typo in a string ;-)
>
> However, this patch looks simple enough, so I'll accept the risk ;-) and
> say
> Acked-by: Christian Boltz <apparmor at cboltz.de>
>
If that wasn't enough
Acked-by: John Johansen <john.johansen at canonical.com>
More information about the AppArmor
mailing list