search

espressif / esp-lwip

Fork of lwIP (https://savannah.nongnu.org/projects/lwip/) with ESP-IDF specific patches
Other
85 stars 129 forks source link

[Bug] Is that wrong? (IDFGH-9835) #54

Closed briliant-ben closed 11 months ago

briliant-ben commented 1 year ago

https://github.com/espressif/esp-lwip/blob/d5e56d06658ae11292be1baea56204f7120b6fa7/test/apps/linux/sys_arch.c#L666

david-cermak commented 1 year ago

It is wrong, yes!

not a big issue though, sys_sem_t is defined as:

typedef struct sys_sem * sys_sem_t;

which means that

sizeof(sys_sem_t*) == sizeof(sys_sem_t) == sizeof(void*)

But it's incorrect and should be fixed, thanks for noticing!

david-cermak commented 11 months ago

Fixed in https://github.com/espressif/esp-lwip/commit/d6b006094bd1cdaa3890da175a9548432f2028a9