From: Hwankyu Jhun <h.jhun@samsung.com>
Date: Wed, 24 Jan 2024 05:00:11 +0000 (+0900)
Subject: Fix crash issue
X-Git-Tag: accepted/tizen/unified/20240124.163236~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0cd97536414fb4a27343ef74488617eb2cb7be6e;hp=7735b4fcaff72fb970f62c25104aa13922a27fe8;p=platform%2Fcore%2Fappfw%2Flaunchpad.git

Fix crash issue

Currently, __stack_chk_fail occurs by array accessment.

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

diff --git a/src/launchpad-process-pool/loader_mount.cc b/src/launchpad-process-pool/loader_mount.cc
index 853a603..b31bb11 100644
--- a/src/launchpad-process-pool/loader_mount.cc
+++ b/src/launchpad-process-pool/loader_mount.cc
@@ -253,19 +253,19 @@ void LoaderMount::OnExecution() {
 
 
 int LoaderMount::CreatePipe(int (*pipe_fd)[2]) {
-  *pipe_fd[0] = -1;
-  *pipe_fd[1] = -1;
+  (*pipe_fd)[0] = -1;
+  (*pipe_fd)[1] = -1;
 
   if (pipe(*pipe_fd) == -1) {
     _E("pipe() is failed. errno(%d)", errno);
     return -1;
   }
 
-  if (fcntl(*pipe_fd[0], F_SETPIPE_SZ, Socket::kSocketMaxBufferSize) == -1)
-    _E("Failed to set pipe size. errno(%d)", errno);
+  if (fcntl((*pipe_fd)[0], F_SETPIPE_SZ, Socket::kSocketMaxBufferSize) == -1)
+    _E("Failed to set pipe size. pipe_fd(%d), errno(%d)", (*pipe_fd)[0], errno);
 
-  if (fcntl(*pipe_fd[1], F_SETPIPE_SZ, Socket::kSocketMaxBufferSize) == -1)
-    _E("Failed to set pipe size. errno(%d)", errno);
+  if (fcntl((*pipe_fd)[1], F_SETPIPE_SZ, Socket::kSocketMaxBufferSize) == -1)
+    _E("Failed to set pipe size. pipe_fd(%d), errno(%d)", (*pipe_fd)[1], errno);
 
   return 0;
 }