Summary: | DragonPlayer crashes after trying to open a file (fileModule() returs 0, libkfilemodule vs kfilemodule) | ||
---|---|---|---|
Product: | [Unmaintained] kdelibs | Reporter: | Javier <javi.azuaga> |
Component: | general | Assignee: | kdelibs bugs <kdelibs-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | andresbajotierra, cfeck, Ch.Ehrlicher, finex, friesoft, kde-windows, martin.sandsmark, neundorf |
Priority: | HI | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Javier
2009-08-25 13:32:48 UTC
It seems like our usage of kfiledialog is correct, so I'm re-assigning to kdelibs. (We're calling it like so: const KUrl url = KFileDialog::getOpenUrl( KUrl("kfiledialog:///dragonplayer"),mimeFilter.join(" "), this, i18n("Select File to Play") ); ) The crash is probably caused by fileModule() returning 0. This should not happen with a standard KDE installation. Javier, can you please check if you can reproduce this when you run from Konsole? There should be a warning about "libkfilemodule". Actually the fix for bug 204874 (on Windows) causes this bug (on Linux). The problem is the naming "libkfilemodule" vs. "kfilemodule". I don't get the crash, because I have installed both (from an older compile): > 280676 25. Aug 18:50 /local/kde4/lib/kde4/kfilemodule.so > 276056 22. Aug 13:21 /local/kde4/lib/kde4/libkfilemodule.so Commit r1014877 changed this to fix Windows, but the crash is now on Linux. Raising severity and priority, adding some CC to clarify. kde-buildsystem does not have a Bugzilla account, so adding Alexander directly. If anyone knows an address (he is on vacation), please add to CC. *** Bug 205026 has been marked as a duplicate of this bug. *** SVN commit 1015602 by cfeck: Use WITH_PREFIX for "kfilemodule" conditionally on WIN32 Temporary? I have no idea for a better fix... If this is the right way to solve this, please close. CCBUG: 205069 M +7 -1 CMakeLists.txt WebSVN link: http://websvn.kde.org/?view=rev&revision=1015602 The patch works for me. I think it's ok although it breaks BC on windows but this shouldn't matter. It's better than adding a hack in KPluginLoader. There has been no proposal for a different way to solve this issue (see comment #5), so I am closing this bug. |