summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2016-01-28 12:06:46 +0300
committerDaniel Vetter <daniel.vetter@ffwll.ch>2016-01-28 10:28:15 +0100
commit15b6b804b6e43297d9c05306554a39d8891966f9 (patch)
treec031cb256936cdc5f4f072150489ac7244ccda6d
parent09859d2a3c2090ae2452cbf1c8fdde30ba38d934 (diff)
drm/vmwgfx: fix a NULL dereference
We dereference "eaction->event" inside the call to drm_send_event_locked() so should hold off on setting it to NULL until afterward. Fixes: fb740cf2492c ("drm: Create drm_send_event helpers") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Link: http://patchwork.freedesktop.org/patch/msgid/20160128090646.GA5824@mwanda Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r--drivers/gpu/drm/vmwgfx/vmwgfx_fence.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c b/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c
index e0edf149d9d5..37c305b62608 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_fence.c
@@ -880,8 +880,8 @@ static void vmw_event_fence_action_seq_passed(struct vmw_fence_action *action)
}
list_del_init(&eaction->fpriv_head);
- eaction->event = NULL;
drm_send_event_locked(dev, eaction->event);
+ eaction->event = NULL;
spin_unlock_irqrestore(&dev->event_lock, irq_flags);
}