> -----Original Message----- > From: Liu Jian <[email protected]> > Sent: Monday, July 20, 2020 5:28 PM > To: Madalin Bucur <[email protected]>; [email protected]; > [email protected]; Laurentiu Tudor <[email protected]>; > [email protected] > Subject: [PATCH v2 net] dpaa_eth: Fix one possible memleak in > dpaa_eth_probe > > When dma_coerce_mask_and_coherent() fails, the alloced netdev need to be > freed. > > Fixes: 060ad66f9795 ("dpaa_eth: change DMA device") > Signed-off-by: Liu Jian <[email protected]> > --- > > v1->v2: > Change targeting from "net-next" to "net" > > drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c > b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c > index 2972244e6eb0..43570f4911ea 100644 > --- a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c > +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c > @@ -2938,7 +2938,7 @@ static int dpaa_eth_probe(struct platform_device > *pdev) > DMA_BIT_MASK(40)); > if (err) { > netdev_err(net_dev, "dma_coerce_mask_and_coherent() > failed\n"); > - return err; > + goto free_netdev; > } > > /* If fsl_fm_max_frm is set to a higher value than the all-common > 1500, > -- > 2.17.1
Acked-by: Madalin Bucur <[email protected]>
