drm/msm: fix leaks if initialization fails
authorPavel Machek <pavel@denx.de>
Mon, 9 Mar 2020 10:14:10 +0000 (11:14 +0100)
committerRob Clark <robdclark@chromium.org>
Thu, 19 Mar 2020 19:09:04 +0000 (12:09 -0700)
We should free resources in unlikely case of allocation failure.

Signed-off-by: Pavel Machek <pavel@denx.de>
Signed-off-by: Rob Clark <robdclark@chromium.org>
drivers/gpu/drm/msm/msm_drv.c

index e4b750b..7d985f8 100644 (file)
@@ -444,8 +444,10 @@ static int msm_drm_init(struct device *dev, struct drm_driver *drv)
        if (!dev->dma_parms) {
                dev->dma_parms = devm_kzalloc(dev, sizeof(*dev->dma_parms),
                                              GFP_KERNEL);
-               if (!dev->dma_parms)
-                       return -ENOMEM;
+               if (!dev->dma_parms) {
+                       ret = -ENOMEM;
+                       goto err_msm_uninit;
+               }
        }
        dma_set_max_seg_size(dev, DMA_BIT_MASK(32));