Bug 338517

Summary:	Error code -9 when trying to import wallet data
Product:	[Applications] kwalletmanager	Reporter:	Peter <kefas65>
Component:	general	Assignee:	Valentin Rusu <valir>
Status:	RESOLVED FIXED
Severity:	critical	CC:	arjunak234
Priority:	HI
Version:	2.0
Target Milestone:	---
Platform:	Kubuntu
OS:	Linux
Latest Commit:	http://commits.kde.org/kwallet/9dc50cdef8a35bd895371a7b206b352285fa7347	Version Fixed In:
Sentry Crash Report:

Description Peter 2014-08-24 10:00:07 UTC

Can't import exported Wallet.kde

Reproducible: Always

Steps to Reproduce:
1.SaveAs Wallet
2.Detete Wallet
3.Import Wallet
Actual Results:  
Error code -9 when trying to import wallet data

Expected Results:  
Open Saved Wallet.kde

Comment 1 Arjun AK 2014-08-26 13:33:31 UTC

Copying the kwl file alone will not do, the wallet's salt should be copied as well. Anyway once #337022 is implemented, we can remove this option.

Comment 2 Arjun AK 2014-09-26 20:29:01 UTC

Patch to fix this: https://git.reviewboard.kde.org/r/120381/

Comment 3 Arjun AK 2014-09-30 07:35:06 UTC

Git commit 9dc50cdef8a35bd895371a7b206b352285fa7347 by Arjun AK.
Committed on 30/09/2014 at 07:19.
Pushed by arjunak into branch 'frameworks'.

Add option to export/import wallets in encrypted form

This option allows the user to export/import all the wallets (both .kwl
as well as .salt files) onto a single tar archive. This commit also removes
the broken "Save as" option.

REVIEW: 120381
Related: bug 337022

M  +1    -0    src/manager/CMakeLists.txt
M  +1    -31   src/manager/kwalleteditor.cpp
M  +0    -1    src/manager/kwalleteditor.h
M  +82   -0    src/manager/kwalletmanager.cpp
M  +4    -0    src/manager/kwalletmanager.h
M  +2    -2    src/manager/kwalletmanager.rc

http://commits.kde.org/kwallet/9dc50cdef8a35bd895371a7b206b352285fa7347

Comment 4 Peter 2014-10-01 18:04:31 UTC

On 30/09/14 08:35, Arjun AK wrote:
> https://bugs.kde.org/show_bug.cgi?id=338517
>
> Arjun AK <arjunak234@gmail.com> changed:
>
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>           Resolution|---                         |FIXED
>               Status|CONFIRMED                   |RESOLVED
>        Latest Commit|                            |http://commits.kde.org/kwal
>                     |                            |let/9dc50cdef8a35bd895371a7
>                     |                            |b206b352285fa7347
>
> --- Comment #3 from Arjun AK <arjunak234@gmail.com> ---
> Git commit 9dc50cdef8a35bd895371a7b206b352285fa7347 by Arjun AK.
> Committed on 30/09/2014 at 07:19.
> Pushed by arjunak into branch 'frameworks'.
>
> Add option to export/import wallets in encrypted form
>
> This option allows the user to export/import all the wallets (both .kwl
> as well as .salt files) onto a single tar archive. This commit also removes
> the broken "Save as" option.
>
> REVIEW: 120381
> Related: bug 337022
>
> M  +1    -0    src/manager/CMakeLists.txt
> M  +1    -31   src/manager/kwalleteditor.cpp
> M  +0    -1    src/manager/kwalleteditor.h
> M  +82   -0    src/manager/kwalletmanager.cpp
> M  +4    -0    src/manager/kwalletmanager.h
> M  +2    -2    src/manager/kwalletmanager.rc
>
> http://commits.kde.org/kwallet/9dc50cdef8a35bd895371a7b206b352285fa7347
>
Thank you for upgrading kwallet.
Could you please give me some hints how to compile it.

Peter

Comment 5 Arjun AK 2014-10-01 18:35:43 UTC

> Could you please give me some hints how to compile it.

https://community.kde.org/Frameworks/Building

But please do note that if you have lost the salt file, there is absolutely no way you can retrieve the saved passwords.