dmaengine: imx-sdma: fix use-after-free on probe error path
authorSven Van Asbroeck <thesven73@gmail.com>
Mon, 24 Jun 2019 14:07:31 +0000 (10:07 -0400)
committerVinod Koul <vkoul@kernel.org>
Fri, 5 Jul 2019 07:28:54 +0000 (12:58 +0530)
commit2b8066c3deb9140fdf258417a51479b2aeaa7622
tree815c860736e7ddbec938d8370d7d6caab4a93635
parent4c89cc73d1da42ae48b5c5dfbfd12304d0b86786
dmaengine: imx-sdma: fix use-after-free on probe error path

If probe() fails anywhere beyond the point where
sdma_get_firmware() is called, then a kernel oops may occur.

Problematic sequence of events:
1. probe() calls sdma_get_firmware(), which schedules the
   firmware callback to run when firmware becomes available,
   using the sdma instance structure as the context
2. probe() encounters an error, which deallocates the
   sdma instance structure
3. firmware becomes available, firmware callback is
   called with deallocated sdma instance structure
4. use after free - kernel oops !

Solution: only attempt to load firmware when we're certain
that probe() will succeed. This guarantees that the firmware
callback's context will remain valid.

Note that the remove() path is unaffected by this issue: the
firmware loader will increment the driver module's use count,
ensuring that the module cannot be unloaded while the
firmware callback is pending or running.

Signed-off-by: Sven Van Asbroeck <TheSven73@gmail.com>
Reviewed-by: Robin Gong <yibin.gong@nxp.com>
[vkoul: fixed braces for if condition]
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/imx-sdma.c