[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 David Edmundson changed: What|Removed |Added CC||k...@davidedmundson.co.uk Summary|kwin_wayland explodes on|kwin_wayland crashes on |screen setup change |screen setup change -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 David Edmundson changed: What|Removed |Added Keywords||wayland --- Comment #1 from David Edmundson --- OutputDeviceModeV2InterfacePrivate::findResource can return a nullptr, guarding is trivial, but it implies that someone is calling setCurrentMode before setModes which should be fixed at the root. -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318
David Edmundson changed:
What|Removed |Added
Ever confirmed|0 |1
Status|REPORTED|CONFIRMED
--- Comment #2 from David Edmundson ---
Edit: It doesn't imply that. We guard that already.
It implies we have a unbound mode for a bound output.
Can you try patching libkscreen:
WaylandOutputDevice::~WaylandOutputDevice()
{
qDeleteAll(m_modes);
kde_output_device_v2_destroy(object());
}
and swap these two lines so we tear down the device first.
--
You are receiving this mail because:
You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 --- Comment #3 from Harald Sitter --- Alas, makes no difference. -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 Nate Graham changed: What|Removed |Added CC||n...@kde.org Severity|normal |crash -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 --- Comment #4 from Vlad Zahorodnii --- Git commit 33df55c5a35fff6dbea59c2143f68eb38eca7f57 by Vlad Zahorodnii. Committed on 08/04/2022 at 08:32. Pushed by vladz into branch 'master'. Add Q_ASSERT in OutputDeviceV2Interface::setCurrentMode() M +1-0src/server/outputdevice_v2_interface.cpp https://invent.kde.org/plasma/kwayland-server/commit/33df55c5a35fff6dbea59c2143f68eb38eca7f57 -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 Vlad Zahorodnii changed: What|Removed |Added Resolution|--- |FIXED Status|CONFIRMED |RESOLVED Latest Commit||https://invent.kde.org/plas ||ma/kwayland-server/commit/b ||40c4e3f4108b9a0be9b4f5d437e ||72f7c2aa3fea --- Comment #5 from Vlad Zahorodnii --- Git commit b40c4e3f4108b9a0be9b4f5d437e72f7c2aa3fea by Vlad Zahorodnii. Committed on 08/04/2022 at 12:56. Pushed by vladz into branch 'master'. Simplify code that announces available modes and current mode If no mode has the current flag set, the first mode object will be announced twice. There's also no benefit from sending the current mode as last. If the current mode changes, the compositor will send the current_mode event, not the mode event. M +11 -24 src/server/outputdevice_v2_interface.cpp https://invent.kde.org/plasma/kwayland-server/commit/b40c4e3f4108b9a0be9b4f5d437e72f7c2aa3fea -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 Vlad Zahorodnii changed: What|Removed |Added Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas |ma/kwayland-server/commit/b |ma/kwayland-server/commit/4 |40c4e3f4108b9a0be9b4f5d437e |0364d2ee670a5a74318ef96c643 |72f7c2aa3fea|762293ca95f0 --- Comment #6 from Vlad Zahorodnii --- Git commit 40364d2ee670a5a74318ef96c643762293ca95f0 by Vlad Zahorodnii. Committed on 08/04/2022 at 16:40. Pushed by vladz into branch 'Plasma/5.24'. Simplify code that announces available modes and current mode If no mode has the current flag set, the first mode object will be announced twice. There's also no benefit from sending the current mode as last. If the current mode changes, the compositor will send the current_mode event, not the mode event. (cherry picked from commit b40c4e3f4108b9a0be9b4f5d437e72f7c2aa3fea) M +11 -24 src/server/outputdevice_v2_interface.cpp https://invent.kde.org/plasma/kwayland-server/commit/40364d2ee670a5a74318ef96c643762293ca95f0 -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 Nate Graham changed: What|Removed |Added Version Fixed In||5.24.5 -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 --- Comment #7 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwayland-server/-/merge_requests/365 -- You are receiving this mail because: You are watching all bug changes.
[kwin] [Bug 452318] kwin_wayland crashes on screen setup change
https://bugs.kde.org/show_bug.cgi?id=452318 --- Comment #8 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwayland-server/-/merge_requests/367 -- You are receiving this mail because: You are watching all bug changes.
