alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

lwip: port: freertos: Use libc malloc for all allocations

Browse Source 
Mixing lwip and libc allocation functions causes heap failures when
LwIPs MEM_STATS feature is enabled. MEM_STATS uses additional
bookkeeping space that is hidden from the caller. If previously
allocated blocks are free'd without taking this into account the heap
will assert. This commit modifies these calls to consistently use libc
malloc
This commit is contained in:
Eric Johnson 2023-05-29 15:05:25 -05:00
parent 903af13e84
commit 178cee5833
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
components/lwip/port/freertos/sys_arch.c
View File

@ -205,7 +205,7 @@ sys_sem_free(sys_sem_t *sem)
err_t err_t
sys_mbox_new(sys_mbox_t *mbox, int size) sys_mbox_new(sys_mbox_t *mbox, int size)
{ {
*mbox = mem_malloc(sizeof(struct sys_mbox_s)); *mbox = malloc(sizeof(struct sys_mbox_s));
if (*mbox == NULL){ if (*mbox == NULL){
LWIP_DEBUGF(ESP_THREAD_SAFE_DEBUG, ("fail to new *mbox\n")); LWIP_DEBUGF(ESP_THREAD_SAFE_DEBUG, ("fail to new *mbox\n"));
return ERR_MEM; return ERR_MEM;
@ -517,7 +517,7 @@ sys_thread_sem_free(void* data) // destructor for TLS semaphore
sys_sem_t* sys_sem_t*
sys_thread_sem_init(void) sys_thread_sem_init(void)
{ {
sys_sem_t *sem = (sys_sem_t*)mem_malloc(sizeof(sys_sem_t*)); sys_sem_t *sem = (sys_sem_t*)malloc(sizeof(sys_sem_t*));
if (!sem){ if (!sem){
ESP_LOGE(TAG, "thread_sem_init: out of memory"); ESP_LOGE(TAG, "thread_sem_init: out of memory");