From cddb6ad619871acc695686d32d66ebb9450deebb Mon Sep 17 00:00:00 2001 From: Nicole Lee Date: Sun, 15 Jan 2023 17:05:20 +0000 Subject: [PATCH] logger_app: allow access vendor_gps_file, vendor_gps_prop, vendor_logger_prop Bug: 261519049 Bug: 261783031 Bug: 261933367 Test: Confirm no selinux denial for these 3 tcontexts Change-Id: I6f919e193693f7521778321f677214ea9f3b4d84 --- legacy/whitechapel_pro/property.te | 2 -- legacy/whitechapel_pro/property_contexts | 9 --------- radio/logger_app.te | 4 ++++ radio/property.te | 2 ++ radio/property_contexts | 9 +++++++++ tracking_denials/logger_app.te | 16 ---------------- 6 files changed, 15 insertions(+), 27 deletions(-) diff --git a/legacy/whitechapel_pro/property.te b/legacy/whitechapel_pro/property.te index ee160863..73b7883e 100644 --- a/legacy/whitechapel_pro/property.te +++ b/legacy/whitechapel_pro/property.te @@ -5,10 +5,8 @@ vendor_internal_prop(vendor_camera_prop) vendor_internal_prop(vendor_camera_debug_prop) vendor_internal_prop(vendor_camera_fatp_prop) vendor_internal_prop(vendor_tcpdump_log_prop) -vendor_internal_prop(vendor_gps_prop) vendor_internal_prop(vendor_ro_sys_default_prop) vendor_internal_prop(vendor_persist_sys_default_prop) -vendor_internal_prop(vendor_logger_prop) vendor_internal_prop(vendor_display_prop) # UWB calibration diff --git a/legacy/whitechapel_pro/property_contexts b/legacy/whitechapel_pro/property_contexts index 3a996757..46c67b7e 100644 --- a/legacy/whitechapel_pro/property_contexts +++ b/legacy/whitechapel_pro/property_contexts @@ -15,19 +15,10 @@ persist.vendor.se. u:object_r:vendor_secure_element_prop ro.vendor.hwc.drm.device u:object_r:vendor_display_prop:s0 persist.vendor.display. u:object_r:vendor_display_prop:s0 -# for logger app -vendor.pixellogger. u:object_r:vendor_logger_prop:s0 -persist.vendor.pixellogger. u:object_r:vendor_logger_prop:s0 -persist.vendor.verbose_logging_enabled u:object_r:vendor_logger_prop:s0 - # vendor default ro.vendor.sys. u:object_r:vendor_ro_sys_default_prop:s0 persist.vendor.sys. u:object_r:vendor_persist_sys_default_prop:s0 -# for gps -vendor.gps. u:object_r:vendor_gps_prop:s0 -persist.vendor.gps. u:object_r:vendor_gps_prop:s0 - #uwb ro.vendor.uwb.calibration. u:object_r:vendor_uwb_calibration_prop:s0 exact string diff --git a/radio/logger_app.te b/radio/logger_app.te index d4106a21..1f1229b0 100644 --- a/radio/logger_app.te +++ b/radio/logger_app.te @@ -1,7 +1,11 @@ userdebug_or_eng(` + allow logger_app vendor_gps_file:file create_file_perms; + allow logger_app vendor_gps_file:dir create_dir_perms; allow logger_app radio_vendor_data_file:file create_file_perms; allow logger_app radio_vendor_data_file:dir create_dir_perms; + set_prop(logger_app, vendor_gps_prop) + set_prop(logger_app, vendor_logger_prop) set_prop(logger_app, vendor_modem_prop) set_prop(logger_app, vendor_ssrdump_prop) ') diff --git a/radio/property.te b/radio/property.te index defcf184..be1eb4f3 100644 --- a/radio/property.te +++ b/radio/property.te @@ -8,6 +8,8 @@ vendor_internal_prop(vendor_rild_prop) vendor_internal_prop(vendor_ssrdump_prop) vendor_internal_prop(vendor_wifi_version) vendor_internal_prop(vendor_imssvc_prop) +vendor_internal_prop(vendor_gps_prop) +vendor_internal_prop(vendor_logger_prop) # Telephony debug app vendor_internal_prop(vendor_telephony_app_prop) diff --git a/radio/property_contexts b/radio/property_contexts index 1ccd9ec1..2cc5b4fd 100644 --- a/radio/property_contexts +++ b/radio/property_contexts @@ -17,6 +17,11 @@ vendor.sys.dmd. u:object_r:vendor_diag_prop:s0 vendor.sys.diag. u:object_r:vendor_diag_prop:s0 persist.vendor.config. u:object_r:vendor_persist_config_default_prop:s0 +# for logger app +vendor.pixellogger. u:object_r:vendor_logger_prop:s0 +persist.vendor.pixellogger. u:object_r:vendor_logger_prop:s0 +persist.vendor.verbose_logging_enabled u:object_r:vendor_logger_prop:s0 + # Modem persist.vendor.modem. u:object_r:vendor_modem_prop:s0 vendor.modem. u:object_r:vendor_modem_prop:s0 @@ -44,3 +49,7 @@ vendor.wlan.firmware.version u:object_r:vendor_wifi_version:s0 # for vendor telephony debug app vendor.config.debug. u:object_r:vendor_telephony_app_prop:s0 + +# for gps +vendor.gps. u:object_r:vendor_gps_prop:s0 +persist.vendor.gps. u:object_r:vendor_gps_prop:s0 diff --git a/tracking_denials/logger_app.te b/tracking_denials/logger_app.te index b65f3bbc..9443bc55 100644 --- a/tracking_denials/logger_app.te +++ b/tracking_denials/logger_app.te @@ -1,19 +1,3 @@ -# b/261519049 -dontaudit logger_app vendor_gps_file:dir { getattr }; -dontaudit logger_app vendor_gps_file:dir { read }; -dontaudit logger_app vendor_gps_file:dir { search }; -dontaudit logger_app vendor_gps_prop:property_service { set }; -dontaudit logger_app vendor_logger_prop:property_service { set }; -# b/261783031 -dontaudit logger_app vendor_gps_file:dir { open }; -# b/261933367 -dontaudit logger_app vendor_gps_file:dir { add_name }; -dontaudit logger_app vendor_gps_file:dir { write }; -dontaudit logger_app vendor_gps_file:file { append }; -dontaudit logger_app vendor_gps_file:file { create }; -dontaudit logger_app vendor_gps_file:file { getattr }; -dontaudit logger_app vendor_gps_file:file { read write open }; -dontaudit logger_app vendor_gps_file:file { setattr }; # b/264489961 userdebug_or_eng(` permissive logger_app;