From 2654c8ed2b1b6dc025185eb5cf513e25a38c4595 Mon Sep 17 00:00:00 2001 From: Nick Chalko Date: Thu, 21 Feb 2019 09:18:49 -0800 Subject: [PATCH] Turn on extcon event observing selinux policy now includes sysfs_extcon Submit after https://android-review.googlesource.com/c/platform/system/sepolicy/+/907257 is merged Bug: 124364409 Test: m services.core Change-Id: I5401e36b5e752387cf7794dcca55da2270d46f66 --- .../java/com/android/server/ExtconUEventObserver.java | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/services/core/java/com/android/server/ExtconUEventObserver.java b/services/core/java/com/android/server/ExtconUEventObserver.java index 775e4c8cf4ed0..6b42b3d06d4be 100644 --- a/services/core/java/com/android/server/ExtconUEventObserver.java +++ b/services/core/java/com/android/server/ExtconUEventObserver.java @@ -162,15 +162,6 @@ public abstract class ExtconUEventObserver extends UEventObserver { /** Does the {@link /sys/class/extcon} directory exist */ public static boolean extconExists() { File extconDir = new File("/sys/class/extcon"); - boolean retVal = extconDir.exists() && extconDir.isDirectory(); - // TODO(b/124364409): return the correct value after selinux policy is updated. - if (retVal) { - Slog.w(TAG, extconDir + " exists " + extconDir.exists() + " isDir " - + extconDir.isDirectory() - + " but reporting it does not exist until selinux policies are updated." - + " see b/124364409" - ); - } - return false; + return extconDir.exists() && extconDir.isDirectory(); } }