Bug 462377 - Skrooge flatpak crashes in Wayland after invoking file import, setGrabPopup warning
Summary: Skrooge flatpak crashes in Wayland after invoking file import, setGrabPopup w...
Status: RESOLVED FIXED
Alias: None
Product: skrooge
Classification: Applications
Component: general (show other bugs)
Version: Trunk from git
Platform: Flatpak Linux
: NOR crash
Target Milestone: 25.1.0
Assignee: Stephane MANKOWSKI
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-29 01:12 UTC by skierpage
Modified: 2023-02-08 14:47 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description skierpage 2022-11-29 01:12:40 UTC
SUMMARY
I've experienced errors importing transactions into Skrooge (because I lost my import CSV fields definition, my fault). When I try File > Import > Import... a second time, the skrooge flatpak under Wayland has crashed three times.

STEPS TO REPRODUCE
1. Run skrooge
2. File > Import > Import... and choose a bad .csv file
3. Dismiss the error message, and/or maximize Skrooge's messages panel, and/or ??? The exact steps aren't clear.
4. Retry File > Import > Import...

OBSERVED RESULT
The file dialog may appear, but the Skrooge window goes unresponsive and crashes. One time the menus and submenus remained on-screen briefly after the skrooge window went away.
Nearly every time in the terminal where I ran the flatpak I see
        ##WARNING: setGrabPopup called with a parent, QtWaylandClient::QWaylandXdgSurface(0x563fbe55d100) which does not match the current topmost grabbing popup, QtWaylandClient::QWaylandXdgSurface(0x563fc33e9120) According to the xdg-shell protocol, this is not allowed. The wayland QPA plugin is currently handling it by setting the parent to the topmost grabbing popup. Note, however, that this may cause positioning errors and popups closing unxpectedly because xdg-shell mandate that child popups close before parents

However, I see this warning even when there's no crash.

EXPECTED RESULT
No crashes

SOFTWARE/OS VERSIONS

Linux/KDE Plasma:
KDE Plasma Version: 5.26.3
KDE Frameworks Version: 5.99.0
Qt Version: 5.15.6, wayland

ADDITIONAL INFORMATION

I'm running the skrooge//master flatpak from kdeapps
      Commit: 08c9cf54185ecf222dd23dd2519322b368bcefd1da97f6fbf15ea86253046ce0
      Parent: e3434ea5c2e3dc00566ed2b90ee2e7c3d8d1f1588a208f671388194b48b34f15
     Subject: Built on Wed Nov 16 13:32:18 CET 2022
        Date: 2022-11-16 12:36:11 +0000
using
      Runtime: org.kde.Platform/x86_64/5.15-21.08
there are updates to both the app and runtime flatpaks, but I'm afraid updating may trigger bug 408970.
Comment 1 Stephane MANKOWSKI 2023-02-08 14:47:30 UTC
Hi,

This issue was not due to wayland but to a real allocation issue in skrooge.
This has been fixed in this commit:
https://invent.kde.org/office/skrooge/-/commit/7d1b4ada4dcacae5f512ff0a8aa934d5c04eb076