[Webkit-unassigned] [Bug 220090] New: [WPE][GTK] Remove unnecessary child setup function from process launcher

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=220090

            Bug ID: 220090
           Summary: [WPE][GTK] Remove unnecessary child setup function
                    from process launcher
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKitGTK
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: mcatanzaro at gnome.org
                CC: bugs-noreply at webkitgtk.org

The process launcher currently uses a child setup function to close the server end of its IPC socket. But this is totally unnecessary because this socket always uses CLOEXEC. So the child setup function is totally redundant.

This is step one towards making Eclipse not crash when its UI process JVM is using a huge amount of memory. In such conditions, fork() will fail due to OOM, but posix_spawn() will still succeed. A child setup function forces GLib to use fork() instead of posix_spawn(). However, this commit is not sufficient to fix Eclipse, because GSubprocess itself sets its own child setup function.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20201222/427eb81d/attachment.htm>