Mirror/squeezelite-esp32
1
0
Fork 0
mirror of https://github.com/sle118/squeezelite-esp32.git synced 2025-12-07 03:57:07 +03:00
Code Issues Packages Projects Releases Wiki Activity

solve memory leak in monitor & potentially http

Browse Source
This commit is contained in:
Philippe G
2020-08-14 18:51:59 -07:00
parent f64a37a633
commit ea3c6696e2
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
components/services/monitor.c
View File

@@ -134,8 +134,7 @@ static void monitor_callback(TimerHandle_t xTimer) {
messaging_post_message(MESSAGING_INFO, MESSAGING_CLASS_STATS,top_a); messaging_post_message(MESSAGING_INFO, MESSAGING_CLASS_STATS,top_a);
FREE_AND_NULL(top_a); FREE_AND_NULL(top_a);
} }
cJSON_free(top); cJSON_Delete(top);
} }
/**************************************************************************************** /****************************************************************************************

4
components/wifi-manager/http_server_handlers.c
View File

@@ -481,7 +481,7 @@ esp_err_t console_cmd_get_handler(httpd_req_t *req){
ESP_LOGD_LOC(TAG, "Error retrieving command json string. "); ESP_LOGD_LOC(TAG, "Error retrieving command json string. ");
httpd_resp_send_err(req, HTTPD_500_INTERNAL_SERVER_ERROR, "Unable to format command"); httpd_resp_send_err(req, HTTPD_500_INTERNAL_SERVER_ERROR, "Unable to format command");
} }
cJSON_free(cmdlist); cJSON_Delete(cmdlist);
ESP_LOGD_LOC(TAG, "done serving [%s]", req->uri); ESP_LOGD_LOC(TAG, "done serving [%s]", req->uri);
return err; return err;
} }
@@ -1139,7 +1139,7 @@ esp_err_t messages_get_handler(httpd_req_t *req){
CHECK_RESET_FREE_MEM_CHECK(before, "after http send"); CHECK_RESET_FREE_MEM_CHECK(before, "after http send");
free(json_text); free(json_text);
CHECK_RESET_FREE_MEM_CHECK(before, "after free json message"); CHECK_RESET_FREE_MEM_CHECK(before, "after free json message");
cJSON_free(json_messages); cJSON_Delete(json_messages);
CHECK_RESET_FREE_MEM_CHECK(before, "after free json"); CHECK_RESET_FREE_MEM_CHECK(before, "after free json");
} }
else { else {