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

Issue 2632483002: VM: Avoid running atexit() handlers during exit() of a failed subprocess spawn (Closed)

Created:
3 years, 11 months ago by kustermann
Modified:
3 years, 11 months ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

VM: Avoid running atexit() handlers during exit() of a failed subprocess spawn Somtimes execve() fails and then we fall back to exit(1) in the fork()ed but not exeve()ed subprocess. One of the issues is e.g. that we don't want to run LSAN in the fork()ed but not execve()ed subprocess, the results will be bogus (plus the output of LSAN is not read on the other side (i.e. the unix pipe is not drained) which blocks the subprocess indefinitely). R=vegorov@google.com Committed: https://github.com/dart-lang/sdk/commit/5dc73cacd207ff07fb6055cee8370f1f2a9e01e7

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M runtime/bin/process_android.cc View 1 chunk +4 lines, -1 line 0 comments Download
M runtime/bin/process_linux.cc View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
kustermann
3 years, 11 months ago (2017-01-12 13:18:11 UTC) #2
Vyacheslav Egorov (Google)
lgtm
3 years, 11 months ago (2017-01-12 13:18:43 UTC) #3
kustermann
3 years, 11 months ago (2017-01-12 13:20:42 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5dc73cacd207ff07fb6055cee8370f1f2a9e01e7 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698