commit dcd883ff85ecaa52037c01a625487029b844f490
author Gabriel Biren <gbiren@google.com> Tue Feb 21 17:55:07 2023 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Tue Feb 21 17:55:07 2023 +0000
tree bbf70702bb0a0cf9ca29391818616b0a06bf3fad
parent 58ce0f2860f8a4234032e8aca3898b34c6357dce
parent 204e17171b4bbaf171e7565d59143f3cee0a70b9

Merge "Free version buffer in the Vendor HAL's XML parsing logic." into udc-dev

wifi/aidl/default/wifi_legacy_hal_factory.cpp

diff --git a/wifi/aidl/default/wifi_legacy_hal_factory.cpp b/wifi/aidl/default/wifi_legacy_hal_factory.cpp
index 60f81ed..0359d6f 100644
--- a/wifi/aidl/default/wifi_legacy_hal_factory.cpp
+++ b/wifi/aidl/default/wifi_legacy_hal_factory.cpp
@@ -128,7 +128,7 @@
 void WifiLegacyHalFactory::initVendorHalsDescriptorList() {
     xmlDocPtr xml;
     xmlNodePtr node, cnode;
-    char* version;
+    char* version = NULL;
     std::string path;
     xmlChar* value;
     wifi_hal_lib_desc desc;
@@ -200,6 +200,10 @@
         }
     skip:
         xmlFreeDoc(xml);
+        if (version) {
+            xmlFree(version);
+            version = NULL;
+        }
     }
     ::closedir(dirPtr);
 }