Skip to content

Commit d6370d9

Browse files
authored
Merge pull request #471 from yosuke-wolfssl/fix_fenrir484
Add remain_len check
2 parents 96a6f78 + 68c138e commit d6370d9

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

src/mqtt_packet.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1132,6 +1132,12 @@ int MqttDecode_ConnectAck(byte *rx_buf, int rx_buf_len,
11321132
if (header_len < 0) {
11331133
return header_len;
11341134
}
1135+
1136+
/* Validate remain_len */
1137+
if (remain_len < 2) {
1138+
return MQTT_TRACE_ERROR(MQTT_CODE_ERROR_MALFORMED_DATA);
1139+
}
1140+
11351141
rx_payload = &rx_buf[header_len];
11361142

11371143
/* Decode variable header */

0 commit comments

Comments
 (0)