From f652d07d61b5fd1501a16c1048f2a8e5e160816f Mon Sep 17 00:00:00 2001 From: Adam Shih Date: Mon, 24 Oct 2022 14:30:52 +0800 Subject: [PATCH] review dmd and modem_logging_control Bug: 254378739 Test: boot with both of them launched Change-Id: I3402c89be55bb8258d0ceb8475cef9fb913d2604 --- legacy/whitechapel_pro/file.te | 2 -- legacy/whitechapel_pro/file_contexts | 3 --- legacy/whitechapel_pro/hwservice.te | 3 --- legacy/whitechapel_pro/hwservice_contexts | 3 --- legacy/whitechapel_pro/property.te | 3 --- legacy/whitechapel_pro/property_contexts | 13 ------------- {legacy/whitechapel_pro => vendor}/dmd.te | 0 vendor/file.te | 2 ++ vendor/file_contexts | 3 +++ vendor/hwservice.te | 3 +++ vendor/hwservice_contexts | 3 +++ .../modem_logging_control.te | 0 vendor/property.te | 3 +++ vendor/property_contexts | 12 ++++++++++++ 14 files changed, 26 insertions(+), 27 deletions(-) rename {legacy/whitechapel_pro => vendor}/dmd.te (100%) create mode 100644 vendor/hwservice.te create mode 100644 vendor/hwservice_contexts rename {legacy/whitechapel_pro => vendor}/modem_logging_control.te (100%) diff --git a/legacy/whitechapel_pro/file.te b/legacy/whitechapel_pro/file.te index f9ea082d..21f15844 100644 --- a/legacy/whitechapel_pro/file.te +++ b/legacy/whitechapel_pro/file.te @@ -2,7 +2,6 @@ type rild_vendor_data_file, file_type, data_file_type; type vendor_hwc_log_file, file_type, data_file_type; type modem_stat_data_file, file_type, data_file_type; -type vendor_slog_file, file_type, data_file_type; type updated_wifi_firmware_data_file, file_type, data_file_type; type tcpdump_vendor_data_file, file_type, data_file_type; type vendor_camera_data_file, file_type, data_file_type; @@ -15,7 +14,6 @@ type uwb_data_vendor, file_type, data_file_type; type powerstats_vendor_data_file, file_type, data_file_type; userdebug_or_eng(` typeattribute tcpdump_vendor_data_file mlstrustedobject; - typeattribute vendor_slog_file mlstrustedobject; ') # sysfs diff --git a/legacy/whitechapel_pro/file_contexts b/legacy/whitechapel_pro/file_contexts index a34cc089..d8681295 100644 --- a/legacy/whitechapel_pro/file_contexts +++ b/legacy/whitechapel_pro/file_contexts @@ -1,6 +1,5 @@ # Binaries /vendor/bin/dmd u:object_r:dmd_exec:s0 -/vendor/bin/modem_logging_control u:object_r:modem_logging_control_exec:s0 /vendor/bin/sced u:object_r:sced_exec:s0 /vendor/bin/vcd u:object_r:vcd_exec:s0 /vendor/bin/chre u:object_r:chre_exec:s0 @@ -139,7 +138,6 @@ /dev/st21nfc u:object_r:nfc_device:s0 /dev/st54spi u:object_r:st54spi_device:s0 /dev/st33spi u:object_r:st33spi_device:s0 -/dev/ttyGS[0-3] u:object_r:serial_device:s0 /dev/logbuffer_tcpm u:object_r:logbuffer_device:s0 /dev/sys/block/bootdevice(/.*)? u:object_r:bootdevice_sysdev:s0 /dev/socket/chre u:object_r:chre_socket:s0 @@ -176,7 +174,6 @@ /dev/block/platform/13200000\.ufs/by-name/vendor_kernel_boot_[ab] u:object_r:custom_ab_block_device:s0 # Data -/data/vendor/slog(/.*)? u:object_r:vendor_slog_file:s0 /data/vendor/modem_stat(/.*)? u:object_r:modem_stat_data_file:s0 /data/vendor/log/hwc(/.*)? u:object_r:vendor_hwc_log_file:s0 /data/vendor/rild(/.*)? u:object_r:rild_vendor_data_file:s0 diff --git a/legacy/whitechapel_pro/hwservice.te b/legacy/whitechapel_pro/hwservice.te index 983e5a3f..5d7456ce 100644 --- a/legacy/whitechapel_pro/hwservice.te +++ b/legacy/whitechapel_pro/hwservice.te @@ -1,6 +1,3 @@ -# dmd servcie -type hal_vendor_oem_hwservice, hwservice_manager_type; - # GRIL service type hal_radioext_hwservice, hwservice_manager_type; diff --git a/legacy/whitechapel_pro/hwservice_contexts b/legacy/whitechapel_pro/hwservice_contexts index 0035ed49..9e820cf5 100644 --- a/legacy/whitechapel_pro/hwservice_contexts +++ b/legacy/whitechapel_pro/hwservice_contexts @@ -1,6 +1,3 @@ -# dmd HAL -vendor.samsung_slsi.telephony.hardware.oemservice::IOemService u:object_r:hal_vendor_oem_hwservice:s0 - # Fingerprint vendor.goodix.hardware.biometrics.fingerprint::IGoodixFingerprintDaemon u:object_r:hal_fingerprint_ext_hwservice:s0 diff --git a/legacy/whitechapel_pro/property.te b/legacy/whitechapel_pro/property.te index e817bb6a..1ca65bd6 100644 --- a/legacy/whitechapel_pro/property.te +++ b/legacy/whitechapel_pro/property.te @@ -1,6 +1,3 @@ -vendor_internal_prop(vendor_diag_prop) -vendor_internal_prop(vendor_slog_prop) -vendor_internal_prop(vendor_persist_config_default_prop) vendor_internal_prop(vendor_cbd_prop) vendor_internal_prop(vendor_carrier_prop) vendor_internal_prop(vendor_nfc_prop) diff --git a/legacy/whitechapel_pro/property_contexts b/legacy/whitechapel_pro/property_contexts index bf2e8886..b5f7fcdc 100644 --- a/legacy/whitechapel_pro/property_contexts +++ b/legacy/whitechapel_pro/property_contexts @@ -1,9 +1,3 @@ -# for dmd -persist.vendor.sys.dm. u:object_r:vendor_diag_prop:s0 -persist.vendor.sys.diag. u:object_r:vendor_diag_prop:s0 -vendor.sys.dmd. u:object_r:vendor_diag_prop:s0 -vendor.sys.diag. u:object_r:vendor_diag_prop:s0 - # Tcpdump_logger persist.vendor.tcpdump.log.alwayson u:object_r:vendor_tcpdump_log_prop:s0 vendor.tcpdump. u:object_r:vendor_tcpdump_log_prop:s0 @@ -12,19 +6,12 @@ vendor.tcpdump. u:object_r:vendor_tcpdump_log_prop:s0 persist.vendor.usb. u:object_r:vendor_usb_config_prop:s0 vendor.usb. u:object_r:vendor_usb_config_prop:s0 -# for slog -vendor.sys.silentlog. u:object_r:vendor_slog_prop:s0 -vendor.sys.exynos.slog. u:object_r:vendor_slog_prop:s0 -persist.vendor.sys.silentlog u:object_r:vendor_slog_prop:s0 - # for cbd vendor.cbd. u:object_r:vendor_cbd_prop:s0 persist.vendor.cbd. u:object_r:vendor_cbd_prop:s0 ro.vendor.config.build_carrier u:object_r:vendor_carrier_prop:s0 -persist.vendor.config. u:object_r:vendor_persist_config_default_prop:s0 - # test battery profile persist.vendor.testing_battery_profile u:object_r:vendor_battery_profile_prop:s0 diff --git a/legacy/whitechapel_pro/dmd.te b/vendor/dmd.te similarity index 100% rename from legacy/whitechapel_pro/dmd.te rename to vendor/dmd.te diff --git a/vendor/file.te b/vendor/file.te index 3d5739d5..60c26a87 100644 --- a/vendor/file.te +++ b/vendor/file.te @@ -1,9 +1,11 @@ # Data type vendor_log_file, file_type, data_file_type; type vendor_rfsd_log_file, file_type, data_file_type; +type vendor_slog_file, file_type, data_file_type; type radio_vendor_data_file, file_type, data_file_type; userdebug_or_eng(` typeattribute radio_vendor_data_file mlstrustedobject; + typeattribute vendor_slog_file mlstrustedobject; ') # Modem diff --git a/vendor/file_contexts b/vendor/file_contexts index ac39badf..6147047e 100644 --- a/vendor/file_contexts +++ b/vendor/file_contexts @@ -1,10 +1,12 @@ # Binaries /vendor/bin/rfsd u:object_r:rfsd_exec:s0 +/vendor/bin/modem_logging_control u:object_r:modem_logging_control_exec:s0 # Data /data/vendor/log/rfsd(/.*)? u:object_r:vendor_rfsd_log_file:s0 /data/vendor/radio(/.*)? u:object_r:radio_vendor_data_file:s0 /data/vendor/log(/.*)? u:object_r:vendor_log_file:s0 +/data/vendor/slog(/.*)? u:object_r:vendor_slog_file:s0 # vendor extra images /mnt/vendor/efs(/.*)? u:object_r:modem_efs_file:s0 @@ -15,6 +17,7 @@ /vendor/firmware(/.*)? u:object_r:vendor_fw_file:s0 # Devices +/dev/ttyGS[0-3] u:object_r:serial_device:s0 /dev/oem_ipc[0-7] u:object_r:radio_device:s0 /dev/umts_boot0 u:object_r:radio_device:s0 /dev/umts_ipc0 u:object_r:radio_device:s0 diff --git a/vendor/hwservice.te b/vendor/hwservice.te new file mode 100644 index 00000000..f6d18508 --- /dev/null +++ b/vendor/hwservice.te @@ -0,0 +1,3 @@ +# dmd servcie +type hal_vendor_oem_hwservice, hwservice_manager_type; + diff --git a/vendor/hwservice_contexts b/vendor/hwservice_contexts new file mode 100644 index 00000000..dd2a1fa6 --- /dev/null +++ b/vendor/hwservice_contexts @@ -0,0 +1,3 @@ +# dmd HAL +vendor.samsung_slsi.telephony.hardware.oemservice::IOemService u:object_r:hal_vendor_oem_hwservice:s0 + diff --git a/legacy/whitechapel_pro/modem_logging_control.te b/vendor/modem_logging_control.te similarity index 100% rename from legacy/whitechapel_pro/modem_logging_control.te rename to vendor/modem_logging_control.te diff --git a/vendor/property.te b/vendor/property.te index 29f53f12..68bb32a5 100644 --- a/vendor/property.te +++ b/vendor/property.te @@ -1,3 +1,6 @@ +vendor_internal_prop(vendor_slog_prop) +vendor_internal_prop(vendor_persist_config_default_prop) +vendor_internal_prop(vendor_diag_prop) vendor_internal_prop(vendor_modem_prop) vendor_internal_prop(vendor_rild_prop) vendor_internal_prop(vendor_ssrdump_prop) diff --git a/vendor/property_contexts b/vendor/property_contexts index f24f8db5..394172d7 100644 --- a/vendor/property_contexts +++ b/vendor/property_contexts @@ -1,3 +1,15 @@ +# for slog +vendor.sys.silentlog. u:object_r:vendor_slog_prop:s0 +vendor.sys.exynos.slog. u:object_r:vendor_slog_prop:s0 +persist.vendor.sys.silentlog u:object_r:vendor_slog_prop:s0 + +# for dmd +persist.vendor.sys.dm. u:object_r:vendor_diag_prop:s0 +persist.vendor.sys.diag. u:object_r:vendor_diag_prop:s0 +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 + # Modem persist.vendor.modem. u:object_r:vendor_modem_prop:s0 vendor.modem. u:object_r:vendor_modem_prop:s0