Merge pull request #130 from underhood/missing_va_end

missing va_end in error case
LiamBindle · Mar 29, 2021 · 9699785 · 9699785
2 parents 915bdf3 + 9d98e1a
commit 9699785
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;
         }
     }