From: Changgyu Choi <changyu.choi@samsung.com>
Date: Fri, 17 Jan 2025 04:39:18 +0000 (+0900)
Subject: Move CloseAllFds() call after printing logs.
X-Git-Tag: accepted/tizen/unified/20250203.113936~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c07e8c2067b62377e8d03ab2641a3afbaa8f735e;p=platform%2Fcore%2Fappfw%2Flaunchpad.git

Move CloseAllFds() call after printing logs.

CloseAllFds() can close dlog fd.

  F/LIBDLOG  (P 4762, T 2650): libdlog's internal state has been destroyed! The user application closed libdlog's file descriptor.

Change-Id: Ia5eec4b1ad1c91a050ad3fe9193f088628ed4810
Signed-off-by: Changgyu Choi <changyu.choi@samsung.com>
---

diff --git a/src/launchpad-process-pool/app_executor.cc b/src/launchpad-process-pool/app_executor.cc
index 657584a9..bf999a6d 100644
--- a/src/launchpad-process-pool/app_executor.cc
+++ b/src/launchpad-process-pool/app_executor.cc
@@ -184,8 +184,8 @@ void AppExecutor::OnExecution() {
   if (!lib_dir.empty())
     setenv("LD_LIBRARY_PATH", lib_dir.c_str(), 1);
 
-  Util::CloseAllFds();
   SECURE_LOGE("Execute application(%s)", app_argv[LoaderArg::Path]);
+  Util::CloseAllFds();
   if (execv(app_argv[LoaderArg::Path], app_argv) < 0) {
     char err_buf[1024];
     fprintf(stderr, "Failed to execute a file. path: %s, errno: %d(%s)\n",