Bug 206052 – Error sending IPC message: Broken pipe

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[crvi]

Get a bunch of these when running epiphany ( Gnome Web ) from command line:

sid@unstable:~$ epiphany
WaylandCompositor requires eglBindWaylandDisplayWL, eglUnbindWaylandDisplayWL and eglQueryWaylandBuffer.
Nested Wayland compositor could not initialize EGL

(WebKitWebProcess:3): Gdk-WARNING **: 08:26:13.891: Failed to read portal settings: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.portal.Desktop was not provided by any .service files

...
...

(WebKitWebProcess:3): Gdk-WARNING **: 08:26:15.126: Failed to read portal settings: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.portal.Desktop was not provided by any .service files

Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe
Error sending IPC message: Broken pipe

[crvi]

sid@unstable:~$ epiphany --version
Web 3.34.1
Powered by WebKitGTK 2.26.2

[Charlie Turner]

Created attachment 400058 [details]
Patch

[Charlie Turner]

I came here while working on tests for https://bugs.webkit.org/show_bug.cgi?id=184845 , where I was seeing these IPC error messages as well (plus ones in the IPC receive side)

[Adrian Perez]

Comment on attachment 400058 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=400058&action=review

> Source/WebKit/Platform/IPC/unix/ConnectionUnix.cpp:525
> +        if (errno == EPIPE || errno == ECONNRESET) {

I would put the EPIPE check inside an OS(LINUX) guard, as it is clearly
something that is specific to the Linux kernel.

  #if OS(LINUX
      if (errno == ECONNRESET || errno == EPIPE)
  #else
      if (errno == ECONRESET)
  #endif
      {
          // ...
      }

[Charlie Turner]

Created attachment 400118 [details]
Patch

Oops, I misunderstood ConnectionUnix has being ConnectionLinux, corrected!

[Charlie Turner]

Comment on attachment 400118 [details]
Patch

Accidentally cleared the r+ flag, resetting and cq+'ing. Hopefully the scripts Do The Right Thing.

[EWS]

[e-mail] does not have reviewer permissions according to https://trac.webkit.org/browser/webkit/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

Rejecting attachment 400118 [details] from commit queue.

[Charlie Turner]

Created attachment 400123 [details]
Patch

Manually set reviewer and resubmit

[EWS]

Committed r262099: <https://trac.webkit.org/changeset/262099>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 400123 [details].