Bug 419822 - Vault crashed after canceling password prompt
Summary: Vault crashed after canceling password prompt
Status: RESOLVED DUPLICATE of bug 418262
Alias: None
Product: Plasma Vault
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR crash
Target Milestone: ---
Assignee: Ivan Čukić
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2020-04-07 23:42 UTC by nuc
Modified: 2020-04-17 14:20 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
automated crash log (15.61 KB, text/plain)
2020-04-07 23:42 UTC, nuc
Details
crash log (11.01 KB, text/plain)
2020-04-17 09:07 UTC, Konstantinos Smanis
Details
crash log with plasma-vault debug symbols (10.94 KB, text/plain)
2020-04-17 09:40 UTC, Konstantinos Smanis
Details
crash log with plasma-vault debug symbols with manually launched kded5 (11.25 KB, text/plain)
2020-04-17 09:59 UTC, Konstantinos Smanis
Details
kded5 terminal output (1.05 KB, text/plain)
2020-04-17 10:02 UTC, Konstantinos Smanis
Details

Note You need to log in before you can comment on or make changes to this bug.
Description nuc 2020-04-07 23:42:18 UTC
Created attachment 127374 [details]
automated crash log

I wanted to try out the vault. I have created a new one and it appeared in the system tray vault manager as usual.

Then I clicked on my vault. The pw prompt opened. But instead of entering my pw I decided to click 'Cancel'.

This has led to a crash. Now when I click on my Vault Manager the list is empty, my vault is gone (I did not attempt a restart yet tho).

Crash log included, I hope it helps.
Comment 1 Nate Graham 2020-04-15 03:05:14 UTC
Cannot reproduce.
Comment 2 nuc 2020-04-15 13:01:29 UTC
Hm, I can reproduce this a 100%
Comment 3 Ivan Čukić 2020-04-15 14:44:58 UTC
Can you provide the versions of Qt, Plasma, cryfs, encfs?
Comment 4 Konstantinos Smanis 2020-04-17 09:07:28 UTC
Created attachment 127602 [details]
crash log

I can reproduce this consistently as well. Cancelling the password prompt crashes kded5 leaving the system in a miserable state. I was trying to access a cryfs-encrypted vault.

Operating System: Gentoo Linux 
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.1
Kernel Version: 5.6.4-gentoo
OS Type: 64-bit

cryfs version: 0.10.2
encfs version: not installed
Comment 5 Ivan Čukić 2020-04-17 09:32:48 UTC
Can you install packages with debugging symbols? (plasma-vault in particular)
Comment 6 Konstantinos Smanis 2020-04-17 09:40:20 UTC
Created attachment 127604 [details]
crash log with plasma-vault debug symbols

Let me know if you need extra debug symbols, e.g., for Qt, as that would take significantly more time to acquire, being on Gentoo.
Comment 7 Ivan Čukić 2020-04-17 09:50:52 UTC
Thanks!

I think debugging symbols for Qt are not needed at this point. Can you also start kded5 manually from terminal and post the output right before the crash?
Comment 8 Konstantinos Smanis 2020-04-17 09:59:48 UTC
Created attachment 127605 [details]
crash log with plasma-vault debug symbols with manually launched kded5
Comment 9 Konstantinos Smanis 2020-04-17 10:02:57 UTC
Created attachment 127606 [details]
kded5 terminal output

After replacing kded5 (kded5 --replace), my vault didn't show up in the Plasma widget, so I had to restart plasmashell as well (kquitapp5 plasmashell && plasmashell), as you may notice in the log.
Comment 10 Ivan Čukić 2020-04-17 10:29:56 UTC
Hm, the output log seems to hint at another place than the crash log. Fun.

Can you compile the master version of plasma-vault and check again?
Comment 11 Konstantinos Smanis 2020-04-17 12:07:51 UTC
Commit 4d11dcf doesn't crash. I picked this one because the next one bumps the KF5 deps to 5.70.0 and I would have to go through a lot of unreleased deps.
Comment 12 Konstantinos Smanis 2020-04-17 12:31:19 UTC
This bug is just a duplicate of #418262; I can confirm that commit f6a924b resolves the issue. Just cherry-pick the fix to the Plasma/5.18 branch.
Comment 13 Ivan Čukić 2020-04-17 12:33:06 UTC
I have to say that you're amazing! :)

Can you check the commit f6a924b37880b7340e796846a6e5474413ca6533 and one before that one. I think that might be the fix that resolved this crash.
Comment 14 Konstantinos Smanis 2020-04-17 12:39:12 UTC
I did, commit f6a924b^ crashes, f6a924b doesn't. I just confirmed that the cherry-pick fixes the issue on the Plasma/5.18 branch as well.
Comment 15 Nate Graham 2020-04-17 13:35:30 UTC

*** This bug has been marked as a duplicate of bug 418262 ***
Comment 16 Nate Graham 2020-04-17 13:38:11 UTC
Ivan, you wanna do the cherry-pick?
Comment 17 Ivan Čukić 2020-04-17 13:42:38 UTC
Done
Comment 18 Nate Graham 2020-04-17 14:20:11 UTC
Thanks!

*** This bug has been marked as a duplicate of bug 418262 ***