Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Update AppArmor policy to by in sync with LXC #256

Closed
wants to merge 1 commit into from
Closed

Update AppArmor policy to by in sync with LXC #256

wants to merge 1 commit into from

Conversation

jdstrand
Copy link

@jdstrand jdstrand commented Nov 7, 2014

Specifically:

  • reorganizes the rules to allow for easier comparison with other container
    policy
  • adds comments for many rules
  • adds bare dbus rule
  • adds ptrace rule to allow ptracing ourselves
  • adds deny mount options=(ro, remount, silent) -> /
  • allows hugetlbfs
  • adds cgmanager mount
  • adds /sys/fs/pstore mount
  • more specific /sys/kernel/security mount options
  • more specific /sys mount options
  • more specific /proc/sys/kernel/* deny rules
  • more specific /proc/sys/net deny rules
  • more specific /sys/class deny rules
  • more specific /sys/devices deny rules
  • more specific /sys/fs deny rules

Closes #8454

Signed-off-by: Jamie Strandboge jamie@ubuntu.com

Specifically:
- reorganizes the rules to allow for easier comparison with other container
  policy
- adds comments for many rules
- adds bare dbus rule
- adds ptrace rule to allow ptracing ourselves
- adds deny mount options=(ro, remount, silent) -> /
- allows hugetlbfs
- adds cgmanager mount
- adds /sys/fs/pstore mount
- more specific /sys/kernel/security mount options
- more specific /sys mount options
- more specific /proc/sys/kernel/* deny rules
- more specific /proc/sys/net deny rules
- more specific /sys/class deny rules
- more specific /sys/devices deny rules
- more specific /sys/fs deny rules

Closes #8454

Signed-off-by: Jamie Strandboge <jamie@ubuntu.com>
@jdstrand
Copy link
Author

jdstrand commented Nov 7, 2014

Note, this will only work with kernels that support AppArmor features from 2.9 (ie, dbus, signal, ptrace and unix rules). Currently, this is Ubuntu 14.10. My suggestions from moby/moby#7276 would ideally be applied along with this patch.

@jdstrand jdstrand closed this Nov 7, 2014
@jdstrand
Copy link
Author

jdstrand commented Nov 7, 2014

I'm going to come up with a patch that should be safe everywhere, then redo the PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant