[apparmor] [PATCH 8/8] Convert codomain to a class

Steve Beattie steve at nxnw.org
Fri Sep 20 19:29:31 UTC 2013


On Fri, Sep 20, 2013 at 12:26:11PM -0700, Steve Beattie wrote:
> On Wed, Sep 11, 2013 at 01:47:47AM -0700, Tyler Hicks wrote:
> > From: John Johansen <john.johansen at canonical.com>
> > 
> > Convert the codomain to a class, and the policy lists that store
> > codomains to stl containers instead of glibc twalk.
> > 
> > Signed-off-by: John Johansen <john.johansen at canonical.com>
> > [tyhicks: Merge with dbus changes and process_file_entries() cleanup]
> > Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> 
> There's still a problem with this patch (even with the other fixes I
> made to it), it's not loading profiles with multiple hats properly:
> 
>   $ cat /tmp/example_profile
>   /tests/regression/apparmor/changehat {
> 
>     ^sub {
>       /proc/*/attr/current w,
>       /tmp/sdtest.1713-15650-z0Mlub/file2 rw,
>     }
> 
>     ^sub2 {
>       /proc/*/attr/current w,
>       /tmp/sdtest.1713-15650-z0Mlub/file2 rw,
>     }
> 
>     ^sub3 {
>       /proc/*/attr/current w,
>       /tmp/sdtest.1713-15650-z0Mlub/file2 rw,
>     }
>   }
> 
>   $ sudo ./apparmor_parser /tmp/example_profile
> 
>   $ sudo grep changehat /sys/kernel/security/apparmor/profiles
>   /tests/regression/apparmor/changehat (enforce)
>   /tests/regression/apparmor/changehat//sub (enforce)
> 
> Rebuilding the parser with debugging enabled for just parser_interface.c
> (via 'make clean all && rm parser_interface.o && make parser_interface.o && make')

Err this should be:

  make clean all && rm parser_interface.o && make parser_interface.o DEBUG=y && make

> gives the following output when loading the profile:
> 
>   $ sudo ./apparmor_parser /tmp/example_profile
>   parser: Serializing policy for /tests/regression/apparmor/changehat.
>   parser: Writing name 'version'
>   parser: Writing name 'profile'
>   parser: Writing name '(null)'
>   parser: Writing name 'flags'
>   parser: Writing name 'caps64'
>   parser: Writing name 'aadfa'
>   parser: Serializing policy for sub.
>   parser: Writing name 'version'
>   parser: Writing name 'profile'
>   parser: Writing name '(null)'
>   parser: Writing name 'flags'
>   parser: Writing name 'caps64'
>   parser: Writing name 'aadfa'
> 
> I'm not sure why it's only picking out the first of the hats.
> 
> -- 
> Steve Beattie
> <sbeattie at ubuntu.com>
> http://NxNW.org/~steve/



> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor


-- 
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/20130920/79e1a101/attachment.pgp>


More information about the AppArmor mailing list