Provide full AppArmor confinement for snaps on 14.04
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
apparmor (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Trusty |
Fix Released
|
High
|
Tyler Hicks | ||
dbus (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Trusty |
Fix Released
|
Medium
|
Tyler Hicks |
Bug Description
= apparmor SRU =
[Rationale]
For backporting snapd to 14.04 LTS, we need to provide proper AppArmor confinement for snaps when running under the 16.04 hardware enablement kernel. The apparmor userspace package in 14.04 is missing support key mediation features such as UNIX domain socket rules, AppArmor policy namespaces, and AppArmor profile stacking. UNIX domain socket mediation is needed by nearly all snaps. AppArmor policy namespaces and profile stacking are needed by the lxd snap.
Unfortunately, it was not feasible to backport the individual features to the 14.04 apparmor package as they're quite complex and have a large number of dependency patches. Additionally, the AppArmor policy abstractions from Ubuntu 16.04 are needed to provide proper snap confinement. Because of these two reasons, the decision to bring 16.04's apparmor package to 14.04 was (very carefully) made.
[Test Case]
https:/
This update will go through the Test Plan as well as manual testing to verify that snap confinement on 14.04 does work. Manual tests include installing snapd in 14.04 and running simple snaps such as pwgen-tyhicks and hello-world, as well as a much more complex snap such as lxd.
The following regression tests from lp:qa-regression-testing (these packages ship an AppArmor profile) can be used to verify that their respective packages do not regress:
test-apache2-
test-apache2-
test-apache2-
test-apache2-
test-apache2-
test-bind9.py
test-clamav.py
test-cups.py
test-dhcp.py
test-mysql.py
test-ntp.py
test-openldap.py
test-rsyslog.py
test-squid.py
test-strongswan.py
test-tcpdump.py
I have pushed changes to lp:qa-regression-testing which pulls in the parser and regression tests from the apparmor 2.8.95~
Additionally, manually testing evince, which is confined by an AppArmor profile, should be done. The manual test should check basic functionality as well as for proper confinement (`ps auxZ` output).
Finally, we need to test that 12.04 -> 14.04 upgrades continue to work. Specifically, the apparmor packages in trusty-proposed and the 12.04 kernel need to be tested together.
[Regression Potential]
High. We must be extremely careful to not regress existing, confined applications in Ubuntu 14.04. We are lucky that the upstream AppArmor project has extensive regression tests and that the Ubuntu Security team adds even more testing via the AppArmor Test Plan.
Care was taken to minimally change how the AppArmor policies are loaded during the boot process. I also verified that the abstractions shipped in apparmor and the profiles shipped in apparmor-profiles are the same across this SRU update.
= dbus SRU =
[Rationale]
For backporting snapd to 14.04 LTS, we need to provide proper D-Bus mediation for snaps when running under the 16.04 hardware enablement kernel. The dbus package in 14.04 is missing support for blocking unrequested reply messages. This functionality was added to the D-Bus AppArmor mediation patches after 14.04 was released but before the patches were merged upstream in dbus. The idea is to prevent a malicious snap from attacking another snap, over D-Bus, with unrequested reply messages and also to prevent two connections from subverting the snap confinement by communicating via unrequested reply messages.
[Test Case]
The upstream AppArmor userspace project has thorough tests for D-Bus mediation, including unrequested replies. Its tests/regressio
To run the dbus_*.sh tests:
$ sudo apt-get install -y bzr libdbus-1-dev
$ bzr branch lp:apparmor # apt-get source apparmor to test the current apparmor
$ cd apparmor/
$ make USE_SYSTEM=1 \
dbus_
$ for t in dbus_{eavesdrop
do sudo VERBOSE=1 bash $t || break; done
The exit code should be 0 and all output lines should start with "ok:".
In addition, the test-dbus.py tests from lp:qa-regression-testing should be ran to verify basic D-Bus functionality.
This update will go through the Test Plan as well as manual testing to verify that snap confinement on 14.04 does work. Manual tests include installing snapd in 14.04 and running simple snaps such as pwgen-tyhicks and hello-world, as well as a much more complex snap such as lxd.
[Regression Potential]
Low. There's no use for unrequested D-Bus reply messages and silently dropping them for AppArmor confined applications should have no unintended side effects. The unrequested reply protections have been present in releases after 14.04 and have not caused any issues.
description: | updated |
Changed in apparmor (Ubuntu): | |
status: | In Progress → Invalid |
Changed in apparmor (Ubuntu Trusty): | |
status: | New → In Progress |
importance: | Undecided → High |
Changed in apparmor (Ubuntu): | |
importance: | High → Undecided |
assignee: | Tyler Hicks (tyhicks) → nobody |
Changed in apparmor (Ubuntu Trusty): | |
assignee: | nobody → Tyler Hicks (tyhicks) |
description: | updated |
description: | updated |
description: | updated |
Changed in apparmor (Ubuntu Trusty): | |
status: | Incomplete → New |
status: | New → In Progress |
Changed in dbus (Ubuntu Trusty): | |
status: | Confirmed → In Progress |
description: | updated |
description: | updated |
description: | updated |
Adding a dbus task because its AppArmor mediation patches need to be updated to provide unrequested reply protection to prevent two D-Bus connections from bypassing security policies by communicating via reply and/or error D-Bus messages.