BUG#36532199: There is a memory leak defect at line 560 in the file /xcom/gcs_xcom_networking

Description: Fixed a memory leak in Gcs_ip_allowlist_entry_hostname::get_value. Previously, the function returned nullptr on error without freeing the allocated memory for a vector. This commit modifies Gcs_ip_allowlist_entry_hostname::get_value to properly delete the vector before returning nullptr in error cases. Change-Id: Iab0076839cccb6c4af7feb3640788ee977f0e9ce

Author: Jaideep Karande

plugin/group_replication/libmysqlgcs/src/bindings/xcom/gcs_xcom_networking.cc

@@ -558,7 +558,10 @@ std::vector<std::pair<std::vector<unsigned char>, std::vector<unsigned char>>>
 
  error = get_address_for_allowlist(ip.second, mask, value);
 
- if (error) return nullptr;
+ if (error) {
+ delete retval;
+ return nullptr;
+ }
 
  retval->push_back(make_pair(value.first, value.second));
  }