diff options
| author |   Florian Fainelli <f.fainelli@gmail.com> | 2017-08-24 16:01:13 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-08-24 18:23:21 -0700 |
| commit | c2062ee3d9615828109ffe8089fbf69bed394d05 (patch) | |
| tree | a9caed4a4de4a9c9f238fd6ab03c572c92807ea1 /net | |
| parent | net: bcmgenet: Be drop monitor friendly (diff) | |
| download | linux-dev-c2062ee3d9615828109ffe8089fbf69bed394d05.tar.xz linux-dev-c2062ee3d9615828109ffe8089fbf69bed394d05.zip | |
net: systemport: Free DMA coherent descriptors on errors
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.
Reported-by: Eric Dumazet <edumazet@gmail.com>
Fixes: 80105befdb4b ("net: systemport: add Broadcom SYSTEMPORT Ethernet MAC driver")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions