diff options
author | Gao Pan <pandy.gao@nxp.com> | 2016-01-08 13:33:15 +0800 |
---|---|---|
committer | Wolfram Sang <wsa@the-dreams.de> | 2016-01-10 10:10:28 +0100 |
commit | c55281531282930de0aace46b2d6b22653937818 (patch) | |
tree | 5c4acab372f83ec5e1312726e949316b9a8b16e0 /drivers | |
parent | a570a27af4d781619dfb5139f3b760e964373c29 (diff) | |
download | op-kernel-dev-c55281531282930de0aace46b2d6b22653937818.zip op-kernel-dev-c55281531282930de0aace46b2d6b22653937818.tar.gz |
i2c: imx: fix i2c resource leak with dma transfer
In i2c_imx_dma_xfer(), when dmaengine_submit() returns error,
the context goto label err_submit and then return. However, the
memory allocated for txdesc has not been freed yet, which
leads to resource leak.
Signed-off-by: Gao Pan <b54642@freescale.com>
Signed-off-by: Fugang Duan <B38611@freescale.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/i2c/busses/i2c-imx.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c index 8b68dbf..a2b132c 100644 --- a/drivers/i2c/busses/i2c-imx.c +++ b/drivers/i2c/busses/i2c-imx.c @@ -387,6 +387,7 @@ static int i2c_imx_dma_xfer(struct imx_i2c_struct *i2c_imx, return 0; err_submit: + dmaengine_terminate_all(dma->chan_using); err_desc: dma_unmap_single(chan_dev, dma->dma_buf, dma->dma_len, dma->dma_data_dir); |