6.8 KiB
title |
---|
Structure |
Description of common structure found across various AppArmor profiles
Programs to not confine
Some programs should not be confined by themselves. For example, tools such as ls
, rm
, diff
or cat
do not have profiles in this project. Let's see why.
These are general tools that in a general context can legitimately access any file in the system. Therefore, the confinement of such tools by a global profile would at best be minimal at worst be a security theater.
It gets even worse. Let's say, we write a profile for cat
. Such a profile would need access to /etc/
. We will add the following rule:
/etc/{,**} rw,
However, as /etc
can contain sensitive files, we now want to explicitly prevent access to these sensitive files. Problems:
- How do we know the exhaustive list of sensitive files in
/etc
? - How do we ensure access to these sensitive files are not required?
- This breaks the principle of mandatory access control. See the first rule of this project that is to only allow what is required. Here we allow everything and blacklist some paths.
It creates even more issues when we want to use this profile in other profiles. Let's take the example of diff
. Using this rule: @{bin}/diff rPx,
will restrict access to the very generic and not very confined diff
profile. Whereas most of the time, we want to restrict diff
to some specific file in our profile:
- In
dpkg
, an internal child profile (rCx -> diff
), allowsdiff
to only access etc config files:
!!! note ""
[apparmor.d/apparmor.d/groups/apt/dpkg](https://github.com/roddhjav/apparmor.d/blob/accf5538bdfc1598f1cc1588a7118252884df50c/apparmor.d/groups/apt/dpkg#L123)
``` aa linenums="123"
profile diff {
include <abstractions/base>
include <abstractions/consoles>
@{bin}/ r,
@{bin}/pager mr,
@{bin}/less mr,
@{bin}/more mr,
@{bin}/diff mr,
owner @{HOME}/.lesshs* rw,
# Diff changed config files
/etc/** r,
# For shell pwd
/root/ r,
}
```
- In
pass
, as it is a dependency of pass. Herediff
inherits pass' profile and has the same access than the pass profile, so it will be allowed to diff password files because more than a genericdiff
it is adiff
for the pass password manager:
!!! note ""
[apparmor.d/apparmor.d/profiles-m-r/pass](https://github.com/roddhjav/apparmor.d/blob/accf5538bdfc1598f1cc1588a7118252884df50c/apparmor.d/profiles-m-r/pass#L20
)
``` aa linenums="20"
@{bin}/diff rix,
```
What if I still want to protect these programs?
You do not protect these programs. Protect the usage you have of these programs. In practice, it means that you should put your development's terminal in a sandbox managed with Toolbox.
!!! example "To sum up"
1. Do not a create profile for programs such as: `rm`, `ls`, `diff`, `cd`, `cat`
2. Do not a create profile for the shell: `bash`, `sh`, `dash`, `zsh`
3. Use [Toolbox].
Abstractions
This project and the apparmor profile official project provide a large selection of abstractions to be included in profiles. They should be used.
For instance, to allow download directory access, instead of writing:
owner @{HOME}/@{XDG_DOWNLOAD_DIR}/{,**} rw,
You should write:
include <abstractions/user-download-strict>
Children profiles
Usually, a child profile is in the children
group. They have the following note:
!!! quote
Note: This profile does not specify an attachment path because it is
intended to be used only via `"Px -> child-open"` exec transitions
from other profiles.
Here is an overview of the current children profile:
-
child-open
: To open resources. Instead of allowing the run of all software in@{bin}/
, the purpose of this profile is to list all GUI programs that can open resources. Ultimately, only sandbox manager programs such asbwrap
,snap
,flatpak
,firejail
should be present here. Until this day, this profile will be a controlled mess. -
child-pager
: Simple access to pager such aspager
,less
andmore
. This profile supposes the pager is reading its data from stdin, not from a file on disk. -
child-systemctl
: Commonsystemctl
action. Do not use it too much as most of the time you will need more privilege than what this profile is giving you.
Browsers
Chromium based browsers share a similar structure. Therefore, they share the same abstraction: abstractions/chromium
that includes most of the profile content.
This abstraction requires the following variables defined in the profile header:
@{name} = chromium
@{domain} = org.chromium.Chromium
@{lib_dirs} = @{lib}/chromium
@{config_dirs} = @{user_config_dirs}/chromium
@{cache_dirs} = @{user_cache_dirs}/chromium
If your application requires chromium to run (like electron) use abstractions/chromium-common
instead.
Udev rules
See the kernel docs to check the major block and char numbers used in /run/udev/data/
.
Special care must be given as sometimes udev numbers are allocated dynamically by the kernel. Therefore, the full range must be allowed:
!!! note ""
[apparmor.d/groups/virt/libvirtd](https://github.com/roddhjav/apparmor.d/blob/b2af7a631a2b8aca7d6bdc8f7ff4fdd5ec94220e/apparmor.d/groups/virt/libvirtd#L188)
``` aa linenums="179"
@{run}/udev/data/c@{dynamic}:@{int} r, # For dynamic assignment range 234 to 254, 384 to 511
```
No New Privileges
No New Privileges is a flag preventing a newly-started program to get more privileges that its parent. So it is a good thing for security. And it is commonly used in systemd unit files (when possible). This flag also prevents transition to other profile because it could be less restrictive than the parent profile (no Px
or Ux
allowed).
The possible solutions are:
- The easiest (and unfortunately less secure) workaround is to ensure the programs do not run with no new privileges flag by disabling
NoNewPrivileges
in the systemd unit (or any other options implying it). - Inherit the current confinement (
ix
) - Stacking