ArduinoCMRI icon indicating copy to clipboard operation
ArduinoCMRI copied to clipboard

Out of bound checking leads to buffer overflow

Open jlbirccyn opened this issue 6 years ago • 1 comments

Hi Michael

At lines https://github.com/madleech/ArduinoCMRI/blob/master/CMRI.cpp#L107, https://github.com/madleech/ArduinoCMRI/blob/master/CMRI.cpp#L115, https://github.com/madleech/ArduinoCMRI/blob/master/CMRI.cpp#L130, test should be >= to prevent out of bound access of the array.

Best regards

jlbirccyn avatar Dec 30 '19 08:12 jlbirccyn

There seems to be no response, a pity. Thank you for that correction, I modified manually. Erik

Erik84750 avatar Jan 16 '22 13:01 Erik84750