[apparmor] [PATCH 4/6] Update swig to export all current interface fns
John Johansen
john.johansen at canonical.com
Fri Feb 18 02:32:25 UTC 2011
On 02/17/2011 05:53 PM, Seth Arnold wrote:
> Looks good, one small question later:
>
> On Thu, Feb 17, 2011 at 5:22 PM, John Johansen
> <john.johansen at canonical.com> wrote:
>> Signed-off-by: John Johansen <john.johansen at canonical.com>
>> ---
>> libraries/libapparmor/swig/SWIG/libapparmor.i | 13 ++++++++++---
>> 1 files changed, 10 insertions(+), 3 deletions(-)
>>
>> diff --git a/libraries/libapparmor/swig/SWIG/libapparmor.i b/libraries/libapparmor/swig/SWIG/libapparmor.i
>> index 014839a..f6d1001 100644
>> --- a/libraries/libapparmor/swig/SWIG/libapparmor.i
>> +++ b/libraries/libapparmor/swig/SWIG/libapparmor.i
>> @@ -2,13 +2,20 @@
>>
>> %{
>> #include "aalogparse.h"
>> -extern int aa_change_hat(const char *subprofile, unsigned long magic_token);
>> -extern int aa_change_profile(const char *profile, unsigned long magic_token);
>
> Is this mistake large enough to warrant a bug report anywhere? Or is
> it a case of no clients, no point bothering? :)
>
I'm in the no clients, no point bothering camp :)
>> +#include "apparmor.h"
>>
>> %}
>>
>> %include "typemaps.i"
>> %include "aalogparse.h"
>> +
>> +/* swig doesn't like the macro magic we do in apparmor.h so the fn prototypes
>> + * are manually inserted here
>> + */
>> +
>> extern int aa_change_hat(const char *subprofile, unsigned long magic_token);
>> -extern int aa_change_profile(const char *profile, unsigned long magic_token);
>> +extern int aa_change_profile(const char *profile);
>> +extern int aa_change_onexec(const char *profile);
>> +extern int aa_change_hatv(const char *subprofiles[], unsigned int token);
>> +extern int aa_change_hat_vargs(unsigned int token, int count, ...);
>
> Thanks
More information about the AppArmor
mailing list