From 78c16642a4539a991189d4cc4106fc326f646a83 Mon Sep 17 00:00:00 2001 From: Joel Galenson Date: Tue, 30 Jan 2018 10:31:30 -0800 Subject: msm8996-common: Fix thermal-engine dac_override denial. We occasionally see a dac_override SELinux denial from thermal-engine trying to access /dev/diag. This seems to be a race between thermal-engine reading the file and init setting its permissions to 666 (they are initially set to 660 and in the diag group). This commit adds thermal-engine to the diag group so that it can read the file even if it is not world-readable. Bug: 72458250 Test: Boot and test wifi and camera. Change-Id: I55adfcf731a5da3ba7bb52d957d64e8e1f26febb --- rootdir/etc/init.qcom.rc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rootdir/etc/init.qcom.rc b/rootdir/etc/init.qcom.rc index b21bb77..0b06ca9 100644 --- a/rootdir/etc/init.qcom.rc +++ b/rootdir/etc/init.qcom.rc @@ -516,7 +516,7 @@ service thermal-engine /vendor/bin/thermal-engine -c /vendor/etc/thermal-engine. socket thermal-recv-client stream 0660 system system socket thermal-recv-passive-client stream 0666 system system socket thermal-send-rule stream 0660 system system - group root + group system root diag socket thermal-send-client stream 0666 system system socket thermal-recv-client stream 0660 system system socket thermal-recv-passive-client stream 0666 system system -- cgit v1.2.3