Send result while handling the launch request

Even if the loader process is not prepared, the launchpad-process-pool
sends the result to the amd. If the initialization of the loader process
is delayed, the caller process cannot get the launched process ID.
To ensure consistency of operation of handling the launch request, the caller
has to get the result from the amd.

Change-Id: Idd523c2a1d3743a39d743833adc801ddb02e2f1a
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 3176a54..d4f0ba4 100644 (file)
--- a/src/launchpad-process-pool/launchpad.cc
+++ b/src/launchpad-process-pool/launchpad.cc
@@ -481,6 +481,7 @@ Launchpad::LaunchResult Launchpad::LaunchRequestDo(
     CheckAndPrepareLoaderContext(loader_context.get());
     CPUBoostController::DoBoost(loader_context->GetPid(),
         CPUBoostController::Level::Strong, 10000);
+    request->SendResult(loader_context->GetPid());
     pending_requests_.push_back(std::move(request));
     return LaunchResult::Pending;
   }

diff --git a/src/launchpad-process-pool/request.cc b/src/launchpad-process-pool/request.cc
index 98a87b5..46d80f6 100644 (file)
--- a/src/launchpad-process-pool/request.cc
+++ b/src/launchpad-process-pool/request.cc
@@ -77,7 +77,11 @@ pid_t Request::GetPid() const {
 }
 
 void Request::SendResult(int result) {
+  if (client_socket_->IsClosed())
+    return;
+
   client_socket_->Send(static_cast<void*>(&result), sizeof(result));
+  client_socket_->Close();
 }
 
 void Request::SetLoaderContext(std::shared_ptr<LoaderContext> context) {