missing va_end in error case

LiamBindle · Mar 24, 2021 · 9d98e1a · 9d98e1a
1 parent e44bef9
commit 9d98e1a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/mqtt.c b/src/mqtt.c
@@ -1482,6 +1482,7 @@ ssize_t mqtt_pack_subscribe_request(uint8_t *buf, size_t bufsz, unsigned int pac
 
         ++num_subs;
         if (num_subs >= MQTT_SUBSCRIBE_REQUEST_MAX_NUM_TOPICS) {
+            va_end(args);
             return MQTT_ERROR_SUBSCRIBE_TOO_MANY_TOPICS;
         }
     }
@@ -1560,6 +1561,7 @@ ssize_t mqtt_pack_unsubscribe_request(uint8_t *buf, size_t bufsz, unsigned int p
 
         ++num_subs;
         if (num_subs >= MQTT_UNSUBSCRIBE_REQUEST_MAX_NUM_TOPICS) {
+            va_end(args);
             return MQTT_ERROR_UNSUBSCRIBE_TOO_MANY_TOPICS;
         }
     }