commit 7bf526d2ef28103444b740f0a27f6863ff302461
author Tri Vo <trong@google.com> Thu Mar 28 16:00:18 2019 -0700
committer android-build-merger <android-build-merger@google.com> Thu Mar 28 16:00:18 2019 -0700
tree 695f9adde527b12062f6a9afc9c0cbc402de230e
parent 39d57cb5cb3db0ce9078b98fd92d4b6dfb57fbe1
parent 2e3f50fd678a65578e9ab1862ddb21d6ff7b8aa8

Merge "Revert "Reland 'Check /dev/binder access before calling defaultServiceManager()'"" am: eb512b6888 am: 93bbcd8c46
am: 2e3f50fd67

Change-Id: Ie67fd8d129434c5c42e1e9179d36195766ed886b

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);