diff options
| author | Eric Biggers <ebiggers@google.com> | 2021-12-19 22:17:20 -0600 |
|---|---|---|
| committer | Eric Biggers <ebiggers@google.com> | 2021-12-20 10:24:15 -0600 |
| commit | d0b9e2c995beb13c70a1549923df482ff773f09b (patch) | |
| tree | b8d89c0ee0a18aa38b0a5df1d06ab6037f8c2190 /pam_fscrypt | |
| parent | 1014b61a6a054b5c82b2be82e13d8ce28befba45 (diff) | |
filesystem: avoid accessing irrelevant filesystems
Forbid 'fscrypt setup' on filesystems that aren't expected to support
encryption (other than the root filesystem), and skip looking for
fscrypt metadata directories on such filesystems. This has two
benefits. First, it avoids the printing of annoying warnings like:
pam_fscrypt[75038]: stat /run/user/0/.fscrypt: permission denied
pam_fscrypt[75038]: stat /run/user/0/.fscrypt/policies: permission denied
pam_fscrypt[75038]: stat /run/user/0/.fscrypt/protectors: permission denied
pam_fscrypt[75038]: stat /sys/firmware/efi/efivars/.fscrypt: invalid argument
pam_fscrypt[75038]: stat /sys/firmware/efi/efivars/.fscrypt/policies: invalid argument
pam_fscrypt[75038]: stat /sys/firmware/efi/efivars/.fscrypt/protectors: invalid argument
pam_fscrypt[75038]: stat /sys/fs/pstore/.fscrypt: permission denied
pam_fscrypt[75038]: stat /sys/fs/pstore/.fscrypt/policies: permission denied
pam_fscrypt[75038]: stat /sys/fs/pstore/.fscrypt/protectors: permission denied
Second, it avoids long delays or side effects on some filesystems.
To do this, introduce an allowlist of filesystem types that fscrypt will
recognize. I wanted to avoid doing this, since this list will need to
be updated in the future, but I don't see a better solution.
Diffstat (limited to 'pam_fscrypt')
0 files changed, 0 insertions, 0 deletions