[Crash] SIGSEGV in CWindowGroupTarget::assignToSpace from ~CGroup during CCompositor::cleanup on exit with grouped windows (v0.56.2)

[sm-steel]

Summary

Hyprland segfaults on every exit (logout via uwsm stop) while grouped windows exist. The crash is in CCompositor::cleanup(): destroying a window destroys its CGroup, and ~CGroup() calls CWindowGroupTarget::assignToSpace() on a target whose space/window is already being torn down.

Side effect: because the compositor dies instead of exiting cleanly, every Chromium/Electron client aborts (SIGTRAP) and systemd-coredump writes their cores during the next login, which makes that login very slow.

Version

Hyprland 0.56.2 built from branch v0.56.2 at commit efb50993780079460b0cbed1363e2166a2de1d9f clean
aquamarine 0.15.1, hyprutils 0.14.2, hyprlang 0.6.8, mesa 26.2.4, linux 7.2.9 (Arch)

Lua config (HyDE). Two monitors, dwindle layout. Started through uwsm.

Steps to reproduce

1. Have a few tab groups open (here: 3–4 groups of 2–3 windows each, created by window rules group = "set" / "set always").
2. Exit Hyprland (uwsm stop, or hl.dispatch(hl.dsp.exec_cmd("uwsm stop")) from the hyprland.shutdown handler as HyDE does).
3. Hyprland segfaults in CCompositor::cleanup().

Reproduced 4 times in 3 days, every logout since the groups were set up. Closing all windows before exiting avoids it.

Backtrace (most frequent, 3 of 4)

#0  n/a (Hyprland + 0x1c32d2)
#1  Layout::CWindowGroupTarget::assignToSpace(Hyprutils::Memory::CSharedPointer<Layout::CSpace> const&, std::optional<Hyprutils::Math::Vector2D>)
#2  Desktop::View::CGroup::~CGroup()
#3  Hyprutils::Memory::CSharedPointer<Desktop::View::CGroup>::_delete(void*)
#4  Desktop::View::CWindow::~CWindow()
#5  Hyprutils::Memory::CSharedPointer<Desktop::View::CWindow>::_delete(void*)
#6  Desktop::CWindowState::clear()
#7  CCompositor::cleanup()
#8  main

Backtrace (variant, same path)

#0  CHyprDropShadowDecoration::updateWindow(Hyprutils::Memory::CSharedPointer<Desktop::View::CWindow>)
#1  Desktop::View::CWindow::updateWindowDecos()
#2  Layout::CWindowGroupTarget::onUpdateSpace()
#3  Layout::CWindowGroupTarget::assignToSpace(Hyprutils::Memory::CSharedPointer<Layout::CSpace> const&, std::optional<Hyprutils::Math::Vector2D>)
#4  Desktop::View::CGroup::~CGroup()
#5  Hyprutils::Memory::CSharedPointer<Desktop::View::CGroup>::_delete(void*)
#6  Desktop::View::CWindow::~CWindow()
#7  Hyprutils::Memory::CSharedPointer<Desktop::View::CWindow>::_delete(void*)
#8  Desktop::CWindowState::clear()
#9  CCompositor::cleanup()
#10 main

Looks like ~CGroup() shouldn't re-assign the group target to a space (or touch decorations) while the compositor is cleaning up. Happy to provide the full coredump info or test a patch.

[github-actions[bot]]

This discussion has been automatically closed.
Discussion format is NOT optional!

[github-actions[bot]]

⚠️ Strike 1/3 issued to @sm-steel by system.