diff --git a/.travis.yml b/.travis.yml index df7f8ed58..942bb426d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,8 +15,8 @@ compiler: - gcc env: - - CONFIG=Release ASAN_OPTIONS=detect_odr_violation=1:leak_check_at_exit=0 - - CONFIG=Debug ASAN_OPTIONS=detect_odr_violation=1:leak_check_at_exit=0 + - CONFIG=Release ASAN_OPTIONS=detect_odr_violation=1 + - CONFIG=Debug ASAN_OPTIONS=detect_odr_violation=1 git: depth: 3 @@ -37,7 +37,7 @@ script: - cmake -DCMAKE_BUILD_TYPE=${CONFIG} -DWITH_GUI_TESTS=ON -DWITH_ASAN=ON -DWITH_XC_HTTP=ON -DWITH_XC_AUTOTYPE=ON -DWITH_XC_YUBIKEY=ON $CMAKE_ARGS .. - make -j2 - if [ "$TRAVIS_OS_NAME" = "linux" ]; then make test ARGS+="-E testgui --output-on-failure"; fi - - if [ "$TRAVIS_OS_NAME" = "linux" ]; then xvfb-run -a --server-args="-screen 0 800x600x24" make test ARGS+="-R testgui --output-on-failure"; fi + - if [ "$TRAVIS_OS_NAME" = "linux" ]; then ASAN_OPTIONS=${ASAN_OPTIONS}:leak_check_at_exit=0 xvfb-run -a --server-args="-screen 0 800x600x24" make test ARGS+="-R testgui --output-on-failure"; fi - if [ "$TRAVIS_OS_NAME" = "osx" ]; then make test ARGS+="--output-on-failure"; fi # Generate snapcraft build when merging into master/develop branches diff --git a/CHANGELOG b/CHANGELOG index 3719f8e4c..931b86a83 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,3 +1,13 @@ +2.2.3 (2017-12-11) +========================= + +- Prevent database corruption when locked [#1219] +- Fixes apply button not saving new entries [#1141] +- Switch to Consolas font on Windows for password edit [#1229] +- Multiple fixes to AppImage deployment [#1115, #1228] +- Fixes multiple memory leaks [#1213] +- Resize message close to 16x16 pixels [#1253] + 2.2.2 (2017-10-22) ========================= diff --git a/CMakeLists.txt b/CMakeLists.txt index ec7682c6c..915dde8f2 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -50,7 +50,7 @@ set(CMAKE_AUTOUIC ON) set(KEEPASSXC_VERSION_MAJOR "2") set(KEEPASSXC_VERSION_MINOR "2") -set(KEEPASSXC_VERSION_PATCH "2") +set(KEEPASSXC_VERSION_PATCH "3") set(KEEPASSXC_VERSION "${KEEPASSXC_VERSION_MAJOR}.${KEEPASSXC_VERSION_MINOR}.${KEEPASSXC_VERSION_PATCH}") # Distribution info diff --git a/share/icons/application/16x16/actions/message-close.png b/share/icons/application/16x16/actions/message-close.png index 4b2f9ca4d..b3a44a232 100644 Binary files a/share/icons/application/16x16/actions/message-close.png and b/share/icons/application/16x16/actions/message-close.png differ diff --git a/share/linux/org.keepassxc.KeePassXC.appdata.xml b/share/linux/org.keepassxc.KeePassXC.appdata.xml index b2a4de6e6..39cddac12 100644 --- a/share/linux/org.keepassxc.KeePassXC.appdata.xml +++ b/share/linux/org.keepassxc.KeePassXC.appdata.xml @@ -1,5 +1,5 @@ - +Copyright 2017 KeePassXC Team --> org.keepassxc.KeePassXC.desktop KeePassXC @@ -50,6 +50,18 @@ + + +
    +
  • Prevent database corruption when locked [#1219]
  • +
  • Fixes apply button not saving new entries [#1141]
  • +
  • Switch to Consolas font on Windows for password edit [#1229]
  • +
  • Multiple fixes to AppImage deployment [#1115, #1228]
  • +
  • Fixes multiple memory leaks [#1213]
  • +
  • Resize message close to 16x16 pixels [#1253]
  • +
+
+

Changes included in this release:

@@ -69,7 +81,8 @@
  • Fixed screen lock and Google fallback settings [#1029]
  • -
    + +

    Changes included in this release:

      diff --git a/share/translations/keepassx_de.ts b/share/translations/keepassx_de.ts index 2fd20f04a..f3bd0e39f 100644 --- a/share/translations/keepassx_de.ts +++ b/share/translations/keepassx_de.ts @@ -571,7 +571,7 @@ Sie können sie speichern oder die Sperre freigeben. Do you want to save the database before locking it? Otherwise your changes are lost. Dieses Datenbank wurde geändert. -Soll sie gespeichert werden bevor sie gesperrt wirt? +Soll sie gespeichert werden bevor sie gesperrt wird? Anderenfalls gehen Ihre Änderungen verloren. diff --git a/share/translations/keepassx_es.ts b/share/translations/keepassx_es.ts index f8176b807..479284898 100644 --- a/share/translations/keepassx_es.ts +++ b/share/translations/keepassx_es.ts @@ -69,7 +69,7 @@ Núcleo: %3 %4 Distribution: %1 - + Distribución: %1 @@ -250,7 +250,7 @@ Por favor seleccione si desea autorizar su acceso. Codec - Codificación + Códec Text is qualified by @@ -302,11 +302,11 @@ Por favor seleccione si desea autorizar su acceso. Original data: - Datos originales: + Dato original: Error(s) detected in CSV file ! - ¡Se detectaron errores en el archivo CSV! + ¡Error(es) detectado(s) en el archivo CSV! more messages skipped] @@ -728,7 +728,7 @@ Do you want to open it anyway? Merge Request - Solicitud de unión + Solicitud de Unión The database file has changed and you have unsaved changes.Do you want to merge your changes? @@ -744,7 +744,7 @@ Do you want to open it anyway? Are you sure you want to permanently delete everything from your recycle bin? - ¿Está seguro(a) que quiere permanentemente eliminar todo de su papelera de reciclaje? + ¿Está seguro que quiere eliminar permanentemente todo de su papelera de reciclaje? @@ -795,7 +795,7 @@ Do you want to open it anyway? Select file - Seleccionar archivo llave + Seleccionar archivo Unable to open file @@ -817,11 +817,11 @@ Do you want to open it anyway? %n week(s) - %n semana%n semanas + %n semana%n semana(s) %n month(s) - %n mes%n meses + %n mes%n mes(es) 1 year @@ -969,7 +969,7 @@ Do you want to open it anyway? Presets - Predeterminado + Programar Notes: @@ -1098,7 +1098,7 @@ Do you want to open it anyway? Custom icon already exists - + El icono personalizado ya existe @@ -1352,7 +1352,7 @@ Esta migración es en único sentido. No podrá abrir la base de datos importada Existing single-instance lock file is invalid. Launching new instance. - + El archivo de bloqueo de instancia única existente no es válido. Lanzando nueva instancia. @@ -1483,7 +1483,7 @@ Esta migración es en único sentido. No podrá abrir la base de datos importada Timed one-time password - Contraseña programada de único uso (TOTP) + Contraseña programada de único uso Copy &TOTP diff --git a/share/translations/keepassx_fr.ts b/share/translations/keepassx_fr.ts index 90580e146..f3fa01920 100644 --- a/share/translations/keepassx_fr.ts +++ b/share/translations/keepassx_fr.ts @@ -29,27 +29,27 @@ Revision: %1 - Revision: %1 + Révision : %1 Libraries: - Bibliothèques: + Bibliothèques : Operating system: %1 CPU architecture: %2 Kernel: %3 %4 - Système d'exploitation: %1 -Architecture CPU: %2 -Kernel: %3 %4 + Système d'exploitation : %1 +Architecture CPU : %2 +Kernel : %3 %4 Enabled extensions: - Extensions activées: + Extensions activées : Report bugs at: <a href="https://github.com/keepassxreboot/keepassxc/issues" style="text-decoration: underline;">https://github.com</a> - Signaler les bugs sur: <a href="https://github.com/keepassxreboot/keepassxc/issues" style="text-decoration: underline;">https://github.com</a> + Signaler les bugs sur : <a href="https://github.com/keepassxreboot/keepassxc/issues" style="text-decoration: underline;">https://github.com</a> KeePassXC is distributed under the terms of the GNU General Public License (GPL) version 2 or (at your option) version 3. @@ -57,7 +57,7 @@ Kernel: %3 %4 Project Maintainers: - Mainteneurs du projet: + Mainteneurs du projet : <a href="https://github.com/keepassxreboot/keepassxc/graphs/contributors">See Contributions on GitHub</a> @@ -65,11 +65,11 @@ Kernel: %3 %4 Include the following information whenever you report a bug: - Inclure l'information suivante lorsque vous signaler un bug: + Inclure l'information suivante lorsque vous signaler un bug : Distribution: %1 - + Distribution : %1 @@ -302,7 +302,7 @@ Veuillez sélectionner si vous souhaitez autoriser l’accès. Original data: - Données originales: + Données originales : Error(s) detected in CSV file ! @@ -397,7 +397,7 @@ Veuillez sélectionner si vous souhaitez autoriser l’accès. Challenge Response: - Challenge-réponse: + Challenge-réponse : @@ -477,15 +477,15 @@ Vous pouvez maintenant la sauvegarder. AES: 256 Bit (default) - AES: 256 Bits (par défault) + AES : 256 Bits (par défaut) Twofish: 256 Bit - Twofish: 256 bits + Twofish : 256 bits Algorithm: - Algorithme: + Algorithme : @@ -737,7 +737,7 @@ Voulez vous l'ouvrir quand même ? Could not open the new database file while attempting to autoreload this database. - La nouvelle base de données ne peux être ouverte pendant qu'un rafraîchissement automatique de l'actuelle est en cours. + La nouvelle base de données ne peut être ouverte pendant qu'un rafraîchissement automatique de l'actuelle est en cours. Empty recycle bin? @@ -1095,11 +1095,11 @@ Voulez vous l'ouvrir quand même ? Hint: You can enable Google as a fallback under Tools>Settings>Security - Astuce: Vous pouvez activer Google en tant que repli sous Outils>Paramètres>Sécurité + Astuce : Vous pouvez activer Google en tant que repli sous Outils>Paramètres>Sécurité Custom icon already exists - + L'icône personnalisée existe déjà @@ -1175,7 +1175,7 @@ Voulez vous l'ouvrir quand même ? Ref: Reference abbreviation - Réf: + Réf : @@ -1353,7 +1353,7 @@ Il s'agit d'une migration à sens unique. Vous ne pourrez pas ouvrir l Existing single-instance lock file is invalid. Launching new instance. - + Le fichier de verrouillage de l’instance unique existant n’est pas valide. Lancement d'une nouvelle instance. @@ -1647,11 +1647,11 @@ Il s'agit d'une migration à sens unique. Vous ne pourrez pas ouvrir l HTTP Port: - Port HTTP: + Port HTTP : Default port: 19455 - Port par défaut: 19455 + Port par défaut : 19455 Re&quest to unlock the database if it is locked @@ -1681,7 +1681,7 @@ Restauration du port 19455 par défaut. &Return advanced string fields which start with "KPH: " - & Retourne les champs avancés de type chaîne qui commencent par "KPH:" + & Retourne les champs avancés de type chaîne qui commencent par "KPH :" Automatically creating or updating string fields is not supported. @@ -1774,7 +1774,7 @@ Ne les changez que si vous savez ce que vous faites. &Length: - &Longueur: + &Longueur : Pick characters from every group @@ -1794,11 +1794,11 @@ Ne les changez que si vous savez ce que vous faites. Entropy: %1 bit - Entropie: %1 bit + Entropie : %1 bit Password Quality: %1 - Qualité du mot de passe: %1 + Qualité du mot de passe : %1 Poor @@ -1830,15 +1830,15 @@ Ne les changez que si vous savez ce que vous faites. Wordlist: - Liste de mots: + Liste de mots : Word Count: - Nombre de mots: + Nombre de mots : Word Separator: - Séparateur de mot: + Séparateur de mot : Copy @@ -2019,7 +2019,7 @@ Veuillez déverrouiller la base de données sélectionnée ou en choisir une qui KeePassXC: New key association request - KeePassXC: nouvelle demande d'association + KeePassXC : nouvelle demande d'association You have received an association request for the above key. @@ -2031,23 +2031,23 @@ attribuez lui un nom unique pour l'identifier et acceptez la. KeePassXC: Overwrite existing key? - KeePassXC: Écraser la clé existante ? + KeePassXC : Écraser la clé existante ? KeePassXC: Update Entry - KeePassXC: Mettre à jour l'entrée + KeePassXC : Mettre à jour l'entrée KeePassXC: Database locked! - KeePassXC: Base de données verrouillée ! + KeePassXC : Base de données verrouillée ! KeePassXC: Removed keys from database - KeePassXC: Les clés ont été effacées de la base de donnée + KeePassXC : Les clés ont été effacées de la base de donnée KeePassXC: No keys found - KeePassXC: Aucune clé trouvée + KeePassXC : Aucune clé trouvée KeePassXC: Settings not available! @@ -2055,11 +2055,11 @@ attribuez lui un nom unique pour l'identifier et acceptez la. KeePassXC: Removed permissions - KeePassXC: Permissions retirées + KeePassXC : Permissions retirées KeePassXC: No entry with permissions found! - KeePassXC: Aucune entrée avec permissions trouvée ! + KeePassXC : Aucune entrée avec permissions trouvée ! @@ -2145,7 +2145,7 @@ attribuez lui un nom unique pour l'identifier et acceptez la. Don't mark database as modified for non-data changes (e.g., expanding groups) - Ne pas indiquer la base de données comme modifiée pour les changements hors-données (par exemple: groupes développés) + Ne pas indiquer la base de données comme modifiée pour les changements hors-données (par exemple : groupes développés) Auto-Type @@ -2227,7 +2227,7 @@ attribuez lui un nom unique pour l'identifier et acceptez la. Key: - Clé: + Clé : Use custom settings @@ -2235,11 +2235,11 @@ attribuez lui un nom unique pour l'identifier et acceptez la. Note: Change these settings only if you know what you are doing. - Attention: modifiez ces paramètres seulement si vous savez ce que vous faites. + Attention : modifiez ces paramètres seulement si vous savez ce que vous faites. Time step: - Période de temps: + Période de temps : 8 digits @@ -2251,7 +2251,7 @@ attribuez lui un nom unique pour l'identifier et acceptez la. Code size: - Taille du code: + Taille du code : sec @@ -2375,7 +2375,7 @@ attribuez lui un nom unique pour l'identifier et acceptez la. Path of the group to list. Default is / - Chemin du groupe à lister. Par défaut: / + Chemin du groupe à lister. Par défaut : / Print the UUIDs of the entries and groups. diff --git a/share/translations/keepassx_id.ts b/share/translations/keepassx_id.ts index f7db658d2..1e04aeb4c 100644 --- a/share/translations/keepassx_id.ts +++ b/share/translations/keepassx_id.ts @@ -69,7 +69,7 @@ Kernel: %3 %4 Distribution: %1 - + Distribusi: %1 diff --git a/share/translations/keepassx_it.ts b/share/translations/keepassx_it.ts index 5125a8b63..582d18970 100644 --- a/share/translations/keepassx_it.ts +++ b/share/translations/keepassx_it.ts @@ -131,7 +131,7 @@ Seleziona se vuoi consentire l'accesso. Auto-Type - KeePassXC - KeePassXC - Auto completamento + KeePassXC - Auto completamento @@ -170,7 +170,7 @@ Seleziona se vuoi consentire l'accesso. Unable to create Key File : - Impossibile creare file chiave: + Impossibile creare file chiave: Select a key file @@ -290,11 +290,11 @@ Seleziona se vuoi consentire l'accesso. Empty fieldname - Nome campo vuoto + Nome campo vuoto column - colonna + colonna Imported from CSV file @@ -302,7 +302,7 @@ Seleziona se vuoi consentire l'accesso. Original data: - Dati originali: + Dati originali: Error(s) detected in CSV file ! @@ -1917,23 +1917,23 @@ Modificale solo se sai quello che stai facendo. QtIOCompressor Internal zlib error when compressing: - Errore interno di zlib durante la compressione: + Errore interno di zlib durante la compressione: Error writing to underlying device: - Errore durante la scrittura nel dispositivo: + Errore durante la scrittura nel dispositivo: Error opening underlying device: - Errore durante l'apertura dal dispositivo: + Errore durante l'apertura dal dispositivo: Error reading data from underlying device: - Errore durante la lettura dal dispositivo: + Errore durante la lettura dal dispositivo: Internal zlib error when decompressing: - Errore interno di zlib durante la decompressione: + Errore interno di zlib durante la decompressione: @@ -1944,7 +1944,7 @@ Modificale solo se sai quello che stai facendo. Internal zlib error: - Errore interno di zlib: + Errore interno di zlib: @@ -2018,7 +2018,7 @@ Sblocca il database selezionato o scegli un altro database sbloccato. KeePassXC: New key association request - KeePassXC: Nuova richiesta di associazione chiave + KeePassXC: Nuova richiesta di associazione chiave You have received an association request for the above key. @@ -2164,7 +2164,7 @@ imposta un nome unico per identificarla ed accettarla. ms - ms + ms Start only a single instance of KeePassXC @@ -2179,7 +2179,7 @@ imposta un nome unico per identificarla ed accettarla. sec - sec + sec Lock databases after inactivity of @@ -2254,7 +2254,7 @@ imposta un nome unico per identificarla ed accettarla. sec - sec + sec diff --git a/share/translations/keepassx_ja.ts b/share/translations/keepassx_ja.ts index 28b7c5336..dd349640f 100644 --- a/share/translations/keepassx_ja.ts +++ b/share/translations/keepassx_ja.ts @@ -69,7 +69,7 @@ CPU アーキテクチャ: %2 Distribution: %1 - + 配布形式: %1 diff --git a/share/translations/keepassx_ko.ts b/share/translations/keepassx_ko.ts index 0b528474d..9b00bbc2f 100644 --- a/share/translations/keepassx_ko.ts +++ b/share/translations/keepassx_ko.ts @@ -69,7 +69,7 @@ CPU 아키텍처: %2 Distribution: %1 - + 배포판: %1 @@ -1096,7 +1096,7 @@ Do you want to open it anyway? Custom icon already exists - + 사용자 정의 아이콘이 이미 존재함 @@ -1350,7 +1350,7 @@ This is a one-way migration. You won't be able to open the imported databas Existing single-instance lock file is invalid. Launching new instance. - + 존재하는 단일 인스턴스 잠금 파일이 잘못되었습니다. 새 인스턴스를 실행합니다. diff --git a/share/translations/keepassx_ru.ts b/share/translations/keepassx_ru.ts index 938d5eefd..3058bcf14 100644 --- a/share/translations/keepassx_ru.ts +++ b/share/translations/keepassx_ru.ts @@ -69,7 +69,7 @@ Kernel: %3 %4 Distribution: %1 - + Дистрибутив: %1 @@ -219,7 +219,7 @@ Please select whether you want to allow access. Replace username and password with references - Заменить имя пользователя и пароль к ссылкам + Использовать ссылки для имени пользователя и пароля Copy history @@ -227,7 +227,7 @@ Please select whether you want to allow access. Append ' - Clone' to title - Добавить' - Клонировать' в заголовок + Добавить к названию « - клон» @@ -579,7 +579,7 @@ Otherwise your changes are lost. "%1" is in edit mode. Discard changes and close anyway? - «%1» в режиме редактирования. + «%1» в режиме правки. Отменить изменения и всё равно закрыть? @@ -741,7 +741,7 @@ Do you want to open it anyway? Empty recycle bin? - Корзина пустая? + Очистить корзину? Are you sure you want to permanently delete everything from your recycle bin? @@ -784,7 +784,7 @@ Do you want to open it anyway? Edit entry - Редактировать запись + Править запись Different passwords supplied. @@ -838,7 +838,7 @@ Do you want to open it anyway? [PROTECTED] Press reveal to view or edit - [Защищён] Нажмите для открытия просмотра или редактирования + [Защищён] Нажмите для открытия просмотра или правки Are you sure you want to remove this attachment? @@ -853,7 +853,7 @@ Do you want to open it anyway? Add - Добавить + Создать Remove @@ -946,7 +946,7 @@ Do you want to open it anyway? EditEntryWidgetMain Title: - Заголовок: + Название: Username: @@ -997,7 +997,7 @@ Do you want to open it anyway? Edit group - Редактировать группу + Править группу Enable @@ -1095,22 +1095,22 @@ Do you want to open it anyway? Hint: You can enable Google as a fallback under Tools>Settings>Security - Подсказка: вы можете включить Google в качестве резервного копирования в меню «Инструменты»> «Настройки»> «Безопасность» + Подсказка: в качестве резервного варианта для получения значков сайтов возможно использовать Google. Включите этот параметр в меню «Инструменты» -> «Настройки» -> «Безопасность» Custom icon already exists - + Пользовательский значок уже существует EditWidgetProperties Created: - Создано: + Создание: Modified: - Изменено: + Изменение: Accessed: @@ -1125,7 +1125,7 @@ Do you want to open it anyway? Entry - Clone - - Клонировать + - клон @@ -1143,7 +1143,7 @@ Do you want to open it anyway? Title - Заголовок + Имя записи Username @@ -1162,7 +1162,7 @@ Do you want to open it anyway? Title - Заголовок + Имя записи Username @@ -1353,7 +1353,7 @@ This is a one-way migration. You won't be able to open the imported databas Existing single-instance lock file is invalid. Launching new instance. - + Запускается новый экземпляр программы, т.к. файл блокировки запуска повреждён. @@ -1452,11 +1452,11 @@ This is a one-way migration. You won't be able to open the imported databas &Add new entry - &Добавить новую запись + &Создать запись &View/Edit entry - &Посмотреть/редактировать запись + &Открыть/править запись &Delete entry @@ -1464,11 +1464,11 @@ This is a one-way migration. You won't be able to open the imported databas &Add new group - &Добавить новую группу + &Создать группу &Edit group - &Редактировать группу + &Править группу &Delete group @@ -1524,7 +1524,7 @@ This is a one-way migration. You won't be able to open the imported databas &Title - &Заголовок + &Имя записи &URL @@ -1556,7 +1556,7 @@ This is a one-way migration. You won't be able to open the imported databas Empty recycle bin - Корзина пустая + Очистить корзину Access error for config file %1 @@ -1701,15 +1701,15 @@ Using default port 19455. &Return only best matching entries - &Возврат только наиболее совпадающих записей + &Показывать только лучшие совпадения Only entries with the same scheme (http://, https://, ftp://, ...) are returned. - Возвращаются только записи с той же схемой (http: //, https: //, ftp: //, ...). + Будут отобраны только записи с совпадающим протоколом (http://, https://, ftp://, ...). &Match URL schemes - &Совпадения схем адресов + &Проверять протокол Password Generator @@ -1879,7 +1879,7 @@ Change them only if you know what you are doing. Title - Заголовок + Имя записи Username @@ -2085,7 +2085,7 @@ give it a unique name to identify and accept it. SettingsWidgetGeneral Remember last databases - Помнить последнюю базу данных + Запоминать последнюю базу данных Automatically save on exit @@ -2117,7 +2117,7 @@ give it a unique name to identify and accept it. Hide window to system tray when minimized - При сворачивании прятать окно в системный лоток + При сворачивании скрывать окно в системный лоток Load previous databases on startup @@ -2129,7 +2129,7 @@ give it a unique name to identify and accept it. Hide window to system tray instead of app exit - Прятать окно в системный лоток вместо выхода + Скрывать окно в системный лоток вместо выхода Minimize window at application startup @@ -2141,11 +2141,11 @@ give it a unique name to identify and accept it. Remember last key files and security dongles - Помнить последние ключевые файлы и ключи безопасности + Запоминать последние использованные файлы ключей и устройства Don't mark database as modified for non-data changes (e.g., expanding groups) - Не помечать базу данных как измененную без изменения данных (например, для расширения групп) + Не помечать базу данных изменённой при действиях, не связанных с изменением данных (например, при распахивании групп) Auto-Type @@ -2153,7 +2153,7 @@ give it a unique name to identify and accept it. Use entry title and URL to match windows for global Auto-Type - Использовать URL и заголовок записи при сопоставлении окон для глобального автоввода + Использовать для поиска URL и название записи Always ask before performing Auto-Type @@ -2184,7 +2184,7 @@ give it a unique name to identify and accept it. Lock databases after inactivity of - Заблокировать базу данных после неактивности длительностью + Блокировать базу данных при отсутствии активности длительностью Show passwords in cleartext by default @@ -2192,7 +2192,7 @@ give it a unique name to identify and accept it. Lock databases after minimizing the window - Блокировать базу данных после сворачивания окна + Блокировать базу данных при сворачивания окна Don't require password repeat when it is visible @@ -2216,7 +2216,7 @@ give it a unique name to identify and accept it. Use Google as fallback for downloading website icons - Использовать Google как резерв для загрузки значков веб-сайтов + Использовать Google в качестве резервного варианта для получения значков веб-сайтов diff --git a/share/translations/keepassx_zh_TW.ts b/share/translations/keepassx_zh_TW.ts index 0dbabb3ab..54614e5cc 100644 --- a/share/translations/keepassx_zh_TW.ts +++ b/share/translations/keepassx_zh_TW.ts @@ -69,7 +69,7 @@ Kernel: %3 %4 Distribution: %1 - + 散佈:%1 @@ -196,7 +196,7 @@ Please select whether you want to allow access. Cha&llenge Response - + 挑戰回應 Refresh @@ -208,7 +208,7 @@ Please select whether you want to allow access. Changing master key failed: no YubiKey inserted. - + 挑戰主金鑰失敗:沒有插入 YubiKey @@ -397,7 +397,7 @@ Please select whether you want to allow access. Challenge Response: - + 挑戰驗證: @@ -1098,7 +1098,7 @@ Do you want to open it anyway? Custom icon already exists - + 自訂圖示已經存在 @@ -2140,7 +2140,7 @@ give it a unique name to identify and accept it. Remember last key files and security dongles - 記住最近的金鑰檔案與安全加密狗 + 記住最近的金鑰檔案與安全鎖 Don't mark database as modified for non-data changes (e.g., expanding groups) diff --git a/snapcraft.yaml b/snapcraft.yaml index 7903eb038..393506f8f 100644 --- a/snapcraft.yaml +++ b/snapcraft.yaml @@ -1,5 +1,5 @@ name: keepassxc -version: 2.2.2 +version: 2.2.3 grade: stable summary: Community-driven port of the Windows application “KeePass Password Safe” description: | diff --git a/src/crypto/SymmetricCipherGcrypt.cpp b/src/crypto/SymmetricCipherGcrypt.cpp index ed031c00f..0b291e693 100644 --- a/src/crypto/SymmetricCipherGcrypt.cpp +++ b/src/crypto/SymmetricCipherGcrypt.cpp @@ -88,6 +88,8 @@ bool SymmetricCipherGcrypt::init() gcry_error_t error; + if(m_ctx != nullptr) + gcry_cipher_close(m_ctx); error = gcry_cipher_open(&m_ctx, m_algo, m_mode, 0); if (error != 0) { setErrorString(error); diff --git a/src/format/KeePass2Repair.cpp b/src/format/KeePass2Repair.cpp index 81ada2fda..8d18457d4 100644 --- a/src/format/KeePass2Repair.cpp +++ b/src/format/KeePass2Repair.cpp @@ -1,5 +1,6 @@ /* * Copyright (C) 2016 Felix Geyer + * Copyright (C) 2017 KeePassXC Team * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -18,36 +19,29 @@ #include "KeePass2Repair.h" #include +#include #include #include "format/KeePass2RandomStream.h" #include "format/KeePass2Reader.h" #include "format/KeePass2XmlReader.h" -KeePass2Repair::KeePass2Repair() - : m_db(nullptr) +KeePass2Repair::RepairOutcome KeePass2Repair::repairDatabase(QIODevice* device, const CompositeKey& key) { -} - -KeePass2Repair::RepairResult KeePass2Repair::repairDatabase(QIODevice* device, const CompositeKey& key) -{ - m_db = nullptr; m_errorStr.clear(); KeePass2Reader reader; reader.setSaveXml(true); - Database* db = reader.readDatabase(device, key, true); + QScopedPointer db(reader.readDatabase(device, key, true)); if (!reader.hasError()) { - delete db; - return NothingTodo; + return qMakePair(NothingTodo, nullptr); } QByteArray xmlData = reader.xmlData(); if (!db || xmlData.isEmpty()) { - delete db; m_errorStr = reader.errorString(); - return UnableToOpen; + return qMakePair(UnableToOpen, nullptr); } bool repairAction = false; @@ -59,8 +53,7 @@ KeePass2Repair::RepairResult KeePass2Repair::repairDatabase(QIODevice* device, c && encodingRegExp.cap(1).compare("utf8", Qt::CaseInsensitive) != 0) { // database is not utf-8 encoded, we don't support repairing that - delete db; - return RepairFailed; + return qMakePair(RepairFailed, nullptr); } } @@ -75,8 +68,7 @@ KeePass2Repair::RepairResult KeePass2Repair::repairDatabase(QIODevice* device, c if (!repairAction) { // we were unable to find the problem - delete db; - return RepairFailed; + return qMakePair(RepairFailed, nullptr); } KeePass2RandomStream randomStream; @@ -84,23 +76,16 @@ KeePass2Repair::RepairResult KeePass2Repair::repairDatabase(QIODevice* device, c KeePass2XmlReader xmlReader; QBuffer buffer(&xmlData); buffer.open(QIODevice::ReadOnly); - xmlReader.readDatabase(&buffer, db, &randomStream); + xmlReader.readDatabase(&buffer, db.data(), &randomStream); if (xmlReader.hasError()) { - delete db; - return RepairFailed; + return qMakePair(RepairFailed, nullptr); } else { - m_db = db; - return RepairSuccess; + return qMakePair(RepairSuccess, db.take()); } } -Database* KeePass2Repair::database() const -{ - return m_db; -} - QString KeePass2Repair::errorString() const { return m_errorStr; diff --git a/src/format/KeePass2Repair.h b/src/format/KeePass2Repair.h index fe2f9dbfe..e7f2c8435 100644 --- a/src/format/KeePass2Repair.h +++ b/src/format/KeePass2Repair.h @@ -1,5 +1,6 @@ /* * Copyright (C) 2016 Felix Geyer + * Copyright (C) 2017 KeePassXC Team * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -20,6 +21,7 @@ #include #include +#include #include "core/Database.h" #include "keys/CompositeKey.h" @@ -36,14 +38,12 @@ public: RepairSuccess, RepairFailed }; + using RepairOutcome = QPair; - KeePass2Repair(); - RepairResult repairDatabase(QIODevice* device, const CompositeKey& key); - Database* database() const; + RepairOutcome repairDatabase(QIODevice* device, const CompositeKey& key); QString errorString() const; private: - Database* m_db; QString m_errorStr; }; diff --git a/src/gui/DatabaseRepairWidget.cpp b/src/gui/DatabaseRepairWidget.cpp index 2b0039408..d3dddf14f 100644 --- a/src/gui/DatabaseRepairWidget.cpp +++ b/src/gui/DatabaseRepairWidget.cpp @@ -1,5 +1,6 @@ /* * Copyright (C) 2016 Felix Geyer + * Copyright (C) 2017 KeePassXC Team * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -69,7 +70,8 @@ void DatabaseRepairWidget::openDatabase() delete m_db; } QApplication::setOverrideCursor(QCursor(Qt::WaitCursor)); - KeePass2Repair::RepairResult repairResult = repair.repairDatabase(&file, masterKey); + auto repairOutcome = repair.repairDatabase(&file, masterKey); + KeePass2Repair::RepairResult repairResult = repairOutcome.first; QApplication::restoreOverrideCursor(); switch (repairResult) { @@ -83,7 +85,7 @@ void DatabaseRepairWidget::openDatabase() emit editFinished(false); return; case KeePass2Repair::RepairSuccess: - m_db = repair.database(); + m_db = repairOutcome.second; MessageBox::warning(this, tr("Success"), tr("The database has been successfully repaired\nYou can now save it.")); emit editFinished(true); return; diff --git a/tests/TestCsvParser.cpp b/tests/TestCsvParser.cpp index 57bc683a2..a292b56bb 100644 --- a/tests/TestCsvParser.cpp +++ b/tests/TestCsvParser.cpp @@ -24,17 +24,12 @@ QTEST_GUILESS_MAIN(TestCsvParser) void TestCsvParser::initTestCase() { - parser = new CsvParser(); -} - -void TestCsvParser::cleanupTestCase() -{ - delete parser; + parser.reset(new CsvParser()); } void TestCsvParser::init() { - file = new QTemporaryFile(); + file.reset(new QTemporaryFile()); if (not file->open()) QFAIL("Cannot open file!"); parser->setBackslashSyntax(false); @@ -51,20 +46,20 @@ void TestCsvParser::cleanup() /****************** TEST CASES ******************/ void TestCsvParser::testMissingQuote() { parser->setTextQualifier(':'); - QTextStream out(file); + QTextStream out(file.data()); out << "A,B\n:BM,1"; QEXPECT_FAIL("", "Bad format", Continue); - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QWARN(parser->getStatus().toLatin1()); } void TestCsvParser::testMalformed() { parser->setTextQualifier(':'); - QTextStream out(file); + QTextStream out(file.data()); out << "A,B,C\n:BM::,1,:2:"; QEXPECT_FAIL("", "Bad format", Continue); - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QWARN(parser->getStatus().toLatin1()); } @@ -72,14 +67,14 @@ void TestCsvParser::testMalformed() { void TestCsvParser::testBackslashSyntax() { parser->setBackslashSyntax(true); parser->setTextQualifier(QChar('X')); - QTextStream out(file); + QTextStream out(file.data()); //attended result: one"\t\"wo out << "Xone\\\"\\\\t\\\\\\\"w\noX\n" << "X13X,X2\\X,X,\"\"3\"X\r" << "3,X\"4\"X,,\n" << "XX\n" << "\\"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.at(0).at(0) == "one\"\\t\\\"w\no"); QVERIFY(t.at(1).at(0) == "13"); @@ -94,10 +89,10 @@ void TestCsvParser::testBackslashSyntax() { } void TestCsvParser::testQuoted() { - QTextStream out(file); + QTextStream out(file.data()); out << "ro,w,\"end, of \"\"\"\"\"\"row\"\"\"\"\"\n" << "2\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.at(0).at(0) == "ro"); QVERIFY(t.at(0).at(1) == "w"); @@ -107,41 +102,41 @@ void TestCsvParser::testQuoted() { } void TestCsvParser::testEmptySimple() { - QTextStream out(file); + QTextStream out(file.data()); out <<""; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 0); } void TestCsvParser::testEmptyQuoted() { - QTextStream out(file); + QTextStream out(file.data()); out <<"\"\""; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 0); } void TestCsvParser::testEmptyNewline() { - QTextStream out(file); + QTextStream out(file.data()); out <<"\"\n\""; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 0); } void TestCsvParser::testEmptyFile() { - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 0); } void TestCsvParser::testNewline() { - QTextStream out(file); + QTextStream out(file.data()); out << "1,2\n\n\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 1); QVERIFY(t.at(0).at(0) == "1"); @@ -150,9 +145,9 @@ void TestCsvParser::testNewline() void TestCsvParser::testCR() { - QTextStream out(file); + QTextStream out(file.data()); out << "1,2\r3,4"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 2); QVERIFY(t.at(0).at(0) == "1"); @@ -163,9 +158,9 @@ void TestCsvParser::testCR() void TestCsvParser::testLF() { - QTextStream out(file); + QTextStream out(file.data()); out << "1,2\n3,4"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 2); QVERIFY(t.at(0).at(0) == "1"); @@ -176,9 +171,9 @@ void TestCsvParser::testLF() void TestCsvParser::testCRLF() { - QTextStream out(file); + QTextStream out(file.data()); out << "1,2\r\n3,4"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 2); QVERIFY(t.at(0).at(0) == "1"); @@ -189,13 +184,13 @@ void TestCsvParser::testCRLF() void TestCsvParser::testComments() { - QTextStream out(file); + QTextStream out(file.data()); out << " #one\n" << " \t # two, three \r\n" << " #, sing\t with\r" << " #\t me!\n" << "useful,text #1!"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 1); QVERIFY(t.at(0).at(0) == "useful"); @@ -203,21 +198,21 @@ void TestCsvParser::testComments() } void TestCsvParser::testColumns() { - QTextStream out(file); + QTextStream out(file.data()); out << "1,2\n" << ",,,,,,,,,a\n" << "a,b,c,d\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(parser->getCsvCols() == 10); } void TestCsvParser::testSimple() { - QTextStream out(file); + QTextStream out(file.data()); out << ",,2\r,2,3\n" << "A,,B\"\n" << " ,,\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 4); QVERIFY(t.at(0).at(0) == ""); @@ -236,11 +231,11 @@ void TestCsvParser::testSimple() { void TestCsvParser::testSeparator() { parser->setFieldSeparator('\t'); - QTextStream out(file); + QTextStream out(file.data()); out << "\t\t2\r\t2\t3\n" << "A\t\tB\"\n" << " \t\t\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 4); QVERIFY(t.at(0).at(0) == ""); @@ -260,10 +255,10 @@ void TestCsvParser::testSeparator() { void TestCsvParser::testMultiline() { parser->setTextQualifier(QChar(':')); - QTextStream out(file); + QTextStream out(file.data()); out << ":1\r\n2a::b:,:3\r4:\n" << "2\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.at(0).at(0) == "1\n2a:b"); QVERIFY(t.at(0).at(1) == "3\n4"); @@ -281,10 +276,10 @@ void TestCsvParser::testEmptyReparsing() void TestCsvParser::testReparsing() { - QTextStream out(file); + QTextStream out(file.data()); out << ":te\r\nxt1:,:te\rxt2:,:end of \"this\n string\":\n" << "2\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QEXPECT_FAIL("", "Wrong qualifier", Continue); @@ -303,10 +298,10 @@ void TestCsvParser::testReparsing() void TestCsvParser::testQualifier() { parser->setTextQualifier(QChar('X')); - QTextStream out(file); + QTextStream out(file.data()); out << "X1X,X2XX,X,\"\"3\"\"\"X\r" << "3,X\"4\"X,,\n"; - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 2); QVERIFY(t.at(0).at(0) == "1"); @@ -324,10 +319,10 @@ void TestCsvParser::testUnicode() { //CORRECT QChar g(0x20AC); //ERROR QChar g("\u20AC"); parser->setFieldSeparator(QChar('A')); - QTextStream out(file); + QTextStream out(file.data()); out << QString("€1A2śA\"3śAż\"Ażac"); - QVERIFY(parser->parse(file)); + QVERIFY(parser->parse(file.data())); t = parser->getCsvTable(); QVERIFY(t.size() == 1); QVERIFY(t.at(0).at(0) == "€1"); diff --git a/tests/TestCsvParser.h b/tests/TestCsvParser.h index 0cf8b94d3..f8c327d63 100644 --- a/tests/TestCsvParser.h +++ b/tests/TestCsvParser.h @@ -22,6 +22,7 @@ #include #include #include +#include #include "core/CsvParser.h" @@ -37,7 +38,6 @@ private slots: void init(); void cleanup(); void initTestCase(); - void cleanupTestCase(); void testUnicode(); void testLF(); @@ -62,8 +62,8 @@ private slots: void testColumns(); private: - QTemporaryFile* file; - CsvParser* parser; + QScopedPointer file; + QScopedPointer parser; CsvTable t; void dumpRow(CsvTable table, int row); }; diff --git a/tests/TestGroup.cpp b/tests/TestGroup.cpp index 2d478bd19..5f654ee3a 100644 --- a/tests/TestGroup.cpp +++ b/tests/TestGroup.cpp @@ -18,9 +18,10 @@ #include "TestGroup.h" -#include -#include #include +#include +#include +#include #include #include "core/Database.h" @@ -75,6 +76,7 @@ void TestGroup::testParenting() QCOMPARE(g3->children().size(), 1); QCOMPARE(g4->children().size(), 0); + QVERIFY(rootGroup->children().at(0) == g1); QVERIFY(rootGroup->children().at(0) == g1); QVERIFY(g1->children().at(0) == g2); QVERIFY(g1->children().at(1) == g3); @@ -99,7 +101,6 @@ void TestGroup::testParenting() g3->setIcon(Uuid::random()); g1->setIcon(2); QCOMPARE(spy.count(), 6); - delete db; QVERIFY(rootGroup.isNull()); @@ -107,7 +108,6 @@ void TestGroup::testParenting() QVERIFY(g2.isNull()); QVERIFY(g3.isNull()); QVERIFY(g4.isNull()); - delete tmpRoot; } @@ -117,18 +117,18 @@ void TestGroup::testSignals() Database* db2 = new Database(); QPointer root = db->rootGroup(); - QSignalSpy spyAboutToAdd(db, SIGNAL(groupAboutToAdd(Group*,int))); + QSignalSpy spyAboutToAdd(db, SIGNAL(groupAboutToAdd(Group*, int))); QSignalSpy spyAdded(db, SIGNAL(groupAdded())); QSignalSpy spyAboutToRemove(db, SIGNAL(groupAboutToRemove(Group*))); QSignalSpy spyRemoved(db, SIGNAL(groupRemoved())); - QSignalSpy spyAboutToMove(db, SIGNAL(groupAboutToMove(Group*,Group*,int))); + QSignalSpy spyAboutToMove(db, SIGNAL(groupAboutToMove(Group*, Group*, int))); QSignalSpy spyMoved(db, SIGNAL(groupMoved())); - QSignalSpy spyAboutToAdd2(db2, SIGNAL(groupAboutToAdd(Group*,int))); + QSignalSpy spyAboutToAdd2(db2, SIGNAL(groupAboutToAdd(Group*, int))); QSignalSpy spyAdded2(db2, SIGNAL(groupAdded())); QSignalSpy spyAboutToRemove2(db2, SIGNAL(groupAboutToRemove(Group*))); QSignalSpy spyRemoved2(db2, SIGNAL(groupRemoved())); - QSignalSpy spyAboutToMove2(db2, SIGNAL(groupAboutToMove(Group*,Group*,int))); + QSignalSpy spyAboutToMove2(db2, SIGNAL(groupAboutToMove(Group*, Group*, int))); QSignalSpy spyMoved2(db2, SIGNAL(groupMoved())); Group* g1 = new Group(); @@ -251,7 +251,7 @@ void TestGroup::testEntries() void TestGroup::testDeleteSignals() { - Database* db = new Database(); + QScopedPointer db(new Database()); Group* groupRoot = db->rootGroup(); Group* groupChild = new Group(); Group* groupChildChild = new Group(); @@ -260,15 +260,13 @@ void TestGroup::testDeleteSignals() groupChildChild->setObjectName("groupChildChild"); groupChild->setParent(groupRoot); groupChildChild->setParent(groupChild); - QSignalSpy spyAboutToRemove(db, SIGNAL(groupAboutToRemove(Group*))); - QSignalSpy spyRemoved(db, SIGNAL(groupRemoved())); + QSignalSpy spyAboutToRemove(db.data(), SIGNAL(groupAboutToRemove(Group*))); + QSignalSpy spyRemoved(db.data(), SIGNAL(groupRemoved())); delete groupChild; QVERIFY(groupRoot->children().isEmpty()); QCOMPARE(spyAboutToRemove.count(), 2); QCOMPARE(spyRemoved.count(), 2); - delete db; - Group* group = new Group(); Entry* entry = new Entry(); @@ -282,7 +280,7 @@ void TestGroup::testDeleteSignals() QCOMPARE(spyEntryRemoved.count(), 1); delete group; - Database* db2 = new Database(); + QScopedPointer db2(new Database()); Group* groupRoot2 = db2->rootGroup(); Group* group2 = new Group(); group2->setParent(groupRoot2); @@ -294,12 +292,11 @@ void TestGroup::testDeleteSignals() delete group2; QCOMPARE(spyEntryAboutToRemove2.count(), 1); QCOMPARE(spyEntryRemoved2.count(), 1); - delete db2; } void TestGroup::testCopyCustomIcon() { - Database* dbSource = new Database(); + QScopedPointer dbSource(new Database()); Uuid groupIconUuid = Uuid::random(); QImage groupIcon(16, 16, QImage::Format_RGB32); @@ -321,7 +318,7 @@ void TestGroup::testCopyCustomIcon() entry->setIcon(entryIconUuid); QCOMPARE(entry->icon(), entryIcon); - Database* dbTarget = new Database(); + QScopedPointer dbTarget(new Database()); group->setParent(dbTarget->rootGroup()); QVERIFY(dbTarget->metadata()->containsCustomIcon(groupIconUuid)); @@ -332,37 +329,34 @@ void TestGroup::testCopyCustomIcon() QVERIFY(dbTarget->metadata()->containsCustomIcon(entryIconUuid)); QCOMPARE(dbTarget->metadata()->customIcon(entryIconUuid), entryIcon); QCOMPARE(entry->icon(), entryIcon); - - delete dbSource; - delete dbTarget; } void TestGroup::testClone() { - Database* db = new Database(); + QScopedPointer db(new Database()); - Group* originalGroup = new Group(); + QScopedPointer originalGroup(new Group()); originalGroup->setParent(db->rootGroup()); originalGroup->setName("Group"); originalGroup->setIcon(42); - Entry* originalGroupEntry = new Entry(); - originalGroupEntry->setGroup(originalGroup); + QScopedPointer originalGroupEntry(new Entry()); + originalGroupEntry->setGroup(originalGroup.data()); originalGroupEntry->setTitle("GroupEntryOld"); originalGroupEntry->setIcon(43); originalGroupEntry->beginUpdate(); originalGroupEntry->setTitle("GroupEntry"); originalGroupEntry->endUpdate(); - Group* subGroup = new Group(); - subGroup->setParent(originalGroup); + QScopedPointer subGroup(new Group()); + subGroup->setParent(originalGroup.data()); subGroup->setName("SubGroup"); - Entry* subGroupEntry = new Entry(); - subGroupEntry->setGroup(subGroup); + QScopedPointer subGroupEntry(new Entry()); + subGroupEntry->setGroup(subGroup.data()); subGroupEntry->setTitle("SubGroupEntry"); - Group* clonedGroup = originalGroup->clone(); + QScopedPointer clonedGroup(originalGroup->clone()); QVERIFY(!clonedGroup->parentGroup()); QVERIFY(!clonedGroup->database()); QVERIFY(clonedGroup->uuid() != originalGroup->uuid()); @@ -387,7 +381,7 @@ void TestGroup::testClone() QVERIFY(clonedSubGroupEntry->uuid() != subGroupEntry->uuid()); QCOMPARE(clonedSubGroupEntry->title(), QString("SubGroupEntry")); - Group* clonedGroupKeepUuid = originalGroup->clone(Entry::CloneNoFlags); + QScopedPointer clonedGroupKeepUuid(originalGroup->clone(Entry::CloneNoFlags)); QCOMPARE(clonedGroupKeepUuid->entries().at(0)->uuid(), originalGroupEntry->uuid()); QCOMPARE(clonedGroupKeepUuid->children().at(0)->entries().at(0)->uuid(), subGroupEntry->uuid()); @@ -406,16 +400,12 @@ void TestGroup::testClone() QCOMPARE(clonedGroupResetTimeInfo->entries().size(), 0); QVERIFY(clonedGroupResetTimeInfo->uuid() != originalGroup->uuid()); QVERIFY(clonedGroupResetTimeInfo->timeInfo().lastModificationTime() != originalGroup->timeInfo().lastModificationTime()); - - delete clonedGroup; - delete clonedGroupKeepUuid; - delete db; } void TestGroup::testCopyCustomIcons() { - Database* dbSource = new Database(); - Database* dbTarget = new Database(); + QScopedPointer dbSource(new Database()); + QScopedPointer dbTarget(new Database()); QImage iconImage1(1, 1, QImage::Format_RGB32); iconImage1.setPixel(0, 0, qRgb(1, 2, 3)); @@ -423,20 +413,20 @@ void TestGroup::testCopyCustomIcons() QImage iconImage2(1, 1, QImage::Format_RGB32); iconImage2.setPixel(0, 0, qRgb(4, 5, 6)); - Group* group1 = new Group(); + QScopedPointer group1(new Group()); group1->setParent(dbSource->rootGroup()); Uuid group1Icon = Uuid::random(); dbSource->metadata()->addCustomIcon(group1Icon, iconImage1); group1->setIcon(group1Icon); - Group* group2 = new Group(); - group2->setParent(group1); + QScopedPointer group2(new Group()); + group2->setParent(group1.data()); Uuid group2Icon = Uuid::random(); dbSource->metadata()->addCustomIcon(group2Icon, iconImage1); group2->setIcon(group2Icon); - Entry* entry1 = new Entry(); - entry1->setGroup(group2); + QScopedPointer entry1(new Entry()); + entry1->setGroup(group2.data()); Uuid entry1IconOld = Uuid::random(); dbSource->metadata()->addCustomIcon(entry1IconOld, iconImage1); entry1->setIcon(entry1IconOld); @@ -463,14 +453,11 @@ void TestGroup::testCopyCustomIcons() QCOMPARE(metaTarget->customIcon(group1Icon).pixel(0, 0), qRgb(1, 2, 3)); QCOMPARE(metaTarget->customIcon(group2Icon).pixel(0, 0), qRgb(4, 5, 6)); - - delete dbTarget; - delete dbSource; } void TestGroup::testFindEntry() { - Database* db = new Database(); + QScopedPointer db(new Database()); Entry* entry1 = new Entry(); entry1->setTitle(QString("entry1")); @@ -541,13 +528,11 @@ void TestGroup::testFindEntry() // An invalid UUID. entry = db->rootGroup()->findEntry(QString("febfb01ebcdf9dbd90a3f1579dc")); QVERIFY(entry == nullptr); - - delete db; } void TestGroup::testFindGroupByPath() { - Database* db = new Database(); + QScopedPointer db(new Database()); Group* group1 = new Group(); group1->setName("group1"); @@ -605,13 +590,11 @@ void TestGroup::testFindGroupByPath() group = db->rootGroup()->findGroupByPath("invalid"); QVERIFY(group == nullptr); - - delete db; } void TestGroup::testPrint() { - Database* db = new Database(); + QScopedPointer db(new Database()); QString output = db->rootGroup()->print(); QCOMPARE(output, QString("[empty]\n")); @@ -651,8 +634,6 @@ void TestGroup::testPrint() output = group1->print(); QVERIFY(!output.contains(QString("group1/\n"))); QVERIFY(output.contains(QString("entry2\n"))); - - delete db; } void TestGroup::testLocate() diff --git a/tests/TestKeePass2Writer.cpp b/tests/TestKeePass2Writer.cpp index 9f0c87be7..f6d3f58ad 100644 --- a/tests/TestKeePass2Writer.cpp +++ b/tests/TestKeePass2Writer.cpp @@ -148,13 +148,15 @@ void TestKeePass2Writer::testRepair() KeePass2Repair repair; QFile file(brokenDbFilename); file.open(QIODevice::ReadOnly); - QCOMPARE(repair.repairDatabase(&file, key), KeePass2Repair::RepairSuccess); - Database* dbRepaired = repair.database(); + auto result = repair.repairDatabase(&file, key); + QCOMPARE(result.first, KeePass2Repair::RepairSuccess); + Database* dbRepaired = result.second; QVERIFY(dbRepaired); QCOMPARE(dbRepaired->rootGroup()->entries().size(), 1); QCOMPARE(dbRepaired->rootGroup()->entries().at(0)->username(), QString("testuser").append(QChar(0x20AC))); QCOMPARE(dbRepaired->rootGroup()->entries().at(0)->password(), QString("testpw")); + delete dbRepaired; } void TestKeePass2Writer::cleanupTestCase() diff --git a/tests/TestSymmetricCipher.cpp b/tests/TestSymmetricCipher.cpp index 74f720a7f..c1e947063 100644 --- a/tests/TestSymmetricCipher.cpp +++ b/tests/TestSymmetricCipher.cpp @@ -202,7 +202,7 @@ void TestSymmetricCipher::testTwofish256CbcEncryption() bool ok; for (int i = 0; i < keys.size(); ++i) { - cipher.init(keys[i], ivs[i]); + QVERIFY(cipher.init(keys[i], ivs[i])); QByteArray ptNext = plainTexts[i]; QByteArray ctPrev = ivs[i]; QByteArray ctCur;