OXRS-IO / OXRS-IO-TouchPanel-ESP32-FW

OXRS compatible firmware for WT32-xxx touch screen displays
https://oxrs.io/docs/firmware/touch-panel-esp32.html
BSD 3-Clause "New" or "Revised" License
20 stars 3 forks source link

Empty dropdown list causes a reboot, tested on 86s #75

Closed smcgann99 closed 1 year ago

smcgann99 commented 1 year ago

Sorry but it looks like I found another issue - this payload causes a reboot - { "tiles": [ { "screen": 1, "tile": 7, "dropDownList": [] } ] }

moinmoin-sh commented 1 year ago

Yes, that's a bug. The FW should be able to handle this correctly (at least no reboot).

Out of curiosity, what was the reason for sending the empty payload?

smcgann99 commented 1 year ago

Incomplete code on my part, its a flow to get a list of pre-sets from a WLED led controller, if controller was offline then the message was created with an empty list. I added a check for that now, but thought I should mention the issue for the benefit of others.