Summary: | Crash at startup | ||
---|---|---|---|
Product: | [Applications] Falkon | Reporter: | Antonio Rojas <arojas> |
Component: | general | Assignee: | Unassigned bugs <unassigned-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | jurajoravec, kevin.kofler |
Priority: | NOR | ||
Version First Reported In: | 25.03.80 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/network/falkon/-/commit/dca93d08742f7c4ed66831ae1bea6078ffc7d716 | Version Fixed In: | 25.04.1 |
Sentry Crash Report: |
Description
Antonio Rojas
2025-03-15 14:06:58 UTC
To reproduce with a clean profile: - Add a folder to the bookmarks toolbar - Add any bookmark inside that folder - Restart falkon -> crash. Can reproduce. Thank you for reporting. A possibly relevant merge request was started @ https://invent.kde.org/network/falkon/-/merge_requests/127 Hello again, I think I have a fix for it. Will check tomorrow or later and backport it also for the upcoming release. Thank you again for testing and reporting. Git commit d145d932c48e080afdd99a0571acfcb2a861e40d by Juraj Oravec. Committed on 20/03/2025 at 20:09. Pushed by jurajo into branch 'master'. Fix crash when loading folder in bookmarks toolbar Signed-off-by: Juraj Oravec <jurajoravec@mailo.com> M +8 -0 src/lib/bookmarks/bookmarkstoolbarbutton.cpp M +1 -1 src/lib/bookmarks/bookmarkstools.cpp M +1 -1 src/lib/history/historymenu.cpp M +2 -2 src/lib/navigation/navigationbar.cpp https://invent.kde.org/network/falkon/-/commit/d145d932c48e080afdd99a0571acfcb2a861e40d Git commit 164475bf836ebf96874c227e0cd3c91aa33f4de7 by Juraj Oravec. Committed on 20/03/2025 at 20:23. Pushed by jurajo into branch 'release/25.04'. Fix crash when loading folder in bookmarks toolbar CherryPicked from: e6ee1e67bb6a6031fbafcb0dce53f5d30af377de Signed-off-by: Juraj Oravec <jurajoravec@mailo.com> M +8 -0 src/lib/bookmarks/bookmarkstoolbarbutton.cpp M +1 -1 src/lib/bookmarks/bookmarkstools.cpp M +1 -1 src/lib/history/historymenu.cpp M +2 -2 src/lib/navigation/navigationbar.cpp https://invent.kde.org/network/falkon/-/commit/164475bf836ebf96874c227e0cd3c91aa33f4de7 Looks like the fix just moves the crash elsewhere: https://bugzilla.redhat.com/show_bug.cgi?id=2361191 Looks like addFolderToMenu needs the same kind of fix you made in addUrlToMenu, connecting to a lambda rather than directly to mApp->getWindow()->statusBar() that is not yet constructed at that point. So something like: - QObject::connect(m, &QMenu::aboutToHide, mApp->getWindow()->statusBar(), &StatusBar::clearMessage); + QObject::connect(m, &QMenu::aboutToHide, [=]() { + mApp->getWindow()->statusBar()->clearMessage(); + }); … - QObject::connect(act, &QAction::hovered, mApp->getWindow()->statusBar(), &StatusBar::clearMessage); + QObject::connect(act, &QAction::hovered, [=]() { + mApp->getWindow()->statusBar()->clearMessage(); + }); Should be fixed in latest commits to master and release/25.04 branch. PS: Made a mistake when writing down bug numbers. Git commit dca93d08742f7c4ed66831ae1bea6078ffc7d716 by Juraj Oravec. Committed on 20/04/2025 at 10:45. Pushed by jurajo into branch 'release/25.04'. Fix crash when creating bookmarks toolbar. Cherry-picked from? 31ba9472369256804400a2db36b3dca3b4be2d73 Signed-off-by: Juraj Oravec <jurajoravec@mailo.com> M +9 -3 src/lib/bookmarks/bookmarkstools.cpp https://invent.kde.org/network/falkon/-/commit/dca93d08742f7c4ed66831ae1bea6078ffc7d716 |