diff options
author | Thomas Hellstrom <thellstrom@vmware.com> | 2012-11-21 10:04:18 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-11-28 05:09:08 -0500 |
commit | d08a9b9cd82f668a5437c23e59063387ca3ecec5 (patch) | |
tree | ba4f33c31d5cf530c244b2ec0fa8bf5865677563 /drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | |
parent | 219e81536c8776b30ccf3262c1ceaeaddf7a231a (diff) |
drm/vmwgfx: Tighten the security around buffer maps
Make sure that other DRM clients can't map the contents of
non-shareable buffer objects.
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c')
-rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c index ef1109c8fec8..96dc84dc34d0 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | |||
@@ -248,13 +248,12 @@ void vmw_evict_flags(struct ttm_buffer_object *bo, | |||
248 | *placement = vmw_sys_placement; | 248 | *placement = vmw_sys_placement; |
249 | } | 249 | } |
250 | 250 | ||
251 | /** | ||
252 | * FIXME: Proper access checks on buffers. | ||
253 | */ | ||
254 | |||
255 | static int vmw_verify_access(struct ttm_buffer_object *bo, struct file *filp) | 251 | static int vmw_verify_access(struct ttm_buffer_object *bo, struct file *filp) |
256 | { | 252 | { |
257 | return 0; | 253 | struct ttm_object_file *tfile = |
254 | vmw_fpriv((struct drm_file *)filp->private_data)->tfile; | ||
255 | |||
256 | return vmw_user_dmabuf_verify_access(bo, tfile); | ||
258 | } | 257 | } |
259 | 258 | ||
260 | static int vmw_ttm_io_mem_reserve(struct ttm_bo_device *bdev, struct ttm_mem_reg *mem) | 259 | static int vmw_ttm_io_mem_reserve(struct ttm_bo_device *bdev, struct ttm_mem_reg *mem) |