[apparmor] RFC [Patch 0/1] Update log parsing interface

[Steve Beattie]

On Tue, Jul 12, 2011 at 03:49:29PM -0700, John Johansen wrote:
> While I am tinkering with the library, I got to wondering why we ever/still
> export the log parsing library interface without the aa_ prefix.

Heh, I was just wondering the same thing as I was poking at the python
library getting all excited about the idea of being to query the library
to determine confinement status rather than manually doing it.

> This patch takes care of the library end of things, and should allow older
> versions of programs calling the library to function.  This patch doesn't
> currently cover the update of the tools calling the library to use the
> new interface, nor does it deal with swig clients breaking if a new
> swig interface is used.

I'm not convinced it's actually backwards compatible in the way
you think it is. IIRC, if you want to offer an old (versioned)
symbol available, you (in the past anyway) needed to do the same
sort of aliasing that's done for the old change_hat (IMMUNIX_1.0 and
APPARMOR_1.0) symbols. 'readelf' output on the generated .so would
tell you for sure.

> What do you think should we make the jump?

But otherwise, I'm all in favor of this.

> If so should the swig interface maintain support for the none aa_ version of
> the function?

I think it needs to stick around for a while, sadly.

Thanks!


-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20110712/972a8892/attachment.pgp>