Re: [PATCH net] net: systemport: Free DMA coherent descriptors on errors

2017-08-24 Thread David Miller
From: Florian Fainelli Date: Thu, 24 Aug 2017 16:01:13 -0700 > In case bcm_sysport_init_tx_ring() is not able to allocate ring->cbs, we > would return with an error, and call bcm_sysport_fini_tx_ring() and it > would see that ring->cbs is NULL and do nothing. This would leak the > coherent DMA de

[PATCH net] net: systemport: Free DMA coherent descriptors on errors

2017-08-24 Thread Florian Fainelli
In case bcm_sysport_init_tx_ring() is not able to allocate ring->cbs, we would return with an error, and call bcm_sysport_fini_tx_ring() and it would see that ring->cbs is NULL and do nothing. This would leak the coherent DMA descriptor area, so we need to free it on error before returning. Report