Revision: 21275
Author: [email protected]
Date: Mon May 12 20:10:47 2014 UTC
Log: Revert "v8::TryCatch now works correctly with ASAN's
UseAfterReturn mode enabled."
This reverts commit r21273.
[email protected]
Review URL: https://codereview.chromium.org/284493003
http://code.google.com/p/v8/source/detail?r=21275
Modified:
/branches/bleeding_edge/include/v8.h
/branches/bleeding_edge/src/api.cc
/branches/bleeding_edge/src/base/macros.h
/branches/bleeding_edge/src/isolate.cc
/branches/bleeding_edge/src/zone.h
=======================================
--- /branches/bleeding_edge/include/v8.h Mon May 12 19:32:12 2014 UTC
+++ /branches/bleeding_edge/include/v8.h Mon May 12 20:10:47 2014 UTC
@@ -5076,8 +5076,6 @@
void SetCaptureMessage(bool value);
private:
- TryCatch* DesanitizedThis();
-
// Make it hard to create heap-allocated TryCatch blocks.
TryCatch(const TryCatch&);
void operator=(const TryCatch&);
@@ -5089,7 +5087,6 @@
void* exception_;
void* message_obj_;
void* message_script_;
- void* asan_fake_stack_handle_;
int message_start_pos_;
int message_end_pos_;
bool is_verbose_ : 1;
=======================================
--- /branches/bleeding_edge/src/api.cc Mon May 12 19:32:12 2014 UTC
+++ /branches/bleeding_edge/src/api.cc Mon May 12 20:10:47 2014 UTC
@@ -6,9 +6,6 @@
#include <string.h> // For memcpy, strlen.
#include <cmath> // For isnan.
-#ifdef V8_USE_ADDRESS_SANITIZER
-#include <sanitizer/asan_interface.h>
-#endif // V8_USE_ADDRESS_SANITIZER
#include "../include/v8-debug.h"
#include "../include/v8-profiler.h"
#include "../include/v8-testing.h"
@@ -1820,16 +1817,6 @@
isolate_->UnregisterTryCatchHandler(this);
}
}
-
-
-v8::TryCatch* v8::TryCatch::DesanitizedThis() {
-#ifdef V8_USE_ADDRESS_SANITIZER
- return TRY_CATCH_FROM_ADDRESS(
- __asan_addr_is_in_fake_stack(asan_fake_stack_handle_, this, NULL,
NULL));
-#else
- return this;
-#endif
-}
bool v8::TryCatch::HasCaught() const {
@@ -1906,11 +1893,6 @@
message_script_ = the_hole;
message_start_pos_ = 0;
message_end_pos_ = 0;
-#ifdef V8_USE_ADDRESS_SANITIZER
- asan_fake_stack_handle_ = __asan_get_current_fake_stack();
-#else
- asan_fake_stack_handle_ = NULL;
-#endif
}
=======================================
--- /branches/bleeding_edge/src/base/macros.h Mon May 12 19:32:12 2014 UTC
+++ /branches/bleeding_edge/src/base/macros.h Mon May 12 20:10:47 2014 UTC
@@ -54,17 +54,15 @@
#define MUST_USE_RESULT V8_WARN_UNUSED_RESULT
-// Define V8_USE_ADDRESS_SANITIZER macros.
+// Define DISABLE_ASAN macros.
#if defined(__has_feature)
#if __has_feature(address_sanitizer)
-#define V8_USE_ADDRESS_SANITIZER 1
+#define DISABLE_ASAN __attribute__((no_sanitize_address))
#endif
#endif
-// Define DISABLE_ASAN macros.
-#ifdef V8_USE_ADDRESS_SANITIZER
-#define DISABLE_ASAN __attribute__((no_sanitize_address))
-#else
+
+#ifndef DISABLE_ASAN
#define DISABLE_ASAN
#endif
=======================================
--- /branches/bleeding_edge/src/isolate.cc Mon May 12 19:32:12 2014 UTC
+++ /branches/bleeding_edge/src/isolate.cc Mon May 12 20:10:47 2014 UTC
@@ -272,14 +272,13 @@
// JS stack. When running without the simulator, the address
// returned will be the address of the C++ try catch handler itself.
Address address = reinterpret_cast<Address>(
- SimulatorStack::RegisterCTryCatch(reinterpret_cast<uintptr_t>(
- that->DesanitizedThis())));
+
SimulatorStack::RegisterCTryCatch(reinterpret_cast<uintptr_t>(that)));
thread_local_top()->set_try_catch_handler_address(address);
}
void Isolate::UnregisterTryCatchHandler(v8::TryCatch* that) {
- ASSERT(thread_local_top()->TryCatchHandler() == that->DesanitizedThis());
+ ASSERT(thread_local_top()->TryCatchHandler() == that);
thread_local_top()->set_try_catch_handler_address(
reinterpret_cast<Address>(that->next_));
thread_local_top()->catcher_ = NULL;
=======================================
--- /branches/bleeding_edge/src/zone.h Mon May 12 19:32:12 2014 UTC
+++ /branches/bleeding_edge/src/zone.h Mon May 12 20:10:47 2014 UTC
@@ -15,6 +15,11 @@
namespace v8 {
namespace internal {
+#if defined(__has_feature)
+ #if __has_feature(address_sanitizer)
+ #define V8_USE_ADDRESS_SANITIZER
+ #endif
+#endif
class Segment;
class Isolate;
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
