606d2fd665
Introduce the add_service() macro which wraps up add/find
permissions for the source domain with a neverallow preventing
others from adding it. Only a particular domain should
add a particular service.
Use the add_service() macro to automatically add a neverallow
that prevents other domains from adding the service.
mediadrmserver was adding services labeled mediaserver_service.
Drop the add permission as it should just need the find
permission.
Additionally, the macro adds the { add find } permission which
causes some existing neverallow's to assert. Adjust those
neverallow's so "self" can always find.
Test: compile and run on hikey and emulator. No new denials were
found, and all services, where applicable, seem to be running OK.
Change-Id: Ibbd2a5304edd5f8b877bc86852b0694732be993c
Signed-off-by: William Roberts <william.c.roberts@intel.com>
64 lines
1.8 KiB
Text
64 lines
1.8 KiB
Text
# healthd - battery/charger monitoring service daemon
|
|
type healthd, domain;
|
|
type healthd_exec, exec_type, file_type;
|
|
|
|
# Write to /dev/kmsg
|
|
allow healthd kmsg_device:chr_file rw_file_perms;
|
|
|
|
# Read access to pseudo filesystems.
|
|
r_dir_file(healthd, sysfs_type)
|
|
r_dir_file(healthd, rootfs)
|
|
r_dir_file(healthd, cgroup)
|
|
|
|
# Read access to system files for passthrough HALs in
|
|
# /{system,vendor,odm}/lib[64]/hw/
|
|
r_dir_file(healthd, system_file)
|
|
|
|
allow healthd self:capability { sys_tty_config };
|
|
allow healthd self:capability sys_boot;
|
|
|
|
allow healthd self:netlink_kobject_uevent_socket create_socket_perms_no_ioctl;
|
|
|
|
wakelock_use(healthd)
|
|
|
|
binder_use(healthd)
|
|
binder_service(healthd)
|
|
binder_call(healthd, system_server)
|
|
binder_call(healthd, hwservicemanager)
|
|
binder_call(healthd, hal_health)
|
|
|
|
# Write to state file.
|
|
# TODO: Split into a separate type?
|
|
allow healthd sysfs:file write;
|
|
|
|
# TODO: added to match above sysfs rule. Remove me?
|
|
allow healthd sysfs_usb:file write;
|
|
|
|
allow healthd sysfs_batteryinfo:file r_file_perms;
|
|
|
|
r_dir_file(healthd, sysfs_type)
|
|
|
|
###
|
|
### healthd: charger mode
|
|
###
|
|
|
|
# Read /sys/fs/pstore/console-ramoops
|
|
# Don't worry about overly broad permissions for now, as there's
|
|
# only one file in /sys/fs/pstore
|
|
allow healthd pstorefs:dir r_dir_perms;
|
|
allow healthd pstorefs:file r_file_perms;
|
|
|
|
allow healthd graphics_device:dir r_dir_perms;
|
|
allow healthd graphics_device:chr_file rw_file_perms;
|
|
allow healthd input_device:dir r_dir_perms;
|
|
allow healthd input_device:chr_file r_file_perms;
|
|
allow healthd tty_device:chr_file rw_file_perms;
|
|
allow healthd ashmem_device:chr_file execute;
|
|
allow healthd self:process execmem;
|
|
allow healthd proc_sysrq:file rw_file_perms;
|
|
|
|
add_service(healthd, batteryproperties_service)
|
|
|
|
# Healthd needs to tell init to continue the boot
|
|
# process when running in charger mode.
|
|
set_prop(healthd, system_prop)
|