Reduce default Argon2 memory and thread settings (keepassxreboot#3672)

* Fix keepassxreboot#3550 * Default memory reduced to 64 MiB (from 128 MiB) and parallelism reduced to 2 threads. This allows for desktop and mobile device compatibility.
aerowolf · Oct 26, 2019 · 744354c · 744354c
1 parent 57a7720
commit 744354c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/gui/dbsettings/DatabaseSettingsWidgetEncryption.cpp b/src/gui/dbsettings/DatabaseSettingsWidgetEncryption.cpp
@@ -400,8 +400,10 @@ void DatabaseSettingsWidgetEncryption::updateFormatCompatibility(int index, bool
 
         if (kdf->uuid() == KeePass2::KDF_ARGON2) {
             auto argon2Kdf = kdf.staticCast<Argon2Kdf>();
-            argon2Kdf->setMemory(128 * 1024);
-            argon2Kdf->setParallelism(static_cast<quint32>(QThread::idealThreadCount()));
+            // Default to 64 MiB of memory and 2 threads
+            // these settings are safe for desktop and mobile devices
+            argon2Kdf->setMemory(1 << 16);
+            argon2Kdf->setParallelism(2);
         }
 
         activateChangeDecryptionTime();