Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adafruit_MQTT (API): the payload of publish method is read-only #114

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adafruit_MQTT (API): the payload of publish method is read-only #114

wants to merge 2 commits into from

Commits on Aug 29, 2020

  1. Adafruit_MQTT::publishPacket: Protect against memory corruption. 

    Avoid memory corruption from happening when data payload provided
in Adafruit_MQTT::publishPacket is greater than MAXBUFFERSIZE.

In order to do that, a helper function is being added to calculate
how much space is available for the payload after subtracting what
is used as the header.

Pull request adafruit#166
Fixes adafruit#109
Fixes adafruit#122

Signed-off-by: Flavio Fernandes <[email protected]>
    @flavio-fernandes
    flavio-fernandes committed Aug 29, 2020
    Configuration menu
    Copy the full SHA
    785b5a4 View commit details
    Browse the repository at this point in the history

  2. Adafruit_MQTT (API): the payload of publish method is read-only 

    The payload param provided in publish is a const. This change
fixes the API to ensure caller that this is the case for all
variations of the publish method.
    @flavio-fernandes
    flavio-fernandes committed Aug 29, 2020
    Configuration menu
    Copy the full SHA
    57009da View commit details
    Browse the repository at this point in the history