diff options
| author | Michael Brüning <michael.bruning@theqtcompany.com> | 2015-02-26 17:17:36 +0100 |
|---|---|---|
| committer | Allan Sandfeld Jensen <allan.jensen@theqtcompany.com> | 2015-06-23 14:51:51 +0000 |
| commit | 49034db5384c062644ffca5348d1f3d2436a74dc (patch) | |
| tree | 1ab4e72f1de09af451946c52c24cef886b440fd4 | |
| parent | 8b36ffb7fbdb0599e2a803a417b32a2d18c7a8d5 (diff) | |
| download | qtwebengine-chromium-49034db5384c062644ffca5348d1f3d2436a74dc.tar.gz | |
<chromium> Always ignore SIGPIPE.
Conditionally running SetupSignalHandlers in the content main runner
led to crashes when SIGPIPE was not ignored.
Task-number: QTBUG-44689
Change-Id: I0eddf52b9f760a23cc66ea5079b87f80d688cbdb
Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>
| -rw-r--r-- | chromium/content/app/content_main_runner.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/chromium/content/app/content_main_runner.cc b/chromium/content/app/content_main_runner.cc index 1fe3b15735a..20d45d2b4db 100644 --- a/chromium/content/app/content_main_runner.cc +++ b/chromium/content/app/content_main_runner.cc @@ -520,6 +520,9 @@ class ContentMainRunnerImpl : public ContentMainRunner { if (params.setup_signal_handlers) { SetupSignalHandlers(); + } else { + // Ignore SIGPIPE even if we are not resetting the other signal handlers. + CHECK(signal(SIGPIPE, SIG_IGN) != SIG_ERR); } g_fds->Set(kPrimaryIPCChannel, |