Skip to content

Commit 8338700

Browse files
Merge branch 'fix/gcc-fanalyzer-warnings' into 'master'
fix: gcc -fanalyzer warnings See merge request espressif/esp-mqtt!209
2 parents d27bd52 + 8bc3bff commit 8338700

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

mqtt_client.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -944,13 +944,17 @@ esp_err_t esp_mqtt_client_set_uri(esp_mqtt_client_handle_t client, const char *u
944944
// This API could be also executed when client is active (need to protect config fields)
945945
MQTT_API_LOCK(client);
946946
// set uri overrides actual scheme, host, path if configured previously
947+
// False-positive leak detection. TODO: GCC-366
948+
#pragma GCC diagnostic push
949+
#pragma GCC diagnostic ignored "-Wanalyzer-malloc-leak"
947950
free(client->config->scheme);
948951
free(client->config->host);
949952
free(client->config->path);
950953

951954
client->config->scheme = create_string(uri + puri.field_data[UF_SCHEMA].off, puri.field_data[UF_SCHEMA].len);
952955
client->config->host = create_string(uri + puri.field_data[UF_HOST].off, puri.field_data[UF_HOST].len);
953956
client->config->path = NULL;
957+
#pragma GCC diagnostic pop
954958

955959
if (puri.field_data[UF_PATH].len || puri.field_data[UF_QUERY].len) {
956960
int asprintf_ret_value;

0 commit comments

Comments
 (0)