kylemanna / keepassx

🔑 This fork adds YubiKey support
https://github.com/keepassx/keepassx/pull/52
Other
38 stars 5 forks source link

KDBX corruption when saving with YubiKey removed #2

Closed kylemanna closed 10 years ago

kylemanna commented 10 years ago

As mentioned by @fabian-z

But perhaps the most serious flaw I found is a database corruption, which is triggered when opening a challenge-response secured database, removing the yubikey and then trying to save the database. The save was not delayed as before with no error and apparently led to database corruption (was reproducable for me)! Could you please adress the corruption problem?

https://github.com/keepassx/keepassx/pull/52#issuecomment-46694026

Steps to reproduce:

  1. Create kdbx with password + yk
  2. Add entry / test data and save
  3. Remove yk
  4. Save kdbx file (will silently fail and encrypt file with garbage)

Possibly other bugs.

Expected behavior:

fabian-z commented 10 years ago

Confirmed fixed as of b7fd29b8 :smiley:

kylemanna commented 10 years ago

This was a big oversight on my part originally. Thanks for catching, reporting, and testing! :beer: