fix(usb): fix warnings found by GNU static analyzer

This commit is contained in:
Alexey Lapshin
2024-06-18 00:17:15 +07:00
parent 9760884e60
commit 3759f41b94
2 changed files with 8 additions and 3 deletions

View File

@@ -1424,8 +1424,11 @@ static dma_buffer_block_t *buffer_block_alloc(usb_transfer_type_t type)
break; break;
} }
dma_buffer_block_t *buffer = calloc(1, sizeof(dma_buffer_block_t)); dma_buffer_block_t *buffer = calloc(1, sizeof(dma_buffer_block_t));
if (buffer == NULL) {
return NULL;
}
void *xfer_desc_list = heap_caps_aligned_calloc(USB_DWC_QTD_LIST_MEM_ALIGN, desc_list_len, sizeof(usb_dwc_ll_dma_qtd_t), MALLOC_CAP_DMA); void *xfer_desc_list = heap_caps_aligned_calloc(USB_DWC_QTD_LIST_MEM_ALIGN, desc_list_len, sizeof(usb_dwc_ll_dma_qtd_t), MALLOC_CAP_DMA);
if (buffer == NULL || xfer_desc_list == NULL) { if (xfer_desc_list == NULL) {
free(buffer); free(buffer);
heap_caps_free(xfer_desc_list); heap_caps_free(xfer_desc_list);
return NULL; return NULL;

View File

@@ -340,8 +340,10 @@ esp_err_t usb_new_phy(const usb_phy_config_t *config, usb_phy_handle_t *handle_r
return ESP_OK; return ESP_OK;
cleanup: cleanup:
free(phy_context->iopins); if (phy_context) {
free(phy_context); free(phy_context->iopins);
free(phy_context);
}
if (p_phy_ctrl_obj->ref_count == 0) { if (p_phy_ctrl_obj->ref_count == 0) {
free(p_phy_ctrl_obj); free(p_phy_ctrl_obj);
p_phy_ctrl_obj = NULL; p_phy_ctrl_obj = NULL;