search

arduino-libraries / ArduinoMqttClient

ArduinoMqttClient Library for Arduino
GNU Lesser General Public License v2.1
186 stars 73 forks source link

Fix buffer overflow introduced in #44. #71

Closed aentinger closed 2 years ago

aentinger commented 2 years ago

PR #44 introduced a buffer overflow bug that will corrupt the heap if the transmit payload size is increased after any send.

Follow-up of #70.

aentinger commented 2 years ago

Sorry @nooj ... GitHub auto-closed the PR after I cleaned up the PR and pushed the cleanup changes onto your master. Consequentially I was not permitted anymore to revert those changes to your master branch, as github revoked the permission to push there with the auto-closing of the PR. Anyway, I'll be merging this PR right now.

nooj commented 2 years ago

Thanks, @aentinger and @per1234!