mirror of
https://github.com/espressif/esp-idf.git
synced 2026-10-01 18:50:34 +03:00
fix(driver_spi): avoid NULL memcpy when private DMA buffer setup fails
Closes https://github.com/espressif/esp-idf/pull/18898
This commit is contained in:
@@ -1178,7 +1178,7 @@ static SPI_MASTER_ISR_ATTR void uninstall_priv_desc(spi_trans_priv_t* trans_buf)
|
||||
|
||||
// copy data from temporary DMA-capable buffer back to trans_desc buffer and free the temporary one.
|
||||
void *orig_rx_buffer = (trans_desc->flags & SPI_TRANS_USE_RXDATA) ? trans_desc->rx_data : trans_desc->rx_buffer;
|
||||
if (trans_buf->buffer_to_rcv != orig_rx_buffer) {
|
||||
if (trans_buf->buffer_to_rcv && trans_buf->buffer_to_rcv != orig_rx_buffer) {
|
||||
memcpy(orig_rx_buffer, trans_buf->buffer_to_rcv, (trans_desc->rxlength + 7) / 8);
|
||||
free(trans_buf->buffer_to_rcv);
|
||||
}
|
||||
|
||||
@@ -414,7 +414,7 @@ static void SPI_SLAVE_ISR_ATTR spi_slave_uninstall_priv_trans(spi_host_device_t
|
||||
if (trans->tx_buffer && (trans->tx_buffer != priv_trans->tx_buffer)) {
|
||||
free(priv_trans->tx_buffer);
|
||||
}
|
||||
if (trans->rx_buffer && (trans->rx_buffer != priv_trans->rx_buffer)) {
|
||||
if (priv_trans->rx_buffer && (trans->rx_buffer != priv_trans->rx_buffer)) {
|
||||
memcpy(trans->rx_buffer, priv_trans->rx_buffer, (trans->length + 7) / 8);
|
||||
free(priv_trans->rx_buffer);
|
||||
}
|
||||
|
||||
@@ -685,7 +685,7 @@ static void s_spi_slave_hd_destroy_priv_trans(spi_host_device_t host, spi_slave_
|
||||
{
|
||||
#if SOC_CACHE_INTERNAL_MEM_VIA_L1CACHE
|
||||
spi_slave_hd_data_t *orig_trans = priv_trans->trans;
|
||||
if (priv_trans->aligned_buffer != orig_trans->data) {
|
||||
if (priv_trans->aligned_buffer && priv_trans->aligned_buffer != orig_trans->data) {
|
||||
if (chan == SPI_SLAVE_CHAN_RX) {
|
||||
memcpy(orig_trans->data, priv_trans->aligned_buffer, orig_trans->trans_len);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user