drm/msm: fix some crashes in submit fail path
authorRob Clark <robdclark@gmail.com>
Tue, 24 May 2016 22:43:26 +0000 (18:43 -0400)
committerRob Clark <robdclark@gmail.com>
Sat, 4 Jun 2016 18:45:49 +0000 (14:45 -0400)
If submit fails, before fence is created or before submit is added to
submit-list, then unitialized fields cause problems in the clean-up
path.

Signed-off-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/msm/msm_gem_submit.c

index b89ca5174863e7ab73c48117605ded1d56626e19..635eff17c3d116545c75d3e1a4d5cd9d652a0fca 100644 (file)
@@ -40,12 +40,14 @@ static struct msm_gem_submit *submit_create(struct drm_device *dev,
 
        submit->dev = dev;
        submit->gpu = gpu;
+       submit->fence = NULL;
        submit->pid = get_pid(task_pid(current));
 
        /* initially, until copy_from_user() and bo lookup succeeds: */
        submit->nr_bos = 0;
        submit->nr_cmds = 0;
 
+       INIT_LIST_HEAD(&submit->node);
        INIT_LIST_HEAD(&submit->bo_list);
        ww_acquire_init(&submit->ticket, &reservation_ww_class);