C client's readPacket can overflow read buffer

[eclipse]

C client's readPacket can overflow read buffer

[icraggs]

I've added a fix for this in readPacket which I believe does most of the job. If the read buffer is very small (less than the header + length of remaining length, between 0 and 5) then that could still be a problem, but buffers of that size are not very useful anyway.

[andrewchambers]

Yes, this library seems to have many buffer overflows and other similar issues, I would not recommend using it when you don't trust your endpoint.

[eden-wang]

I found the problem too. When read buffer is not enough, I disgard the rest data. Here is the main codes

/* erase useless data */
read_bytes = 0;
while (read_bytes < rem_len || expired(timer))
{
    read_bytes += c->ipstack->mqttread(c->ipstack,
            c->readbuf,
            min(c->readbuf_size, rem_len - read_bytes),
            left_ms(timer));
}

[AkremToujani]

Hi Teams,
I'm using paho mqtt client with NUCLEO STM32 board,
I fixed the KeepAlive Interval to 30 min (1800s) and yield(100), but, after 9 hours the client has disconnected.

any help please
thanks a lot