Modify Unref() method of LoaderContext

The ref count should not be decreased when the value is zero.

Change-Id: I88ac89ed6f6fa9af753324723a632779a09eb28b
Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>

diff --git a/src/launchpad-process-pool/launchpad.cc b/src/launchpad-process-pool/launchpad.cc
index 64dab34..62cb876 100644 (file)
--- a/src/launchpad-process-pool/launchpad.cc
+++ b/src/launchpad-process-pool/launchpad.cc
@@ -603,13 +603,15 @@ void Launchpad::OnSigchldReceived(pid_t pid) {
 void Launchpad::OnLoaderPrepared(LoaderContext* loader_context) {
   _W("Loader is prepared. name(%s), pid(%d)",
       loader_context->GetLoaderName().c_str(), loader_context->GetPid());
-  loader_context->Unref();
+  if (loader_context->RefCount() == 0)
+    return;
 
   auto iter = pending_requests_.begin();
   while (iter != pending_requests_.end()) {
     auto request = *iter;
     auto context = request->GetAvailableLoaderContext();
     if (context != nullptr && context.get() == loader_context) {
+      loader_context->Unref();
       pending_requests_.erase(iter);
       LaunchRequestDo(request);
       LaunchRequestComplete(request);

diff --git a/src/launchpad-process-pool/loader_context.cc b/src/launchpad-process-pool/loader_context.cc
index 17b327e..a1b1cfc 100644 (file)
--- a/src/launchpad-process-pool/loader_context.cc
+++ b/src/launchpad-process-pool/loader_context.cc
@@ -571,7 +571,8 @@ void LoaderContext::Ref() {
 }
 
 void LoaderContext::Unref() {
-  ref_count_--;
+  if (ref_count_ > 0)
+    ref_count_--;
 }
 
 uint32_t LoaderContext::RefCount() const {