Bug 426282 - KFind crashes on 'Open containing folder(s)' when Dolphin is not installed
Summary: KFind crashes on 'Open containing folder(s)' when Dolphin is not installed
Status: RESOLVED WORKSFORME
Alias: None
Product: kfind
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR crash
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords: drkonqi
Depends on:
Blocks:
 
Reported: 2020-09-07 15:17 UTC by Jan Klos
Modified: 2022-12-15 05:16 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 Jan Klos 2020-09-07 15:17:41 UTC
Application: kfind (20.08.1)

Qt Version: 5.15.0
Frameworks Version: 5.73.0
Operating System: Linux 5.8.7-1-MANJARO x86_64
Windowing system: X11
Distribution: Manjaro Linux

-- Information about the crash:
- What I was doing when the application crashed:
1. Clicked 'Open containing folder(s)'
2. Funnily enough, Krusader DOES get opened at the correct directory, so the menu entry actually works - it jsut crashed KFind right after that

- Custom settings of the application:
Uninstalled Dolphin - my default FM is Krusader. Everything works with Dolphin, but even though directories are associated with Krusader, KFind starts Dolphin instead, not respecting the association.

It would be nice if KFind respected file associations and utilized the default file manager even when Dolphin IS installed.

The crash can be reproduced every time.

-- Backtrace:
Application: KFind (kfind), signal: Segmentation fault

[KCrash Handler]
#4  0x00007ff6549575b4 in KJob::setError(int) () from /usr/lib/libKF5CoreAddons.so.5
#5  0x00007ff654fcf898 in ?? () from /usr/lib/libKF5KIOWidgets.so.5
#6  0x00007ff65394ea06 in ?? () from /usr/lib/libQt5Core.so.5
#7  0x00007ff654956ced in KJob::result(KJob*, KJob::QPrivateSignal) () from /usr/lib/libKF5CoreAddons.so.5
#8  0x00007ff65495780c in KJob::finishJob(bool) () from /usr/lib/libKF5CoreAddons.so.5
#9  0x00007ff65394ea06 in ?? () from /usr/lib/libQt5Core.so.5
#10 0x00007ff654956ced in KJob::result(KJob*, KJob::QPrivateSignal) () from /usr/lib/libKF5CoreAddons.so.5
#11 0x00007ff65495780c in KJob::finishJob(bool) () from /usr/lib/libKF5CoreAddons.so.5
#12 0x00007ff65394ea06 in ?? () from /usr/lib/libQt5Core.so.5
#13 0x00007ff6539441b2 in QObject::event(QEvent*) () from /usr/lib/libQt5Core.so.5
#14 0x00007ff6543df702 in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/lib/libQt5Widgets.so.5
#15 0x00007ff65391779a in QCoreApplication::notifyInternal2(QObject*, QEvent*) () from /usr/lib/libQt5Core.so.5
#16 0x00007ff65391a283 in QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) () from /usr/lib/libQt5Core.so.5
#17 0x00007ff653970cd4 in ?? () from /usr/lib/libQt5Core.so.5
#18 0x00007ff65213943c in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
#19 0x00007ff6521871d9 in ?? () from /usr/lib/libglib-2.0.so.0
#20 0x00007ff652138221 in g_main_context_iteration () from /usr/lib/libglib-2.0.so.0
#21 0x00007ff653970311 in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib/libQt5Core.so.5
#22 0x00007ff65391611c in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib/libQt5Core.so.5
#23 0x00007ff6545e2baa in QDialog::exec() () from /usr/lib/libQt5Widgets.so.5
#24 0x000055e6df6ecbfc in ?? ()
#25 0x00007ff6532e8152 in __libc_start_main () from /usr/lib/libc.so.6
#26 0x000055e6df6ecd2e in _start ()
[Inferior 1 (process 13388) detached]

Possible duplicates by query: bug 426090, bug 425619, bug 423132, bug 422962, bug 421930.

Reported using DrKonqi
Comment 1 Ahmad Samir 2020-09-29 18:49:39 UTC
This bug has two issues:
- the crash, this should be fixed by https://invent.kde.org/frameworks/kio/-/merge_requests/145
- the issue of using dolphin even though the default file manager is set to krusader
Comment 2 Christoph Feck 2020-09-29 22:53:25 UTC
Thanks Ahmad, I am glad I added a competent developer to the CC list ;)
Comment 3 David Faure 2020-10-10 18:26:30 UTC
Git commit 180dfe9666c53385fe51f8da530578c369ee80c8 by David Faure, on behalf of Ahmad Samir.
Committed on 10/10/2020 at 18:26.
Pushed by dfaure into branch 'master'.

OpenFileManagerWindowJob: Fix crash when falling back to KRun strategy

When falling back to the KRun strategy create the OpenFileManagerWindowKRunStrategy
object on the heap, otherwise the object is destroyed when it goes out of
scope and then we crash in emitResultProxy trying to access job member in
AbstractOpenFileManagerWindowStrategy.

M  +23   -9    src/widgets/openfilemanagerwindowjob.cpp
M  +2    -2    src/widgets/openfilemanagerwindowjob.h

https://invent.kde.org/frameworks/kio/commit/180dfe9666c53385fe51f8da530578c369ee80c8
Comment 4 Ahmad Samir 2020-12-09 21:25:43 UTC
(In reply to Ahmad Samir from comment #1)
> This bug has two issues:
> - the crash, this should be fixed by
> https://invent.kde.org/frameworks/kio/-/merge_requests/145
> - the issue of using dolphin even though the default file manager is set to
> krusader

For the second issue, this a side-effect of using the D-Bus method to launch a file manager when "open containing folder" is used; Krusader doesn't have a org.freedesktop.FileManager1 dbus service like e.g. Dolphin, Nautilus, Thunar... etc have.

Another issue is that whichever file manager is running first claims the org.freedesktop.FileManager1 d-bus address, and that one is launched by "open containing folder"... an interesting problem.
Comment 5 Justin Zobel 2022-11-15 22:56:36 UTC
Thank you for reporting this issue in KDE software. As it has been a while since this issue was reported, can we please ask you to see if you can reproduce the issue with a recent software version?

If you can reproduce the issue, please change the status to "REPORTED" when replying. Thank you!
Comment 6 Bug Janitor Service 2022-11-30 05:16:18 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 7 Bug Janitor Service 2022-12-15 05:16:32 UTC
This bug has been in NEEDSINFO status with no change for at least
30 days. The bug is now closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!