|Summary:||KGpg crashed when canceling overwriting last file to decrypt|
|Product:||[Applications] kgpg||Reporter:||Toralf Förster <toralf.foerster>|
|Component:||general||Assignee:||Rolf Eike Beer <kde>|
|Latest Commit:||Version Fixed In:||4.8.2|
|Attachments:||New crash information added by DrKonqi|
Description Toralf Förster 2012-02-22 21:22:08 UTC
Comment 1 Toralf Förster 2012-02-22 21:24:27 UTC
Ah - the crash occured again if I press "cancel" in the "overwrite ?" dialog window.
Comment 2 Rolf Eike Beer 2012-03-21 16:47:09 UTC
Git commit 42f8b1b51b09cb4b64fadf7e6cdd5ce359b0fa74 by Rolf Eike Beer. Committed on 21/03/2012 at 17:41. Pushed by dakon into branch 'KDE/4.8'. fix crash when list of files to decrypt is empty This happens when the last file in the list would overwrite an existing file and instead of overwriting the decryption of that file was skipped. FIXED-IN:4.8.2 M +3 -3 kgpgexternalactions.cpp http://commits.kde.org/kgpg/42f8b1b51b09cb4b64fadf7e6cdd5ce359b0fa74
Comment 3 Toralf Förster 2012-03-21 17:55:52 UTC
Created attachment 69793 [details] New crash information added by DrKonqi kgpg (2.7.1) on KDE Platform 4.8.1 (4.8.1) using Qt 4.8.0 - What I was doing when the application crashed: I patched kgpg with the given change set. Now the app doesn't cored, but it crashed and opened the KDEs crash handler ;/ -- Backtrace (Reduced): #7 QUrl::scheme (this=0xa006369) at io/qurl.cpp:4492 #8 0xb6383a22 in isLocalFile (url=...) at /var/tmp/portage/kde-base/kdelibs-4.8.1-r1/work/kdelibs-4.8.1/kdecore/io/kurl.cpp:906 #9 0x080e2c3c in KGpgExternalActions::decryptFile (this=0xa320e48) at /var/tmp/portage/kde-base/kgpg-4.8.1/work/kgpg-4.8.1/kgpgexternalactions.cpp:295 #10 0x080e2f23 in KGpgExternalActions::decryptFile (this=0xa320e48) at /var/tmp/portage/kde-base/kgpg-4.8.1/work/kgpg-4.8.1/kgpgexternalactions.cpp:318 #11 0x080bab61 in KGpgApp::newInstance (this=0xa0a4208) at /var/tmp/portage/kde-base/kgpg-4.8.1/work/kgpg-4.8.1/kgpg.cpp:178
Comment 4 Rolf Eike Beer 2012-03-21 21:28:59 UTC
You did something wrong. Line 295 now is "droppedUrls.isEmpty()". Your backtrace shows it is "droppedUrls.first().isLocalFile()", which is the code as it was before the patch.
Comment 5 Toralf Förster 2012-03-22 09:44:02 UTC
(In reply to comment #4) > You did something wrong. Line 295 now is "droppedUrls.isEmpty()". Your > backtrace shows it is "droppedUrls.first().isLocalFile()", which is the code > as it was before the patch. Right - Gentoo's portage fools me : https://bugs.gentoo.org/show_bug.cgi?id=409293 I fear.
Comment 6 Rolf Eike Beer 2012-04-08 15:59:00 UTC
*** Bug 297680 has been marked as a duplicate of this bug. ***