search

BojanJurca / Esp32_oscilloscope

ESP32 oscilloscope - see the signals through Web browser the way ESP32 sees them
Creative Commons Zero v1.0 Universal
741 stars 84 forks source link

Memory leak #11

Closed BojanJurca closed 2 years ago

BojanJurca commented 2 years ago

Memory leak is caused by sendBufferSemaphore = xSemaphoreCreateMutex in oscilloscope.h which creates a new semaphore on the heap each time oscilloscope is run.

This is replaced by xSemaphoreCreateBinaryStatic which creates semaphore on the stack.

I'll keep this issue opened until the next release. Meanwhile please replace oscilloscope.h file from release package with the latest one found in repository.

kine90 commented 2 years ago

I am trying to compile the last version, but I get this error: "'xSemaphoreCreateBinaryStatic' was not declared in this scope"

BojanJurca commented 2 years ago

The use of IDF 4.4 fixes the problem.