search

MaJerle / stm32-ws2811-ws2812-ws2812b-ws281x-tim-pwm-dma-timer

WS2811, WS2812, WS2812B or compatible leds driver with STM32, TIM, PWM and DMA with minimum RAM required
192 stars 43 forks source link

Buffer overflow in led_fill_led_pwm_data() function #8

Closed enrico1036 closed 9 months ago

enrico1036 commented 9 months ago

The row mentioned below leads to a buffer overflow because led_fill_counter is continuously incremented and is never zeroed. But in general I think this row can be deleted because it is useless. The array led_fill_counter_arr in not referenced elsewhere.

https://github.com/MaJerle/stm32-ws2811-ws2812-ws2812b-ws281x-tim-pwm-dma-timer/blob/0b125183043a18487a2707ff331841205ca092fc/projects/stm32g0b1_nucleo_large_buffer/Core/Src/main.c#L326

MaJerle commented 9 months ago

Correct. You can make a pull request?