Handle skip-check-reference option

A new installer option is introduced. When skip-check-reference flag is
set, installer will skip checking reference at signature validation step.

Requires:
 - https://review.tizen.org/gerrit/139436

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

diff --git a/src/common/installer_context.h b/src/common/installer_context.h
index 348a199..95b1000 100644 (file)
--- a/src/common/installer_context.h
+++ b/src/common/installer_context.h
@@ -357,6 +357,11 @@ class InstallerContext {
    * @brief External package mount object if delta update with external
    */
   std::unique_ptr<ExternalMount> external_mount;
+
+  /**
+   * @brief skip-check-reference flag received from pkgmgr_installer
+   */
+  Property<bool> skip_check_reference;
 };
 
 }  // namespace common_installer

diff --git a/src/common/pkgmgr_interface.cc b/src/common/pkgmgr_interface.cc
index a001fb6..3040fe1 100644 (file)
--- a/src/common/pkgmgr_interface.cc
+++ b/src/common/pkgmgr_interface.cc
@@ -273,4 +273,8 @@ bool PkgMgrInterface::GetDebugMode() const {
   return (pkgmgr_installer_get_debug_mode(pi_) == 1);
 }
 
+bool PkgMgrInterface::GetIsSkipCheckReference() const {
+  return (pkgmgr_installer_get_skip_check_reference(pi_) == 1);
+}
+
 }  // namespace common_installer

diff --git a/src/common/pkgmgr_interface.h b/src/common/pkgmgr_interface.h
index 86beb19..202ce6c 100644 (file)
--- a/src/common/pkgmgr_interface.h
+++ b/src/common/pkgmgr_interface.h
@@ -170,6 +170,15 @@ class PkgMgrInterface {
   bool GetDebugMode() const;
 
   /**
+   * Returns True if the request has skip-check-reference flag.
+   *         Otherwise, return false;
+   *
+   * \return True if the request has skip-check-reference flag.
+   *         Otherwise, return false;
+   */
+  bool GetIsSkipCheckReference() const;
+
+  /**
    * Get Raw pointer to pkgmgr_installer object
    * NOTE: It should not be used (PkgMgrInterface can destroy it
    *

diff --git a/src/common/step/configuration/step_configure.cc b/src/common/step/configuration/step_configure.cc
index e8018fd..79ab53e 100644 (file)
--- a/src/common/step/configuration/step_configure.cc
+++ b/src/common/step/configuration/step_configure.cc
@@ -38,6 +38,7 @@ Step::Status StepConfigure::process() {
   SetupRequestType();
   SetupFileCreationMask();
   SetupDebugMode();
+  SetupSkipCheckReference();
 
   if (!SetupRootAppDirectory())
     return Status::CONFIG_ERROR;
@@ -287,5 +288,9 @@ void StepConfigure::SetupDebugMode() {
   context_->debug_mode.set(pkgmgr_->GetDebugMode());
 }
 
+void StepConfigure::SetupSkipCheckReference() {
+  context_->skip_check_reference.set(pkgmgr_->GetIsSkipCheckReference());
+}
+
 }  // namespace configuration
 }  // namespace common_installer

diff --git a/src/common/step/configuration/step_configure.h b/src/common/step/configuration/step_configure.h
index 45bffb0..a8894fd 100644 (file)
--- a/src/common/step/configuration/step_configure.h
+++ b/src/common/step/configuration/step_configure.h
@@ -66,6 +66,7 @@ class StepConfigure : public Step {
   void SetupIsKeepRWData();
   void SetupIsPartialRW();
   void SetupDebugMode();
+  void SetupSkipCheckReference();
 
   PkgMgrPtr pkgmgr_;
 

diff --git a/src/common/step/security/step_check_signature.cc b/src/common/step/security/step_check_signature.cc
index a5bcaf4..d2411ca 100644 (file)
--- a/src/common/step/security/step_check_signature.cc
+++ b/src/common/step/security/step_check_signature.cc
@@ -151,7 +151,8 @@ Step::Status StepCheckSignature::process() {
       context_->request_type.get() ==
           ci::RequestType::ReadonlyUpdateUninstall ||
       (getuid() == 0 &&
-      (context_->request_type.get() == ci::RequestType::ManifestDirectInstall ||
+      (context_->skip_check_reference.get() ||
+      context_->request_type.get() == ci::RequestType::ManifestDirectInstall ||
       context_->request_type.get() == ci::RequestType::ManifestDirectUpdate ||
       context_->request_type.get() == ci::RequestType::ManifestPartialInstall ||
       context_->request_type.get() == ci::RequestType::ManifestPartialUpdate)))