Fix static analysis issue

Fix uninitialized member variable
Use of auto that causes a copy

Change-Id: I82802ced0e02ca4db7473f929323b28ae4f4d2c4
Signed-off-by: Ilho Kim <ilho159.kim@samsung.com>

diff --git a/src/pkg_upgrade/src/db_upgrader.cc b/src/pkg_upgrade/src/db_upgrader.cc
index ef8c93147686588fca25965280fb7580df9e816f..e216ce6acfcc8887c614966cd00b4c10f7c57d96 100644 (file)
--- a/src/pkg_upgrade/src/db_upgrader.cc
+++ b/src/pkg_upgrade/src/db_upgrader.cc
@@ -60,7 +60,7 @@ bool DbUpgrader::Upgrade() {
 bool DbUpgrader::SetRwPkg(const std::string& pkgid) {
   try {
     Database database(kDbPath, SQLITE_OPEN_READWRITE);
-    auto q = Database::Sql(kSetRwPkgQuery).Bind(pkgid);
+    auto q = std::move(Database::Sql(kSetRwPkgQuery).Bind(pkgid));
     auto guard = database.CreateTransactionGuard();
     auto r = database.Exec(q);
     if (!r) {

diff --git a/src/pkg_upgrade/src/pkg_upgrader.cc b/src/pkg_upgrade/src/pkg_upgrader.cc
index 9bc2cbd1de2c419a457a1984536742bddaabf003..72f4706e1638ee06b024903b32346a218ed45a80 100644 (file)
--- a/src/pkg_upgrade/src/pkg_upgrader.cc
+++ b/src/pkg_upgrade/src/pkg_upgrader.cc
@@ -23,7 +23,8 @@
 namespace common_fota {
 
 PkgUpgrader::PkgUpgrader(const PkgContext& context, PkgOperation pkg_op)
-  : type_(context.GetType()), loc_(context.GetLocation()), op_(pkg_op),
+  : result_(PkgUpgradeResult::UNKNOWN), type_(context.GetType()),
+    loc_(context.GetLocation()), op_(pkg_op),
     id_(context.GetId()), version_(context.GetVersion()),
     backend_(context.GetId(), context.GetType(), context.GetLocation(),
         pkg_op, context.IsRemovable()) {