New issue
Advanced search Search tips

Issue 911834 link

Starred by 1 user
Status: Fixed
Owner:
jperaza@chromium.org
Closed: Dec 14
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

DumpWithoutCrashing() crashes non-browser processes

Project Member Reported by jperaza@google.com, Dec 4 
Chrome Version: 73.0.3630.0
OS: android

Calls to DumpWithoutCrashing() should produce a dump without crashing.

DumpWithoutCrashing() with Crashpad is attempting to call CrashpadClient.DumpWithoutCrash(), which segfaults because the real
signal handler is the Chromium-specific SandboxedHandler:
https://cs.chromium.org/chromium/src/components/crash/content/app/crashpad_linux.cc?rcl=03f011b451f7d4d9b879d0403a6c3446b5831c7a&l=83
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 6 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6b424b920a9170a35eb79ffb283078f4598f6716

commit 6b424b920a9170a35eb79ffb283078f4598f6716
Author: Joshua Peraza <jperaza@chromium.org>
Date: Thu Dec 06 18:45:00 2018

android: Implement DumpWithoutCrashing() from child processes

Bug:  911834 
Change-Id: I1901745a79f40748791d5c9a0dee1f45fb7ca88a
Reviewed-on: https://chromium-review.googlesource.com/c/1362189
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Joshua Peraza <jperaza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614427}
[modify] https://crrev.com/6b424b920a9170a35eb79ffb283078f4598f6716/components/crash/content/app/crashpad.cc
[modify] https://crrev.com/6b424b920a9170a35eb79ffb283078f4598f6716/components/crash/content/app/crashpad_linux.cc
[modify] https://crrev.com/6b424b920a9170a35eb79ffb283078f4598f6716/components/crash/content/browser/crash_handler_host_linux.cc

Comment 2 by jperaza@chromium.org, Dec 14

Status: Fixed (was: Started)
►

Sign in to add a comment