Always try sendPacket at least once, as in the C++ client eclipse-pah…

…o#223
CIPop · Aug 11, 2023 · 3273a8e · 3273a8e
1 parent 023f9a8
commit 3273a8e
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/MQTTClient-C/src/MQTTClient.c b/MQTTClient-C/src/MQTTClient.c
@@ -36,16 +36,18 @@ static int sendPacket(MQTTClient* c, int32_t length, Timer* timer)
     int rc = MQTTCLIENT_FAILURE,
         sent = 0;
 
-    while (sent < length && !TimerIsExpired(timer))
+    while (sent < length)
     {
         rc = c->ipstack->mqttwrite(c->ipstack, &c->buf[sent], length - sent, TimerLeftMS(timer));
-        if (rc < 0)  // there was an error writing the data
+        if (rc < 0)  /* there was an error writing the data */
             break;
         sent += rc;
+        if (TimerIsExpired(timer)) /* only check expiry after at least one attempt to write */
+            break;
     }
     if (sent == length)
     {
-        TimerCountdown(&c->last_sent, c->keepAliveInterval); // record the fact that we have successfully sent the packet
+        TimerCountdown(&c->last_sent, c->keepAliveInterval); /* record the fact that we have successfully sent the packet */
         rc = MQTTCLIENT_SUCCESS;
     }
     else