Bug 308656 - Crash after close split view and restore file from Trash
Summary: Crash after close split view and restore file from Trash
Status: RESOLVED NOT A BUG
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 2.1
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-19 12:45 UTC by Alexey Shildyakov
Modified: 2012-10-19 14:25 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Crash info from DrKonqi (9.18 KB, text/plain)
2012-10-19 12:45 UTC, Alexey Shildyakov 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Shildyakov 2012-10-19 12:45:04 UTC 
1. Open split view. Left side - the folder on USB drive where the file in trash will restore to. Right side - the trash.
2. Restore image file from trash. The file was shown in the left view with thumbnail.
3. Close right view
4. Crash

Reproducible: Couldn't Reproduce
Comment 1 Alexey Shildyakov 2012-10-19 12:45:21 UTC 
Created attachment 74648 [details]
Crash info from DrKonqi
Comment 2 Frank Reininghaus 2012-10-19 13:18:25 UTC 
Thanks for the report!

You probably have the debugging patch for bug 304986 applied, right? I'm asking because I think that a crash with such a backtrace cannot happen with the unpatched Dolphin.
Comment 3 Alexey Shildyakov 2012-10-19 13:52:02 UTC 
Yep, patch was installed. I forgot to recompile the package.
Is this a reason for the crash?
Comment 4 Frank Reininghaus 2012-10-19 14:25:58 UTC 
Thanks for the quick reply.

Yes, the patch is the reason for the crash. In the interest of getting as much useful output as possible, I printed the current state of some variables in all places where they can change. However, when the view is closed, this can cause a crash because the destructor of the class indirectly calls the debug output code, which tries to access data from the KFileItemModel which has already been deleted.

Sorry for the trouble, I should have mentioned this in the other bug report.