Bug 231913

Summary: [Testcase] KFileDialog::getOpenUrl() with kfiledialog:///-style startdir crashes on windows
Product: [Unmaintained] kfile Reporter: Thomas Friedrichsmeier <thomas.friedrichsmeier>
Component: generalAssignee: KDE-Windows <kde-windows>
Status: RESOLVED FIXED    
Severity: crash CC: lpapp, ps_ml, thomas.friedrichsmeier, toddrme2178
Priority: NOR    
Version: 4.5 and older   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Testcase application

Description Thomas Friedrichsmeier 2010-03-23 17:00:02 UTC
Version:            (using KDE 4.4.1)
OS:                MS Windows
Installed from:    MS Windows

KFileDialog::getOpenUrl() with a kfiledialog://-style startDir crashes on Windows the *second* time it is used (i.e. once an stored setting exists).

In code:
// select a file, here
KFileDialog::getOpenUrl (KUrl ("kfiledialog:///<testdir>"), i18n("*|All files"), this, i18n("Open File..."));
// try again, and it will crash
KFileDialog::getOpenUrl (KUrl ("kfiledialog:///<testdir>"), i18n("*|All files"), this, i18n("Open File...")).url()

Instructions on using the testcase (to be attached in a minute):
1) Click the topmost button.
2) Select a file and click "OK".
3) Click the topmost button again.
Comment 1 Thomas Friedrichsmeier 2010-03-23 17:01:53 UTC
Created attachment 42205 [details]
Testcase application

If it helps, the following CMakeLists.txt can be used to compile:

PROJECT(kfdtest)
FIND_PACKAGE(KDE4 REQUIRED)
ADD_DEFINITIONS(${QT_DEFINITIONS} ${KDE4_DEFINITIONS})
INCLUDE_DIRECTORIES(${KDE4_INCLUDE_DIR} ${QT_INCLUDES})
LINK_DIRECTORIES(${KDE4_LIB_DIR})
KDE4_ADD_EXECUTABLE(kfdtest main.cpp)
TARGET_LINK_LIBRARIES(kfdtest ${KDE4_KDECORE_LIBS} ${KDE4_KUI_LIBS} ${KDE4_KFILE_LIBS})
Comment 2 Thomas Friedrichsmeier 2010-03-23 17:03:46 UTC
Forgot to add:

- This happens on windows, only, not Linux.
- Also found in 4.4.65.
- Was present as early as KDE 4.2.3 (sorry for not reporting it, then).
Comment 3 Myriam Schweingruber 2012-09-07 14:47:02 UTC
Is this still valid with KDE 4.9.0 or newer?
Comment 4 Todd 2012-09-07 14:51:46 UTC
I don't think KDE 4.9.0 is available for windows yet, which would make it impossible to confirm at this point.
Comment 5 Laszlo Papp 2012-09-07 14:56:25 UTC
Not impossible since this can be built from source, so a developer at least could verify. :-)

Unfortunately no installers for end users yet as far as I know.
Comment 6 Thomas Friedrichsmeier 2012-09-07 17:11:44 UTC
(In reply to comment #3)
> Is this still valid with KDE 4.9.0 or newer?

I will test once I find the time to update to a more recent KDE on windows, which may take a while.

Of course, actually, one of the benefits of a testcase (as provided) is that *you* can test this, yourself.
Comment 7 Thomas Friedrichsmeier 2012-10-08 09:03:07 UTC
No longer reproducable with KDE 4.7.0.