Fix static analysis issues 02/290902/1
authorHwankyu Jhun <h.jhun@samsung.com>
Tue, 4 Apr 2023 22:55:30 +0000 (22:55 +0000)
committerHwankyu Jhun <h.jhun@samsung.com>
Tue, 4 Apr 2023 22:55:30 +0000 (22:55 +0000)
Fixed:
 - resource leak
 - forward null

Change-Id: I24116eb324737cdb27f1780d8fed244a3458d9fc
Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>
src/launchpad-process-pool/launchpad.cc

index 4634cfc..6885aed 100644 (file)
@@ -2463,6 +2463,7 @@ void Launchpad::HandlePrepareAppDefinedLoaderRequest(
     if (context == nullptr) {
       _E("Out of memory");
       request->SendResult(-ENOMEM);
+      return;
     }
   } else {
     loader_id = context->loader_id;
@@ -2666,8 +2667,10 @@ void Launchpad::OnIOEventReceived(int fd, int condition) {
   auto pkt_auto = std::unique_ptr<app_pkt_t, decltype(std::free)*>(
       pkt, std::free);
   auto* kb = bundle_decode(pkt->data, pkt->len);
-  if (kb == nullptr)
+  if (kb == nullptr) {
+    close(client_fd);
     return;
+  }
 
   tizen_base::Bundle b(kb, false, true);
   auto request = std::make_shared<Request>(