CDPP / AMDA_Kernel

Commit e69479a5afb9c54604548c5f4c4c2b933513ae6d

Authored by Benjamin Renard
1 parent 0ec23281
Exists in master and in 100 other branches 10038, 10098, 5390, 5770, 6728, 7616, 7811, 7875, 8034_7628, 8038, 8041, 8045_8044, 8295, 8459, 8528, 8928, 9016, 9016Bis, 9017, 9017-new, 9049, 9084, 9259, 9326, 9347, 9383, 9719, 9719.new, AllLegends, CDF_ISTP, CDF_ISTP_HOPEFULLY_ALL_WORKING, FER_10400, FER_10412, FER_10663, FER_10765, FER_11000, FER_11100, FER_11242, FER_11277, FER_11500, FER_11537, FER_11588, FER_11651, FER_19836, FER_5344, FER_6031, FER_6598, FER_6966, FER_7318, FER_7618, FER_7857, FER_8455, FER_9629, FER_CR41, FER_CR42, FER_CR43, FER_TFCat, Histo1D2D, JUNO_JEDI, MAZ_10936, MAZ_6018, MAZ_9016_New, MAZ_FIX_9016, SolarOrbiter, SpeasyGet, add_path, akka-centos7, akka-scatter-multi-xparams, amdadev, amdadev_test, centos7, cu2022, das2, density_maps, develop, evol-5638, evol-6371, evol_9042, expression-parser-bre, feature/9259, frames-transformation, livraison-akka, pickle, rm_5639, rm_5776, rm_6015, rm_6595, rm_6599, rm_6616, rm_6616_bis, rm_6912, rm_7926, rm_8050, rm_8932, shue, shue97, sum-components-indexes, test-merge, tickplot, ubuntu

Fix some memory leaks in VOTable reader

Showing 1 changed file with 12 additions and 0 deletions   Show diff stats
Inline Side-by-side
src/ParamGetImpl/LocalFileInterface/FileReaderVOTable.cc
  Diff comments   View file @e69479a
@@ -83,7 +83,10 @@ void FileReaderVOTable::getVOTableInfo(void) { @@ -83,7 +83,10 @@ void FileReaderVOTable::getVOTableInfo(void) {
83 (const xmlChar*) "ID"); 83 (const xmlChar*) "ID");
84 84
85 if (groupId != NULL) 85 if (groupId != NULL)
  86 + {
86 crtGroupId = std::string((const char*) groupId); 87 crtGroupId = std::string((const char*) groupId);
  88 + xmlFree((xmlChar *) groupId);
  89 + }
87 else 90 else
88 crtGroupId = ""; 91 crtGroupId = "";
89 } 92 }
@@ -110,13 +113,22 @@ void FileReaderVOTable::getVOTableInfo(void) { @@ -110,13 +113,22 @@ void FileReaderVOTable::getVOTableInfo(void) {
110 paramInfo->setGroupId(crtGroupId); 113 paramInfo->setGroupId(crtGroupId);
111 114
112 if (paramName != NULL) 115 if (paramName != NULL)
  116 + {
113 paramInfo->setName((const char *) paramName); 117 paramInfo->setName((const char *) paramName);
  118 + xmlFree((xmlChar *) paramName);
  119 + }
114 120
115 if (paramSize != NULL) 121 if (paramSize != NULL)
  122 + {
116 paramInfo->setSize((const char *) paramSize); 123 paramInfo->setSize((const char *) paramSize);
  124 + xmlFree((xmlChar *) paramSize);
  125 + }
117 126
118 if (paramValue != NULL) 127 if (paramValue != NULL)
  128 + {
119 paramInfo->setValue((const char *) paramValue); 129 paramInfo->setValue((const char *) paramValue);
  130 + xmlFree((xmlChar *) paramValue);
  131 + }
120 132
121 LOG4CXX_INFO(gLogger, "PARAM : " << tagName << 133 LOG4CXX_INFO(gLogger, "PARAM : " << tagName <<
122 ", GROUP ID = " << paramInfo->getGroupId() << 134 ", GROUP ID = " << paramInfo->getGroupId() <<