Commit 3549904d authored by Andy Paicu's avatar Andy Paicu Committed by Commit Bot

Revert "GWP-ASan: Add Linux crash handler support"

This reverts commit 70034734.

Reason for revert: crbug.com/962875

Original change's description:
> GWP-ASan: Add Linux crash handler support
> 
> Linux crashpad support is now sufficiently enabled to allow running the
> GWP-ASan crash handler unit tests under Linux. (The client tests are
> still blocked on allocator shim updates.)
> 
> Change-Id: I409cebaa9cdaed5da546266d0af470d2b25a09ab
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1610605
> Auto-Submit: Vlad Tsyrklevich <vtsyrklevich@chromium.org>
> Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
> Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#659296}

TBR=vtsyrklevich@chromium.org,vitalybuka@chromium.org

Change-Id: Idc53070073192a7a02260fe62c6a8514f2b1179c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1611599Reviewed-by: default avatarAndy Paicu <andypaicu@chromium.org>
Commit-Queue: Andy Paicu <andypaicu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#659439}
parent b09c0b86
......@@ -8,9 +8,9 @@ source_set("unit_tests") {
"//components/gwp_asan/common:unit_tests",
]
if (is_win || is_mac) {
deps += [ "//components/gwp_asan/client:unit_tests" ]
}
if (is_win || is_mac || is_linux) {
deps += [ "//components/gwp_asan/crash_handler:unit_tests" ]
deps += [
"//components/gwp_asan/client:unit_tests",
"//components/gwp_asan/crash_handler:unit_tests",
]
}
}
......@@ -8,7 +8,6 @@ static_library("crash_handler") {
sources = [
"crash_analyzer.cc",
"crash_analyzer.h",
"crash_analyzer_linux.cc",
"crash_analyzer_mac.cc",
"crash_analyzer_win.cc",
"crash_handler.cc",
......
// Copyright 2019 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 "components/gwp_asan/crash_handler/crash_analyzer.h"
#include <signal.h>
#include "third_party/crashpad/crashpad/snapshot/exception_snapshot.h"
namespace gwp_asan {
namespace internal {
crashpad::VMAddress CrashAnalyzer::GetAccessAddress(
const crashpad::ExceptionSnapshot& exception) {
if (exception.Exception() != SIGSEGV && exception.Exception() != SIGBUS)
return 0;
return exception.ExceptionAddress();
}
} // namespace internal
} // namespace gwp_asan
......@@ -270,15 +270,13 @@ class CrashHandlerTest : public base::MultiProcessTest,
EXPECT_TRUE(proto_.has_allocation());
EXPECT_TRUE(proto_.allocation().has_thread_id());
EXPECT_NE(proto_.allocation().thread_id(),
static_cast<uint64_t>(base::kInvalidThreadId));
EXPECT_NE(proto_.allocation().thread_id(), base::kInvalidThreadId);
EXPECT_GT(proto_.allocation().stack_trace_size(), 0);
EXPECT_EQ(proto_.has_deallocation(), has_deallocation);
if (has_deallocation) {
EXPECT_TRUE(proto_.deallocation().has_thread_id());
EXPECT_NE(proto_.deallocation().thread_id(),
static_cast<uint64_t>(base::kInvalidThreadId));
EXPECT_NE(proto_.deallocation().thread_id(), base::kInvalidThreadId);
EXPECT_EQ(proto_.allocation().thread_id(),
proto_.deallocation().thread_id());
EXPECT_GT(proto_.deallocation().stack_trace_size(), 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment