Fix setting privilege level

- Set PLATFORM level privilege when install preload package.
- Remove setting PLATFORM level privilege when manifest direct install

Change-Id: Ia7786ba79a9825c2aab2bfbb2b1c1ba6890b21a5
Signed-off-by: Sangyoon Jang <s89.jang@samsung.com>

diff --git a/src/common/step/step_check_signature.cc b/src/common/step/step_check_signature.cc
index 9c864ed..635d2f7 100644 (file)
--- a/src/common/step/step_check_signature.cc
+++ b/src/common/step/step_check_signature.cc
@@ -262,6 +262,9 @@ Step::Status StepCheckSignature::process() {
     }
   }
 
+  if (context_->is_preload_request.get())
+    level = PrivilegeLevel::PLATFORM;
+
   LOG(INFO) << "Privilege level: " << PrivilegeLevelToString(level);
   context_->privilege_level.set(level);
 

diff --git a/src/common/step/step_configure.cc b/src/common/step/step_configure.cc
index c9a61eb..d5739b0 100644 (file)
--- a/src/common/step/step_configure.cc
+++ b/src/common/step/step_configure.cc
@@ -85,13 +85,6 @@ Step::Status StepConfigure::process() {
       context_->unpacked_dir_path.set(package_directory);
       context_->pkg_path.set(package_directory);
       context_->xml_path.set(xml_path);
-
-      // TODO(t.iwanek): setting privilege level here should be removed because
-      // of the fact that many apps may not have PLATFORM level. User can
-      // malform platform xml or just reinstall app with direct-manifest
-      // installation mode and gain "system" background category
-      context_->privilege_level.set(PrivilegeLevel::PLATFORM);
-
       break;
     }
     default: