Skip to content

Commit 83feb9c

Browse files
Fix issue comments
1 parent af2c957 commit 83feb9c

File tree

2 files changed

+4
-14
lines changed

2 files changed

+4
-14
lines changed

result.txt

Lines changed: 0 additions & 14 deletions
This file was deleted.

source/core_mqtt_serializer.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5410,18 +5410,22 @@ MQTTStatus_t MQTT_DeserializeDisconnect( const MQTTPacketInfo_t * pPacket,
54105410
/* Validate the arguments. */
54115411
if( ( pPacket == NULL ) || ( pPacket->pRemainingData == NULL ) )
54125412
{
5413+
LogError( ( "pPacket and pPacket->pRemainingData must not be NULL." ) );
54135414
status = MQTTBadParameter;
54145415
}
54155416
else if( pDisconnectInfo == NULL )
54165417
{
5418+
LogError( ( "pDisconnectInfo must not be NULL." ) );
54175419
status = MQTTBadParameter;
54185420
}
54195421
else if( maxPacketSize == 0U )
54205422
{
5423+
LogError( ( "maxPacketSize must not be 0." ) );
54215424
status = MQTTBadParameter;
54225425
}
54235426
else if( pPacket->remainingLength >= MQTT_REMAINING_LENGTH_INVALID )
54245427
{
5428+
LogError( ( "pPacket->remainingLength must be less than 268435456." ) );
54255429
status = MQTTBadResponse;
54265430
}
54275431

0 commit comments

Comments
 (0)