Review Request 120388: Do not sync if wallet file does not exist

Arjun AK arjunak234 at gmail.com
Sat Jan 10 17:12:34 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120388/
-----------------------------------------------------------

(Updated Jan. 10, 2015, 5:12 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, Àlex Fiestas, Teo Mrnjavac, and Valentin Rusu.


Repository: kwallet


Description
-------

1. Create a new wallet "foo"
2. immediately remove the 'kwalletd5' directory
3. `KWalletD::timedOutSync()` will call sync on a nonexistant wallet (which will create "foo.kwl")

Kwalletmanager still lists the wallet and new data can be added to the wallet, but this data will be lost once the wallet is closed since the salt has been deleted and therefore the wallet cannot be opened again.


Diffs
-----

  src/runtime/kwalletd/backend/kwalletbackend.cc b072cec 
  src/runtime/kwalletd/kwalletd.h b94df15 
  src/runtime/kwalletd/kwalletd.cpp 882eb72 

Diff: https://git.reviewboard.kde.org/r/120388/diff/


Testing
-------


Thanks,

Arjun AK

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150110/10609bd2/attachment.html>


More information about the Kde-frameworks-devel mailing list