Browse Source

Merge branch 'fix/memory_leaks_identified_by_cppcheck_v3.3' into 'release/v3.3'

Fix/memory leaks identified by cppcheck (v3.3)

See merge request espressif/esp-idf!14505
Mahavir Jain 4 years ago
parent
commit
f69d7cafce

+ 1 - 0
components/esp_http_client/lib/http_auth.c

@@ -51,6 +51,7 @@ static int md5_printf(char *md, const char *fmt, ...)
     va_start(ap, fmt);
     len = vasprintf((char **)&buf, fmt, ap);
     if (buf == NULL) {
+        va_end(ap);
         return ESP_FAIL;
     }
 

+ 1 - 1
components/esp_http_client/lib/http_header.c

@@ -161,8 +161,8 @@ int http_header_set_format(http_header_handle_t header, const char *key, const c
     char *buf = NULL;
     va_start(argptr, format);
     len = vasprintf(&buf, format, argptr);
-    HTTP_MEM_CHECK(TAG, buf, return 0);
     va_end(argptr);
+    HTTP_MEM_CHECK(TAG, buf, return 0);
     if (buf == NULL) {
         return 0;
     }