Fix buffer overflow #3

Merged

thilo-hub merged 1 commit from fixes into main

2024-08-21 12:58:15 +03:00

thilo-hub commented

2024-08-21 10:10:45 +03:00

(Migrated from github.com)

reset_delay should be initialized before malloc to prevent overflow

okhsunrog commented

2024-08-21 12:58:08 +03:00

(Migrated from github.com)

LGTM, thanks for the fix!

You can create a PR with support for different color orders. For example, it can be an enum with 2 options. The value would be set in ws28xx_init and stored in the internal state of the driver, then used in the ws28xx_update function. I might work on this myself in my spare time, but for now, I'm doing all of my new ESP32-C3 projects using Rust, so this library isn't actively supported.

LGTM, thanks for the fix! You can create a PR with support for different color orders. For example, it can be an enum with 2 options. The value would be set in ws28xx_init and stored in the internal state of the driver, then used in the ws28xx_update function. I might work on this myself in my spare time, but for now, I'm doing all of my new ESP32-C3 projects using Rust, so this library isn't actively supported.