Hitting assertion in kio-trash (KF5KIOCore)

[Kevin Funk]

On Wednesday 16 April 2014 21:31:01 Kevin Funk wrote:
> Hey,
> 
> While running unit tests from kdevplatform I hit the following assert in
> trash/trashimpl.cpp (from workspace/kio-extras)
> 
> Output of the unit test (which attempts to trash some folders):
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 1"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 2"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 3"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 4"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 5"
> trying to create  "/home/krf/.local/share/Trash/info/.trashinfo 6"
> ASSERT: "fileId.endsWith( QLatin1String(".trashinfo") )" in file
> /home/krf/devel/src/kf5/kde/workspace/kio-extras/trash/trashimpl.cpp, line
> 272 kioslave: ####### CRASH ###### protocol = trash pid = 6144 signal = 6
> /home/krf/devel/install/kf5/lib/x86_64-linux-gnu/libKF5KIOCore.so.5(+0x8d60
> 9) [0x7fbbbfe79609]
> (...)
> 
> Looking at the code, that assert looks very suspicious. The code above the
> assert seems to generate file ids such as '.trashinfo' or '.trashinfo
> SOME_NUMBER'. In case of the latter, the assert will always be hit.
> 
> Any ideas? Did KIO::RenameDialog::suggestName change?
> 
> Reproducable with:
> ./plugins/git/tests/kdevgit-test testRemoveEmptyFolder
> 
> from kdevplatform.git.
> 
> Greets

Hey,

https://git.reviewboard.kde.org/r/117951/ fixed this, btw.

It was a porting bug (KUrl -> QUrl) within trashimpl.cpp

Greets

-- 
Kevin Funk