Skip to content

Commit b8123b2

Browse files
committed
only null terminate payload if there is enough free space
1 parent 1810964 commit b8123b2

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/MQTTClient.cpp

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,14 @@ void MQTTClient_messageHandler(MQTT::MessageData &messageData) {
99
memcpy(topic, messageData.topicName.lenstring.data, (size_t)len);
1010
topic[len] = '\0';
1111

12-
// null terminate payload
12+
// get payload
1313
char * payload = (char *)message.payload;
14-
payload[message.payloadlen] = '\0';
14+
15+
// null terminate payload if enough space is available
16+
if(message.payloadlen < MQTT_BUFFER_SIZE) {
17+
payload[message.payloadlen] = '\0';
18+
}
19+
1520
messageReceived(String(topic), String(payload), (char*)message.payload, (unsigned int)message.payloadlen);
1621
}
1722

0 commit comments

Comments
 (0)