From 3b1c0a8e4685d6de33eda1ecd1c0634153b94f5c Mon Sep 17 00:00:00 2001 From: Tri Vo Date: Thu, 28 Mar 2019 21:25:00 +0000 Subject: Revert "Reland 'Check /dev/binder access before calling defaultServiceManager()'" This reverts commit a526f3d41640506ebd457c2ed91b923d6f187d0b. Reason for revert: b/129483782 Isolated apps don't have seccomp permissions to access() syscall. CTS seccomp test is broken. Change-Id: I90ddf2016c8d2e433d2c8a4f8dd5e8563104533e --- ashmemd_client.cpp | 5 ----- 1 file changed, 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 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 sm = android::defaultServiceManager(); sp binder = sm->checkService(String16("ashmem_device_service")); return interface_cast(binder); -- cgit v1.2.3