From 5b1c33e45f84c34e1a78796a8550f17abf989a6d Mon Sep 17 00:00:00 2001 From: Max Bires Date: Fri, 20 Oct 2017 10:28:24 -0700 Subject: [PATCH] Removing stats service registration in system server Introduced in ag/2837473, this stats service is not actually up and running fully. It is creating boot time selinux denials due to lack of permissions for system server to start the service. Since it cannot run due to system server being unable to register it with servicemanager, the registration code is being removed for now to clean up boot denials. Removing this boot denial will allow boot-time presubmit tests for SELinux to be put into place. Service definition and sepolicy for this feature can be added when the feature is approved in the future. Bug: 67468181 Test: servicemanager does not generate denials on boot Change-Id: Id6a29cc6abb8ebf5a3f41da4311a555f41dd80b3 --- services/java/com/android/server/SystemServer.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/services/java/com/android/server/SystemServer.java b/services/java/com/android/server/SystemServer.java index b78fcddee1197..d9db22e9b62d2 100644 --- a/services/java/com/android/server/SystemServer.java +++ b/services/java/com/android/server/SystemServer.java @@ -1551,11 +1551,6 @@ public final class SystemServer { traceEnd(); } - // Statsd helper - traceBeginAndSlog("StartStatsCompanionService"); - mSystemServiceManager.startService(StatsCompanionService.Lifecycle.class); - traceEnd(); - // Before things start rolling, be sure we have decided whether // we are in safe mode. final boolean safeMode = wm.detectSafeMode();