2016-04-17 5 views
2

Следующий метод wifi_json() продолжает увеличивать память, когда я вызываю его в цикле while для тестирования. Это потому, что структуры, используемые в методе, не освобождаются или я не использую бесплатный метод, чтобы освободить память правильно?Память не может быть освобождена в C

#include "MiCO.h" 
#include "json_c/json.h" 
#define os_json_log(M, ...) custom_log("JSON", M, ##__VA_ARGS__) 
void wifi_json() 
{ 
    os_json_log("%s","start"); 
    struct json_object *recv_json_object_device_wifi=NULL; 
    recv_json_object_device_wifi=json_object_new_object();  

    struct json_object *recv_json_object_array_wifi=NULL; 
    recv_json_object_array_wifi=json_object_new_array(); //wifi array 

    struct json_object *recv_json_object_wifi=NULL; //recv wifi 
    recv_json_object_wifi=json_object_new_object(); 

    struct json_object *wifi_object=NULL; //wifi 
    wifi_object=json_object_new_object(); 

    //wifi 
json_object_object_add(wifi_object, "connected", json_object_new_string("1"));  
json_object_object_add(wifi_object, "strength", json_object_new_string("60")); 
json_object_object_add(wifi_object, "ip", json_object_new_string("60")); 
json_object_object_add(wifi_object, "mac", json_object_new_string("60")); 
json_object_object_add(wifi_object, "ssid", json_object_new_string("60")); 
json_object_object_add(recv_json_object_wifi,"WIFI",wifi_object); 

json_object_array_add(recv_json_object_array_wifi,recv_json_object_wifi); 
json_object_array_add(recv_json_object_array_wifi,recv_json_object_wifi); 
json_object_array_add(recv_json_object_array_wifi,recv_json_object_wifi); 

json_object_object_add(recv_json_object_device_wifi,"WIFI_Info",recv_json_object_array_wifi); 
os_json_log("%s",json_object_to_json_string(recv_json_object_device_wifi)); 

free(recv_json_object_device_wifi); 
recv_json_object_device_wifi=NULL; 

free(recv_json_object_array_wifi); 
recv_json_object_array_wifi=NULL; 

json_object_put(recv_json_object_wifi); 
json_object_put(wifi_object); 


json_object_put(recv_json_object_array_wifi); 
json_object_put(recv_json_object_device_wifi); 
} 
int application_start(void) 
{ 
while(1) 
{ 
    wifi_json(); 
    os_json_log("Free memory has %d bytes", MicoGetMemoryInfo()->free_memory) ; 
    } 
    } 

ответ

1

Я думаю, ваша проблема в os_json_log("%s",json_object_to_json_string(recv_json_object_device_wifi));.

Вам следует назначить указатель на него и освободить его позже.

const char *str = json_object_to_json_string(recv_json_object_device_wifi); 
os_json_log("%s", str); 
free(str); 

Далее, просто делает

json_object_put(recv_json_object_device_wifi); 

Достаточно, поскольку это корневой объект, к которому добавляются все остальные. Не нужно звонить json_object_put на любого из своих детей.

2

Вы должны освободить память, созданную этими двумя, а:

struct json_object *recv_json_object_wifi=NULL; //recv wifi 
recv_json_object_wifi=json_object_new_object(); 

struct json_object *wifi_object=NULL; //wifi 
wifi_object=json_object_new_object(); 
+0

Я попытался освободить эти 2 объекта, но не повлиял. бесплатно (recv_json_object_wifi); recv_json_object_wifi = NULL; бесплатный (wifi_object); wifi_object = NULL; –

+1

@vajoja Если вы не позвоните в malloc, вы не должны звонить бесплатно. Поэтому используйте 'json_object_put' вместо' free' – Zaffy

Смежные вопросы