Handle allocated memory

Signed-off-by: Ilho Kim <ilho159.kim@samsung.com>

diff --git a/src/common/parcel/appinfo_parcelable.cc b/src/common/parcel/appinfo_parcelable.cc
index 5153201..eb169ed 100644 (file)
--- a/src/common/parcel/appinfo_parcelable.cc
+++ b/src/common/parcel/appinfo_parcelable.cc
@@ -380,7 +380,13 @@ void AppInfoParcelable::ReadSplashscreens(tizen_base::Parcel* parcel, GList** li
 AppInfoParcelable::AppInfoParcelable(std::vector<application_x *>&& app_list)
     : app_list_(std::move(app_list)) {}
 
-const std::vector<application_x *> AppInfoParcelable::GetAppInfo() {
+AppInfoParcelable::~AppInfoParcelable() {
+  for (auto app : app_list_) {
+    pkgmgrinfo_basic_free_application(app);
+  }
+}
+
+const std::vector<application_x *>& AppInfoParcelable::GetAppInfo() {
   return app_list_;
 }
 

diff --git a/src/common/parcel/appinfo_parcelable.hh b/src/common/parcel/appinfo_parcelable.hh
index 959ee7d..4346154 100644 (file)
--- a/src/common/parcel/appinfo_parcelable.hh
+++ b/src/common/parcel/appinfo_parcelable.hh
@@ -23,7 +23,8 @@ class EXPORT_API AppInfoParcelable : public AbstractParcelable {
 
   AppInfoParcelable() = default;
   AppInfoParcelable(std::vector<application_x *>&& app_list);
-  const std::vector<application_x *> GetAppInfo();
+  ~AppInfoParcelable();
+  const std::vector<application_x *>& GetAppInfo();
 
   void WriteToParcel(tizen_base::Parcel* parcel) const override;
   void ReadFromParcel(tizen_base::Parcel* parcel) override;

diff --git a/src/common/parcel/pkginfo_parcelable.cc b/src/common/parcel/pkginfo_parcelable.cc
index 2c0d5ee..464c1a4 100644 (file)
--- a/src/common/parcel/pkginfo_parcelable.cc
+++ b/src/common/parcel/pkginfo_parcelable.cc
@@ -437,7 +437,13 @@ package_x *PkgInfoParcelable::ReadPackage(tizen_base::Parcel* parcel) {
 PkgInfoParcelable::PkgInfoParcelable(std::vector<package_x *>&& pkg_list)
     : pkg_list_(std::move(pkg_list)) {}
 
-const std::vector<package_x *> PkgInfoParcelable::GetPkgInfo() {
+PkgInfoParcelable::~PkgInfoParcelable() {
+  for (auto pkg : pkg_list_) {
+    pkgmgrinfo_basic_free_package(pkg);
+  }
+}
+
+const std::vector<package_x *>& PkgInfoParcelable::GetPkgInfo() {
   return pkg_list_;
 }
 

diff --git a/src/common/parcel/pkginfo_parcelable.hh b/src/common/parcel/pkginfo_parcelable.hh
index f0232f3..a30a31f 100644 (file)
--- a/src/common/parcel/pkginfo_parcelable.hh
+++ b/src/common/parcel/pkginfo_parcelable.hh
@@ -23,7 +23,8 @@ class EXPORT_API PkgInfoParcelable : public AbstractParcelable {
 
   PkgInfoParcelable() = default;
   PkgInfoParcelable(std::vector<package_x *>&& pkg_list);
-  const std::vector<package_x *> GetPkgInfo();
+  ~PkgInfoParcelable();
+  const std::vector<package_x *>& GetPkgInfo();
 
   void WriteToParcel(tizen_base::Parcel* parcel) const override;
   void ReadFromParcel(tizen_base::Parcel* parcel) override;