| Message ID | 20220824093657.3365027-1-yangyingliang@huawei.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [-next] parisc: ccio-dma: Add missing iounmap in error path in ccio_probe() | expand |
On 8/24/22 11:36, Yang Yingliang wrote: > Add missing iounmap() before return from ccio_probe(), if ccio_init_resources() > fails. > > Fixes: d46c742f827f ("parisc: ccio-dma: Handle kmalloc failure in ccio_init_resources()") > Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> applied. Thanks! Helge > --- > drivers/parisc/ccio-dma.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/parisc/ccio-dma.c b/drivers/parisc/ccio-dma.c > index f223afe47d10..a66386043aa6 100644 > --- a/drivers/parisc/ccio-dma.c > +++ b/drivers/parisc/ccio-dma.c > @@ -1546,6 +1546,7 @@ static int __init ccio_probe(struct parisc_device *dev) > } > ccio_ioc_init(ioc); > if (ccio_init_resources(ioc)) { > + iounmap(ioc->ioc_regs); > kfree(ioc); > return -ENOMEM; > }
diff --git a/drivers/parisc/ccio-dma.c b/drivers/parisc/ccio-dma.c index f223afe47d10..a66386043aa6 100644 --- a/drivers/parisc/ccio-dma.c +++ b/drivers/parisc/ccio-dma.c @@ -1546,6 +1546,7 @@ static int __init ccio_probe(struct parisc_device *dev) } ccio_ioc_init(ioc); if (ccio_init_resources(ioc)) { + iounmap(ioc->ioc_regs); kfree(ioc); return -ENOMEM; }
Add missing iounmap() before return from ccio_probe(), if ccio_init_resources() fails. Fixes: d46c742f827f ("parisc: ccio-dma: Handle kmalloc failure in ccio_init_resources()") Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> --- drivers/parisc/ccio-dma.c | 1 + 1 file changed, 1 insertion(+)