diff options
author | Robert Griebl <robert.griebl@qt.io> | 2021-08-15 00:36:19 +0200 |
---|---|---|
committer | Robert Griebl <robert.griebl@qt.io> | 2021-08-17 11:44:37 +0200 |
commit | d92730320d4e6af4c7c839ff1b941870b6d5ab14 (patch) | |
tree | 61830570279093cf5245a177f9fed2ea0a045040 | |
parent | bd366525d7c0bc65d711b772a70b223f7c86651c (diff) | |
download | qtapplicationmanager-d92730320d4e6af4c7c839ff1b941870b6d5ab14.tar.gz |
Clazy: fix potential crashes
Pick-to: 5.15
Change-Id: Idb99f9d668850486afa49164bbcfe906efd50caf
Reviewed-by: Dominik Holland <dominik.holland@qt.io>
-rw-r--r-- | src/manager-lib/package.cpp | 6 | ||||
-rw-r--r-- | src/manager-lib/packagemanager.cpp | 3 | ||||
-rw-r--r-- | src/notification-lib/notification.cpp | 2 |
3 files changed, 7 insertions, 4 deletions
diff --git a/src/manager-lib/package.cpp b/src/manager-lib/package.cpp index e7226cb6..3b68b402 100644 --- a/src/manager-lib/package.cpp +++ b/src/manager-lib/package.cpp @@ -205,7 +205,8 @@ QString Package::name() const QVariantMap Package::names() const { QVariantMap names; - for (auto it = info()->names().cbegin(); it != info()->names().cend(); ++it) + const auto nmap = info()->names(); + for (auto it = nmap.cbegin(); it != nmap.cend(); ++it) names.insert(it.key(), it.value()); return names; } @@ -228,7 +229,8 @@ QString Package::description() const QVariantMap Package::descriptions() const { QVariantMap descriptions; - for (auto it = info()->descriptions().cbegin(); it != info()->descriptions().cend(); ++it) + const auto dmap = info()->descriptions(); + for (auto it = dmap.cbegin(); it != dmap.cend(); ++it) descriptions.insert(it.key(), it.value()); return descriptions; } diff --git a/src/manager-lib/packagemanager.cpp b/src/manager-lib/packagemanager.cpp index 61e41546..16bafa0b 100644 --- a/src/manager-lib/packagemanager.cpp +++ b/src/manager-lib/packagemanager.cpp @@ -1443,10 +1443,11 @@ bool PackageManager::finishedPackageInstall(const QString &id) // cleanup package->unblock(); - delete package; // remove the package from the package db d->database->removePackageInfo(package->info()); + + delete package; break; } } diff --git a/src/notification-lib/notification.cpp b/src/notification-lib/notification.cpp index c9ef3032..87e65d0a 100644 --- a/src/notification-lib/notification.cpp +++ b/src/notification-lib/notification.cpp @@ -76,7 +76,7 @@ Notification::Notification(QObject *parent, Notification::ConstructionMode mode) : QObject(parent) { if (mode == Dynamic) - componentComplete(); + Notification::componentComplete(); } /*! |