From 234513830ef97aabccba7ef8d7c945c63a1f81c2 Mon Sep 17 00:00:00 2001 From: Sangyoon Jang Date: Fri, 18 Aug 2017 16:38:11 +0900 Subject: [PATCH] Skip security path register when ReadonlyUpdateUninstall Change-Id: I14ec76de118c53005b32d4e4c1aaf8b2dbfa9def Signed-off-by: Sangyoon Jang --- src/common/step/security/step_update_security.cc | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/common/step/security/step_update_security.cc b/src/common/step/security/step_update_security.cc index 95147c8..bc6c3bb 100644 --- a/src/common/step/security/step_update_security.cc +++ b/src/common/step/security/step_update_security.cc @@ -23,14 +23,16 @@ Step::Status StepUpdateSecurity::process() { } return Status::SECURITY_ERROR; } - if (!RegisterSecurityContextForPath( - context_->pkgid.get(), context_->pkg_path.get(), context_->uid.get(), - context_->is_readonly_package.get(), &error_message)) { - if (!error_message.empty()) { - LOG(ERROR) << "error_message: " << error_message; - on_error(Status::SECURITY_ERROR, error_message); + if (context_->request_type.get() != RequestType::ReadonlyUpdateUninstall) { + if (!RegisterSecurityContextForPath( + context_->pkgid.get(), context_->pkg_path.get(), context_->uid.get(), + context_->is_readonly_package.get(), &error_message)) { + if (!error_message.empty()) { + LOG(ERROR) << "error_message: " << error_message; + on_error(Status::SECURITY_ERROR, error_message); + } + return Status::SECURITY_ERROR; } - return Status::SECURITY_ERROR; } LOG(DEBUG) << "Security context updated"; return Status::OK; -- 2.7.4