Alexays / Waybar

Highly customizable Wayland bar for Sway and Wlroots based compositors. :v: :tada:
MIT License
6.65k stars 708 forks source link

Segfault when hovering, subsurface not mapped. #3542

Open esiefker opened 2 months ago

esiefker commented 2 months ago

I have a fresh install of Waybar on a fresh install of Sway on a fresh install of Debian Sid. That's Waybar v0.10.4, sway version 1.9, and fonts-font-awesome is installed.

Waybar reliably crashes when I hover over icons. I imagine there's a tool tip that's supposed to be displayed?

Here is the output when crashing, I'll post the full debug in an attached file.

(waybar:21744): Gdk-WARNING **: 12:45:06.914: Couldn't map window 0x557b78639550 as subsurface because its parent is not mapped.

(waybar:21744): Gdk-WARNING **: 12:45:06.914: Couldn't map window 0x557b78639550 as subsurface because its parent is not mapped.
Segmentation fault

waybar-debug.txt

rtripault commented 2 months ago

Confirmed here as well, setting "tooltip": false in each module configuration appears to prevent the crash

Vladimir-csp commented 2 months ago

This also happens when trying to open menus of tray icons, not only tooltips. Also happens on Hyprland on Debian sid. Sadly, I didn't catch what upgrade started it.

mntmn commented 2 months ago

Our users (on MNT Reform/Pocket Reform) also started experiencing this, and we tracked it down to a recent change in libgtk-3, rolling back a certain patch for libgtk-3 fixed it for us, but idk if the root cause needs to be addressed in waybar: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1079292

josch commented 2 months ago

Issue reported for gtk here: https://gitlab.gnome.org/GNOME/gtk/-/issues/6958

esiefker commented 2 months ago

Oh Debian... Thanks for tracking that down.

jbicha commented 2 months ago

GTK has closed the bug saying that it needs to be fixed in waybar.

Note that the change that triggered this issue hasn't yet made it into a stable gtk3 3.24 release but it's expected to happen eventually.

I'll revert this change in Debian for now since Debian was only cherry-picking the change early. (This only affected Debian Unstable and distros based on that; it didn't reach Debian Testing or Ubuntu or Debian Stable.)

alebastr commented 2 months ago

I'll revert this change in Debian for now since Debian was only cherry-picking the change early. (This only affected Debian Unstable and distros based on that; it didn't reach Debian Testing or Ubuntu or Debian Stable.)

Thanks for reverting the patch. I'm afraid that you'll have to wait for the next GTK release to get that functionality in Debian, and then ensure that gtk-layer-shell is updated or patched to support it.

Two things worth knowing here:

I filed https://github.com/wmww/gtk-layer-shell/issues/187 to notify the maintainer about the future breakage.