commit 2fd4b8732ccc5f852728c61413a91e2704d46f46
author Tri Vo <trong@google.com> Mon Mar 25 20:17:19 2019 +0000
committer Tri Vo <trong@google.com> Mon Mar 25 20:17:19 2019 +0000
tree a6de32c09aa4203a5f196fc8f536c9bdb712ecf1
parent 41b7c708217752a4b30cc022b5917957274c47e2

Revert "Check /dev/binder access before calling defaultServiceManager()"

This reverts commit 41b7c708217752a4b30cc022b5917957274c47e2.

Reason for revert: boot denial on sailfish/marlin

Change-Id: I3dad32c3b07bcab0a0bfffb9a06d66f767af7391

ashmemd_client.cpp

diff --git a/ashmemd_client.cpp b/ashmemd_client.cpp
index 04227ba..3380209 100644
--- a/ashmemd_client.cpp
+++ b/ashmemd_client.cpp
@@ -28,11 +28,6 @@
 namespace ashmemd {
 
 sp<IAshmemDeviceService> getAshmemService() {
-    // Calls to defaultServiceManager() crash the process if it doesn't have appropriate
-    // binder permissions. Check these permissions proactively.
-    if (access("/dev/binder", R_OK | W_OK) != 0) {
-        return nullptr;
-    }
     sp<IServiceManager> sm = android::defaultServiceManager();
     sp<IBinder> binder = sm->checkService(String16("ashmem_device_service"));
     return interface_cast<IAshmemDeviceService>(binder);