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

Unified Diff: syzygy/agent/asan/unittest_util.cc

Issue 2363733003: Make syzyasan_rtl compile in 64 bit (Closed)
Patch Set: Bug fixes Created 4 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: syzygy/agent/asan/unittest_util.cc
diff --git a/syzygy/agent/asan/unittest_util.cc b/syzygy/agent/asan/unittest_util.cc
index dbbaa065a282c3b024ea76511254b1805ff611d0..79e4378941d143392cc6069ee8ee31ace3892a83 100644
--- a/syzygy/agent/asan/unittest_util.cc
+++ b/syzygy/agent/asan/unittest_util.cc
@@ -185,6 +185,7 @@ void TestWithAsanLogger::SetUp() {
// Save the environment we found.
std::unique_ptr<base::Environment> env(base::Environment::Create());
env->GetVar(kSyzygyRpcInstanceIdEnvVar, &old_logger_env_);
+ env->GetVar(common::kSyzyAsanOptionsEnvVar, &old_asan_options_env_);
// Configure the environment (to pass the instance id to the agent DLL).
// We append "-0" to the process id to avoid potential conflict with other
@@ -193,6 +194,7 @@ void TestWithAsanLogger::SetUp() {
AppendToLoggerEnv(base::StringPrintf("%ls,%ls",
kSyzyAsanRtlDll,
instance_id_.c_str()));
+ env->UnSetVar(common::kSyzyAsanOptionsEnvVar);
log_contents_read_ = false;
StartLogger();
@@ -204,6 +206,7 @@ void TestWithAsanLogger::TearDown() {
// Restore the environment variable as we found it.
std::unique_ptr<base::Environment> env(base::Environment::Create());
env->SetVar(kSyzygyRpcInstanceIdEnvVar, old_logger_env_);
+ env->SetVar(common::kSyzyAsanOptionsEnvVar, old_asan_options_env_);
}
bool TestWithAsanLogger::LogContains(const base::StringPiece& message) {

Powered by Google App Engine
This is Rietveld 408576698