MQTT::Client::sendPacket send loop uses wrong length

[jpwsutton]

migrated from Bugzilla #460389
status RESOLVED severity normal in component MQTT-Embedded-C for 1.1
Reported in version unspecified on platform Other
Assigned to: Ian Craggs

On 2015-02-19 18:15:55 -0500, Joe Planisky wrote:

MQTT::Client::sendPacket uses a 'while' loop to ensure all data in sendBuf is sent. However, in the call to 'ipstack.write', the length parameter is never updated to account for data already sent.

This line:

    rc = ipstack.write(&sendbuf[sent], length, timer.left_ms());

should be:

    rc = ipstack.write(&sendbuf[sent], length-sent, timer.left_ms());

On 2015-02-20 06:12:09 -0500, Ian Craggs wrote:

Thanks Joe. Fixed in master branch.