kiss-lang/lime
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Memory leak fix in curl bindings (#1856)

Browse Source
This commit is contained in:
player-03
2025-08-31 23:20:16 -04:00
committed by GitHub
parent 0b1ab0a245 65520709d3
commit 2bdd41ef57
1 changed files with 21 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
project/src/net/curl/CURLBindings.cpp
View File

@@ -57,6 +57,19 @@ namespace lime {
std::map<void*, CURL_XferInfo*> xferInfoValues;
Mutex curl_gc_mutex;
void free_header_values (const std::vector<char*>* values) {
if (values) {
for (auto it = values->begin (); it != values->end (); ++it) {
free (*it);
}
}
}
void gc_curl (value handle) {
@@ -121,6 +134,7 @@ namespace lime {
std::vector<char*>* values = headerValues[handle];
headerCallbacks.erase (handle);
headerValues.erase (handle);
free_header_values (values);
delete callback;
delete values;
@@ -254,6 +268,7 @@ namespace lime {
std::vector<char*>* values = headerValues[handle];
headerCallbacks.erase (handle);
headerValues.erase (handle);
free_header_values (values);
delete callback;
delete values;
@@ -581,6 +596,7 @@ namespace lime {
}
free_header_values (values);
values->clear ();
}
@@ -688,6 +704,7 @@ namespace lime {
}
free_header_values (values);
values->clear ();
}
@@ -1691,8 +1708,9 @@ namespace lime {
{
curl_gc_mutex.Lock ();
if (headerCallbacks.find (handle) == headerCallbacks.end ()) {
if (headerCallbacks.find (handle) != headerCallbacks.end ()) {
free_header_values (headerValues[handle]);
delete headerCallbacks[handle];
delete headerValues[handle];
@@ -2118,8 +2136,9 @@ namespace lime {
{
curl_gc_mutex.Lock ();
if (headerCallbacks.find (handle) == headerCallbacks.end ()) {
if (headerCallbacks.find (handle) != headerCallbacks.end ()) {
free_header_values (headerValues[handle]);
delete headerCallbacks[handle];
delete headerValues[handle];