Merge branch 'bugfix/coverity_fixes_v5.3' into 'release/v5.3'

fix(nimble): Address fixes for coverity reported issues (v5.3)

See merge request espressif/esp-idf!35369
This commit is contained in:
Rahul Tank
2025-01-05 22:51:20 +08:00
3 changed files with 4 additions and 2 deletions

View File

@ -25,6 +25,7 @@ struct pkt_queue *pkt_queue_create(void)
}
if (osi_mutex_new(&queue->lock) != 0) {
osi_free(queue);
return NULL;
}
struct pkt_queue_header *p = &queue->header;
STAILQ_INIT(p);

View File

@ -170,7 +170,7 @@ static esp_err_t set_config_service(void *config, const char *service_name, cons
/* XXX Does it even make any sense to set truncated mfg_data ? The
* only reason to not return failure from here is provisioning
* should continue as it is with error prints for mfg_data length */
mfg_data_len = MAX_BLE_MANUFACTURER_DATA_LEN - sizeof(ble_config->device_name) - 2;
mfg_data_len = (size_t)(MAX_BLE_MANUFACTURER_DATA_LEN - sizeof(ble_config->device_name) - 2);
}
ble_config->manufacturer_data = custom_manufacturer_data;
@ -213,6 +213,7 @@ static esp_err_t set_config_endpoint(void *config, const char *endpoint_name, ui
realloc(ble_config->nu_lookup, (ble_config->nu_lookup_count + 1) * sizeof(protocomm_ble_name_uuid_t)));
if (!lookup_table) {
ESP_LOGE(TAG, "Error allocating memory for EP-UUID lookup table");
free(copy_ep_name);
return ESP_ERR_NO_MEM;
}