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

Issue 164373005: Revert 251265 "Setuid sandbox: exit(2) on SIGABRT" (Closed)

Created:
6 years, 10 months ago by jln (very slow on Chromium)
Modified:
6 years, 10 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 251265 "Setuid sandbox: exit(2) on SIGABRT" > Setuid sandbox: exit(2) on SIGABRT > > The setuid sandbox waits on its one child and then dies > afterwards. > > When receiving SIGABRT, instead of dumping core, simply exit the > process. There is no interesting information to be gathered from > knowing that the process is inside waitid(2), one should look at > the child process instead. > > This patch hopes to reduce red herrings. > > BUG=334345 > NOTRY=true > R=jorgelo@chromium.org > > Review URL: https://codereview.chromium.org/165673002 TBR=jln@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251269

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -15 lines) Patch
M trunk/src/sandbox/linux/suid/sandbox.c View 2 chunks +0 lines, -15 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jln (very slow on Chromium)
6 years, 10 months ago (2014-02-14 07:30:58 UTC) #1
jln (very slow on Chromium)
6 years, 10 months ago (2014-02-14 07:31:05 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r251269 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698