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

Unified Diff: src/trusted/platform_qualify/arch/arm/nacl_dep_qualify_arch.c

Issue 3539011: Enable check for DEP / NX page protection (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client/
Patch Set: Modify windows check to use Noel's signal interface Created 10 years, 2 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: src/trusted/platform_qualify/arch/arm/nacl_dep_qualify_arch.c
===================================================================
--- src/trusted/platform_qualify/arch/arm/nacl_dep_qualify_arch.c (revision 0)
+++ src/trusted/platform_qualify/arch/arm/nacl_dep_qualify_arch.c (revision 0)
@@ -0,0 +1,49 @@
+/*
+ * Copyright 2010 The Native Client 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 <stddef.h>
+#include <stdint.h>
+#include "native_client/src/trusted/platform_qualify/nacl_dep_qualify.h"
+#include "native_client/src/include/nacl_macros.h"
+
+/* Assembled equivalent of "bx lr" */
+#define INST_BX_LR 0xE12FFF1E
+
+int NaClCheckDEP() {
+ /*
+ * We require DEP, so forward this call to the OS-specific check routine.
+ */
+ /* TODO(cbiffle) or (jvoung): enable this when ARM HW test bots have
+ * an upgraded kernel that does not have "the" signal handler bug.
+ * http://code.google.com/p/nativeclient/issues/detail?id=969
+ *
+ return NaClAttemptToExecuteData();
+ */
+ return 1;
+}
+
+nacl_void_thunk NaClGenerateThunk(char *buf, size_t size_in_bytes) {
+ /*
+ * Place a "bx lr" at the next aligned address after buf. Instructions
+ * are always little-endian, regardless of data setting.
+ */
+ char *aligned_buf = (char *) (((uintptr_t) buf + 3) & ~3);
+
+ if (aligned_buf + 4 > buf + size_in_bytes) return 0;
+
+ aligned_buf[0] = (char) (INST_BX_LR >> 0);
+ aligned_buf[1] = (char) (INST_BX_LR >> 8);
+ aligned_buf[2] = (char) (INST_BX_LR >> 16);
+ aligned_buf[3] = (char) (INST_BX_LR >> 24);
+
+ /*
+ * ISO C prevents a direct data->function cast, because the pointers aren't
+ * guaranteed to be the same size. For our platforms this is fine, but we
+ * verify at compile time anyway before tricking the compiler:
+ */
+ NACL_ASSERT_SAME_SIZE(char *, nacl_void_thunk);
+ return (nacl_void_thunk) (uintptr_t) aligned_buf;
+}
Property changes on: src/trusted/platform_qualify/arch/arm/nacl_dep_qualify_arch.c
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « src/trusted/platform_qualify/arch/arm/nacl_dep_qualify.c ('k') | src/trusted/platform_qualify/arch/x86_32/nacl_dep_qualify.c » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698