hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use json_object_set_new

Browse Source 
The non-set case should only be used when a copy of an existing object is
needed. This fixes a memory leak in Cache::do_get_info.
This commit is contained in:
Markus Mäkelä
2020-03-09 07:51:08 +02:00
parent 6f45545e09
commit f7d502d1e1
5 changed files with 7 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
server/modules/filter/cache/cache.cc vendored
View File

@ -202,7 +202,7 @@ json_t* Cache::do_get_info(uint32_t what) const
// failure. // failure.
} }
json_object_set(pInfo, "rules", pArray); json_object_set_new(pInfo, "rules", pArray);
} }
} }
} }

3
server/modules/filter/cache/cachept.cc vendored
View File

@ -112,8 +112,7 @@ json_t* CachePT::get_info(uint32_t what) const
if (pThreadInfo) if (pThreadInfo)
{ {
json_object_set(pInfo, key, pThreadInfo); json_object_set_new(pInfo, key, pThreadInfo);
json_decref(pThreadInfo);
} }
} }
} }

3
server/modules/filter/cache/cachesimple.cc vendored
View File

@ -90,8 +90,7 @@ json_t* CacheSimple::do_get_info(uint32_t what) const
if (CACHE_RESULT_IS_OK(result)) if (CACHE_RESULT_IS_OK(result))
{ {
json_object_set(pInfo, "storage", pStorageInfo); json_object_set_new(pInfo, "storage", pStorageInfo);
json_decref(pStorageInfo);
} }
} }

9
server/modules/filter/cache/lrustorage.cc vendored
View File

@ -54,8 +54,7 @@ cache_result_t LRUStorage::do_get_info(uint32_t what,
{ {
m_stats.fill(pLru); m_stats.fill(pLru);
json_object_set(*ppInfo, "lru", pLru); json_object_set_new(*ppInfo, "lru", pLru);
json_decref(pLru);
} }
json_t* pStorage_info; json_t* pStorage_info;
@ -64,8 +63,7 @@ cache_result_t LRUStorage::do_get_info(uint32_t what,
if (CACHE_RESULT_IS_OK(result)) if (CACHE_RESULT_IS_OK(result))
{ {
json_object_set(*ppInfo, "real_storage", pStorage_info); json_object_set_new(*ppInfo, "real_storage", pStorage_info);
json_decref(pStorage_info);
} }
} }
@ -596,8 +594,7 @@ static void set_integer(json_t* pObject, const char* zName, size_t value)
if (pValue) if (pValue)
{ {
json_object_set(pObject, zName, pValue); json_object_set_new(pObject, zName, pValue);
json_decref(pValue);
} }
} }

3
server/modules/filter/cache/storage/storage_inmemory/inmemorystorage.cc vendored
View File

@ -282,8 +282,7 @@ static void set_integer(json_t* pObject, const char* zName, size_t value)
if (pValue) if (pValue)
{ {
json_object_set(pObject, zName, pValue); json_object_set_new(pObject, zName, pValue);
json_decref(pValue);
} }
} }