Fix hybrid pkg uninstallation

Removal of last app in pkg removed also pkg information from DB.
This meant that subsequent Cynara policy removal could not calculate
proper Smack label of the app, hence not removing policy & keeping
artifacts in Cynara DB.

Change-Id: Ib647b16f5e0d46e4f31bbaa7b823f04071e827d7

diff --git a/src/common/include/service_impl.h b/src/common/include/service_impl.h
index 85416d06a0d5e12434e7190503f1912773bd927a..f02453a15d11fcb4474a6db3660447fefee3d7c8 100644 (file)
--- a/src/common/include/service_impl.h
+++ b/src/common/include/service_impl.h
@@ -394,7 +394,7 @@ private:
                                 app_inst_req &req,
                                 UninstallHelper &uh);
 
-    void appUninstallCynaraPolicies(app_inst_req::app &app,
+    void appUninstallCynaraPolicies(std::string &processLabel,
                                     app_inst_req &req,
                                     UninstallHelper &uh);
 

diff --git a/src/common/privilege_db.cpp b/src/common/privilege_db.cpp
index fb7fadb4ddabc8e130d98db70f645ea0c2b469e0..644ad7804362d5884490b903f288f5afcc935799 100644 (file)
--- a/src/common/privilege_db.cpp
+++ b/src/common/privilege_db.cpp
@@ -570,7 +570,7 @@ bool PrivilegeDb::IsPackageHybrid(const std::string& pkgName)
         if (command->Step())
             isHybrid = command->GetColumnInteger(0);
 
-        LogDebug("Package " << pkgName << "has is_hybrid set to " << isHybrid);
+        LogDebug("Package " << pkgName << " has is_hybrid set to " << isHybrid);
 
         return (isHybrid > 0);
     });

diff --git a/src/common/service_impl.cpp b/src/common/service_impl.cpp
index 0729bbfe0ff48d2222f3f3f89f839122fdadf751..084a4bf51f74e6e8e26eb543da8c2a7665bbbe70 100644 (file)
--- a/src/common/service_impl.cpp
+++ b/src/common/service_impl.cpp
@@ -1003,11 +1003,10 @@ void ServiceImpl::appUninstallPrivileges(app_inst_req::app &app, app_inst_req &r
     uh.removeApps.push_back(removeApp);
 }
 
-void ServiceImpl::appUninstallCynaraPolicies(app_inst_req::app &app, app_inst_req &req,
+void ServiceImpl::appUninstallCynaraPolicies(std::string &processLabel, app_inst_req &req,
         UninstallHelper &ui)
 {
-    std::string processLabel = getAppProcessLabel(app.appName, req.pkgName);
-    LogDebug("Generated uninstall parameters: pkgName=" << req.pkgName
+    LogDebug("Removing Cynara policy for: pkgName=" << req.pkgName
              << " Smack label=" << processLabel);
 
     bool global = req.installationType == SM_APP_INSTALL_GLOBAL ||
@@ -1100,16 +1099,25 @@ int ServiceImpl::appUninstall(const Credentials &creds, app_inst_req &req)
         m_privilegeDb.GetPackagesInfo(uh.pkgsInfo);
         getPkgsProcessLabels(uh.pkgsInfo, uh.pkgsProcessLabels);
 
+        LogDebug("Uninstalling pkg: " << req.pkgName << " with " << req.apps.size() << " apps");
+        for (auto &app: req.apps) {
+            LogDebug("-- Removing app: " << app.appName);
+        }
+
         // [db] begin
         ScopedTransaction trans(m_privilegeDb);
 
         for (auto &app : req.apps) {
             if (app.appName.empty())
                 continue;
+            // Cynara client (Smack label) needs to be calculated before we'll remove app from privileges db
+            // Otherwise, last app being removed from pkg will not be able to determine correct Smack label
+            // if the pkg is hybrid
+            std::string processLabel = getAppProcessLabel(app.appName, req.pkgName);
             // [db] remove app
             appUninstallPrivileges(app, req, uh);
             // [cynara] update app policy
-            appUninstallCynaraPolicies(app, req, uh);
+            appUninstallCynaraPolicies(processLabel, req, uh);
         }
 
         // [db] commit