Fix memory leak

After using glob(), the glob_t should be freed using globfree()
After using xmlReadFile(), the return value shoud be freed using xmlFreeDoc()

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

diff --git a/src/wgt/step/security/step_check_extension_privileges.cc b/src/wgt/step/security/step_check_extension_privileges.cc
index affeb47..796022d 100755 (executable)
--- a/src/wgt/step/security/step_check_extension_privileges.cc
+++ b/src/wgt/step/security/step_check_extension_privileges.cc
@@ -60,6 +60,7 @@ common_installer::Step::Status StepCheckExtensionPrivileges::process() {
     for (unsigned int i = 0; i < glob_result.gl_pathc; ++i) {
       xmlFiles.insert(glob_result.gl_pathv[i]);
     }
+    globfree(&glob_result);
   }
   GList* privileges = nullptr;
   BOOST_SCOPE_EXIT_ALL(privileges) {

diff --git a/src/wgt/utils/extension_config_parser.cc b/src/wgt/utils/extension_config_parser.cc
index 7a448a4..684c292 100644 (file)
--- a/src/wgt/utils/extension_config_parser.cc
+++ b/src/wgt/utils/extension_config_parser.cc
@@ -42,6 +42,7 @@ std::unique_ptr<parser::DictionaryValue>
   std::unique_ptr<parser::DictionaryValue> result(new parser::DictionaryValue);
   if (dv)
     result->Set(reinterpret_cast<const char*>(root_node->name), dv.release());
+  xmlFreeDoc(doc);
   return result;
 }