From ff4eae5bb1485bb0fe93d636cacdddf2af4d2842 Mon Sep 17 00:00:00 2001 From: morris Date: Tue, 30 Jun 2026 16:52:26 +0800 Subject: [PATCH] fix(spi_slave): free DMA-private buffers when transaction queue is full spi_slave_queue_trans calls spi_slave_setup_priv_trans to allocate DMA buffers, then tries xQueueSend. If the queue is full the function returns ESP_ERR_TIMEOUT without freeing those buffers, leaking up to 2 * max_transfer_sz per failed call. Call spi_slave_uninstall_priv_trans before returning the timeout. --- components/esp_driver_spi/src/gpspi/spi_slave.c | 1 + 1 file changed, 1 insertion(+) diff --git a/components/esp_driver_spi/src/gpspi/spi_slave.c b/components/esp_driver_spi/src/gpspi/spi_slave.c index 4251b6dc44e..986c79fa815 100644 --- a/components/esp_driver_spi/src/gpspi/spi_slave.c +++ b/components/esp_driver_spi/src/gpspi/spi_slave.c @@ -452,6 +452,7 @@ esp_err_t SPI_SLAVE_ATTR spi_slave_queue_trans(spi_host_device_t host, const spi r = xQueueSend(spihost[host]->trans_queue, (void *)&priv_trans, ticks_to_wait); if (!r) { + spi_slave_uninstall_priv_trans(host, &priv_trans); return ESP_ERR_TIMEOUT; } esp_intr_enable(spihost[host]->intr);