From: Piotr Kosko/Tizen API (PLT) /SRPOL/Engineer/Samsung Electronics <p.kosko@samsung.com>
Date: Fri, 5 Apr 2024 06:15:25 +0000 (+0200)
Subject: [package] Fixed potential dereference of nullptr
X-Git-Tag: accepted/tizen/unified/20240410.084246^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fheads%2Faccepted%2Ftizen_unified_x;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[package] Fixed potential dereference of nullptr

[Verification] Compiles without error.

Change-Id: Ic3af3b88a53077bacb7cbe4abfc17038910b4557
---

diff --git a/src/package/package_info_provider.cc b/src/package/package_info_provider.cc
index 5844fac..acd077d 100644
--- a/src/package/package_info_provider.cc
+++ b/src/package/package_info_provider.cc
@@ -234,6 +234,11 @@ namespace {
 void GetSize(const std::string& id, int service_mode, picojson::object* out) {
   ScopeLogger();
   pkgmgr_client* pc = pkgmgr_client_new(PC_REQUEST);
+  if (!pc) {
+    LogAndReportError(
+        PlatformResult(ErrorCode::UNKNOWN_ERR, "Failed to get size"), out,
+        ("pkgmgr_client_new returned nullptr"));
+  }
   int size = pkgmgr_client_usr_request_service(PM_REQUEST_GET_SIZE, service_mode, pc, NULL,
                                                id.c_str(), getuid(), NULL, NULL, NULL);
   pkgmgr_client_free(pc);