Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(656)

Unified Diff: third_party/crazy_linker/crazy_linker/src/crazy_linker_ashmem.cpp

Issue 23717023: Android: Add chrome-specific dynamic linker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase to fix build. Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/crazy_linker/crazy_linker/src/crazy_linker_ashmem.cpp
diff --git a/third_party/crazy_linker/crazy_linker/src/crazy_linker_ashmem.cpp b/third_party/crazy_linker/crazy_linker/src/crazy_linker_ashmem.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..4bcb9e3e76beb523003a6beb12c1e3513c7a45ce
--- /dev/null
+++ b/third_party/crazy_linker/crazy_linker/src/crazy_linker_ashmem.cpp
@@ -0,0 +1,89 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "crazy_linker_ashmem.h"
+
+#include <fcntl.h>
+#include <string.h>
+#include <sys/ioctl.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+#include <unistd.h>
+
+#include <linux/ashmem.h>
+
+#include "crazy_linker_system.h"
+#include "crazy_linker_memory_mapping.h"
+
+namespace crazy {
+
+bool AshmemRegion::Allocate(size_t region_size, const char* region_name) {
+ int fd = TEMP_FAILURE_RETRY(open("/dev/ashmem", O_RDWR));
+ if (fd < 0)
+ return false;
+
+ if (ioctl(fd, ASHMEM_SET_SIZE, region_size) < 0)
+ goto ERROR;
+
+ if (region_name) {
+ char buf[256];
+ strlcpy(buf, region_name, sizeof(buf));
+ if (ioctl(fd, ASHMEM_SET_NAME, buf) < 0)
+ goto ERROR;
+ }
+
+ Reset(fd);
+ return true;
+
+ERROR:
+ ::close(fd);
+ return false;
+}
+
+bool AshmemRegion::SetProtectionFlags(int prot) {
+ return ioctl(fd_, ASHMEM_SET_PROT_MASK, prot) == 0;
+}
+
+// static
+bool AshmemRegion::CheckFileDescriptorIsReadOnly(int fd) {
+ const size_t map_size = PAGE_SIZE;
+ ScopedMemoryMapping map;
+
+ // First, check that trying to map a page of the region with PROT_WRITE
+ // fails with EPERM.
+ if (map.Allocate(NULL, map_size, MemoryMapping::CAN_WRITE, fd)) {
+ LOG("%s: Region could be mapped writable. Should not happen.\n",
+ __FUNCTION__);
+ errno = EPERM;
+ return false;
+ }
+ if (errno != EPERM) {
+ LOG_ERRNO("%s: Region failed writable mapping with unexpected error",
+ __FUNCTION__);
+ return false;
+ }
+
+ // Second, check that it can be mapped PROT_READ, but cannot be remapped
+ // with PROT_READ | PROT_WRITE through mprotect().
+ if (!map.Allocate(NULL, map_size, MemoryMapping::CAN_READ, fd)) {
+ LOG_ERRNO("%s: Failed to map region read-only", __FUNCTION__);
+ return false;
+ }
+ if (map.SetProtection(MemoryMapping::CAN_READ_WRITE)) {
+ LOG_ERRNO("%s: Region could be remapped read-write. Should not happen.\n",
+ __FUNCTION__);
+ return false;
+ }
+ if (errno != EACCES) {
+ LOG_ERRNO(
+ "%s: Region failed to be remapped read-write with unexpected error",
+ __FUNCTION__);
+ return false;
+ }
+
+ // Everything's good.
+ return true;
+}
+
+} // namespace crazy

Powered by Google App Engine
This is Rietveld 408576698