sirleech/Webduino

addCommand

timwilky opened this issue · 1 comments

addCommand does not return a status, therefore there is no indication that use that would exceed the array bounds of the m_commands array has been dropped. Therefore a default maximum of only 8 commands can exist

When the dropped command is then requested, the failure page is then delivered.

quick fix to immediate issue, make the array size user defined in the header to stop dropped commands

open to pull requests on this one. I like the idea of making the array size a #define since that makes it more obvious.