diff options
| author | Kees Cook <keescook@chromium.org> | 2019-04-08 09:07:06 -0700 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2019-04-10 04:24:48 -0700 |
| commit | e33c1b9923775d17ad246946fe67fcb9be288677 (patch) | |
| tree | 11ff1477557e9608883f7d8986cb54f4fadfe239 /include | |
| parent | 771acc7e4a6e5dba779cb1a7fd851a164bc81033 (diff) | |
apparmor: Restore Y/N in /sys for apparmor's "enabled"
Before commit c5459b829b71 ("LSM: Plumb visibility into optional "enabled"
state"), /sys/module/apparmor/parameters/enabled would show "Y" or "N"
since it was using the "bool" handler. After being changed to "int",
this switched to "1" or "0", breaking the userspace AppArmor detection
of dbus-broker. This restores the Y/N output while keeping the LSM
infrastructure happy.
Before:
$ cat /sys/module/apparmor/parameters/enabled
1
After:
$ cat /sys/module/apparmor/parameters/enabled
Y
Reported-by: David Rheinsberg <david.rheinsberg@gmail.com>
Reviewed-by: David Rheinsberg <david.rheinsberg@gmail.com>
Link: https://lkml.kernel.org/r/CADyDSO6k8vYb1eryT4g6+EHrLCvb68GAbHVWuULkYjcZcYNhhw@mail.gmail.com
Fixes: c5459b829b71 ("LSM: Plumb visibility into optional "enabled" state")
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions