search

cbm80amiga / ST7789_AVR

Fast SPI library for the ST7789 IPS display optimized for AVR Arduinos
GNU General Public License v3.0
20 stars 1 forks source link

ST7789_lib_Numeric_display shows stack smash in showVal #1

Closed donwade closed 3 months ago

donwade commented 3 months ago

Retargeted for esp32. This was detected by a trapped backtrace. char txt[10] too small increased to char txt[25] and stack smash went away. Could be a smaller allocation but I don't have enough time to find the best value cause I don't know what this obtusely written function does.

cbm80amiga commented 3 months ago

Thank you for noticing it. Already fixed it - txt buffer should have length of w+1