summaryrefslogtreecommitdiffstats
path: root/ashmemd_client.cpp
diff options
context:
space:
mode:
authorTri Vo <trong@google.com>2019-03-25 20:17:19 +0000
committerTri Vo <trong@google.com>2019-03-25 20:17:19 +0000
commit2fd4b8732ccc5f852728c61413a91e2704d46f46 (patch)
treea6de32c09aa4203a5f196fc8f536c9bdb712ecf1 /ashmemd_client.cpp
parent41b7c708217752a4b30cc022b5917957274c47e2 (diff)
downloadplatform_system_ashmemd-2fd4b8732ccc5f852728c61413a91e2704d46f46.tar.gz
platform_system_ashmemd-2fd4b8732ccc5f852728c61413a91e2704d46f46.tar.bz2
platform_system_ashmemd-2fd4b8732ccc5f852728c61413a91e2704d46f46.zip
Revert "Check /dev/binder access before calling defaultServiceManager()"
This reverts commit 41b7c708217752a4b30cc022b5917957274c47e2. Reason for revert: boot denial on sailfish/marlin Change-Id: I3dad32c3b07bcab0a0bfffb9a06d66f767af7391
Diffstat (limited to 'ashmemd_client.cpp')
-rw-r--r--ashmemd_client.cpp5
1 files changed, 0 insertions, 5 deletions
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 android {
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);