Bug 326603

Summary: Import Wallet... option no longer works
Product: [Applications] kwalletmanager Reporter: Marcus Harrison <marcus>
Component: generalAssignee: Valentin Rusu <valir>
Status: REPORTED ---    
Severity: normal CC: arjunak234, lueck, martin.sandsmark, p.temple
Priority: NOR    
Version First Reported In: 1.10   
Target Milestone: ---   
Platform: unspecified   
OS: Other   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Marcus Harrison 2013-10-25 07:07:44 UTC 
Selecting File-->Import Wallet... opens the file open dialogue. Selecting the wallet file from the dialogue prompts the password for that file. The password is entered correctly. After this, nothing else happens. Passwords that are in the wallet file selected, but not in the current wallet, are not imported. The old dialogue prompt whether to skip or replace existing entries does not appear. Attempting to import the wallet file again does not prompt for a password and nothing happens.

Reproducible: Always
Comment 1 Burkhard Lück 2013-11-21 14:37:04 UTC 
confirmed with 4.12 compiled from sources
Comment 2 Arjun AK 2014-10-01 19:10:57 UTC 
Can you reproduce this in 4.14? Is this related to #338517
Comment 3 Marcus Harrison 2014-10-01 20:56:46 UTC 
Funnily enough, having observed some pretty large changes to KWallet I attempted to reproduce this with a simple test case:

1. Using File-->Save As... to save my wallet to transferable media

2. Transferring the saved file to a new computer also running 4.14

3. Using the File-->Import Wallet... option to attempt to import the wallet file.

However, this failed because a new .salt file is needed, which I assume is mixed with the password in the encryption/decryption process. The wallet file saved with the Save As... option would not accept the correct password.
Comment 4 Burkhard Lück 2014-10-02 08:22:03 UTC 
(In reply to Arjun AK from comment #2)
> Can you reproduce this in 4.14? Is this related to #338517

$ kwalletmanager --version
Qt: 4.8.6
KDE: 4.14.0
KDE-Passwortverwaltung: 2.0

Now I get the error mentioned in #338517 (Error code -9)
Comment 5 Christoph Feck 2014-10-25 17:09:05 UTC 
Reopening based on recent feedback.
Comment 6 Marcus Harrison 2014-10-28 11:26:41 UTC 
I would like to ask quickly - why is a salt needed in this context?
Comment 7 Andrew Crouthamel 2018-11-12 02:55:01 UTC 
Dear Bug Submitter,

This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond.

Thank you for helping us make KDE software even better for everyone!
Comment 8 Andrew Crouthamel 2018-11-21 04:22:59 UTC 
Dear Bug Submitter,

This is a reminder that this bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? This bug will be moved back to REPORTED Status for manual review later, which may take a while. If you are able to, please lend us a hand.

Thank you for helping us make KDE software even better for everyone!