Kunzisoft / KeePassDX

Lightweight vault and password manager for Android, KeePassDX allows editing encrypted data in a single file in KeePass format and fill in the forms in a secure way.
https://www.keepassdx.com/
GNU General Public License v3.0
4.31k stars 260 forks source link

(Crowd)fund security audit #561

Open rugk opened 4 years ago

rugk commented 4 years ago

It would be good if this software could be professionally audited once, at least… (with all the growing complexity and features to be implemented)

I know that's hard and likely too much to crowdfund. A support program like EU Horizon or so may be more realistic or something similar… :thinking:

CaseOf commented 1 year ago

Hello, just to let you know that KeePassXC was finally audited after 6 years of existence : https://keepassxc.org/blog/2023-04-15-audit-report/ If there is a way for KeePassDX to be audited too, that would be great!

J-Jamet commented 1 year ago

The code is open and accessible, If a security expert wants to do an audit, he is of course welcome. :)

shuvashish76 commented 1 year ago

From KPXC audit report pdf [Memory Protection and Deallocation, page 25]

KeePassDX password manager decides not to protect even passwords by default, strangely, it is an option with KDBX4 format.

Any specific reason for this? If discussed before please post the issue link. Thanks.

J-Jamet commented 1 year ago

Link to KeePassXC audit : https://keepassxc.org/assets/pdf/KeePassXC-Review-V1-Molotnikov.pdf

No specific reason, the configuration remained the same as KeePassDroid even after refactoring the code in order to keep the same behavior and not have any side effects. Because of the nature of the memory partitioning per application on Android and the JVM, there is in my opinion little risk at this level, but I will change this default configuration and check that it does not have any harmful behavior (especially on the search, so in a first time apply the configuration only on the password).

J-Jamet commented 1 year ago

KeePassDroid configuration here also to be modified for users who still use it.

J-Jamet commented 1 year ago

https://github.com/Kunzisoft/KeePassDX/commit/3f63fa9c304a7521ff843d785d96993820158274