Fix memory leak

xmlDocPtr allocated by xmlParseFile should be freed

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

diff --git a/src/tpk/step/pkgmgr/step_convert_xml.cc b/src/tpk/step/pkgmgr/step_convert_xml.cc
index af824d7..9a1d1a5 100644 (file)
--- a/src/tpk/step/pkgmgr/step_convert_xml.cc
+++ b/src/tpk/step/pkgmgr/step_convert_xml.cc
@@ -100,6 +100,8 @@ bool StepConvertXml::ConvertXml(xmlDocPtr doc) {
 
 common_installer::Step::Status StepConvertXml::process() {
   xmlDocPtr doc = xmlParseFile(xml_path_.string().c_str());
+  auto doc_ptr = std::unique_ptr<std::remove_pointer<xmlDocPtr>::type,
+      decltype(xmlFreeDoc)*>(doc, xmlFreeDoc);
   if (!doc) {
     LOG(ERROR) << "Failed to parse xml file";
     return Step::Status::MANIFEST_ERROR;