redonkable/alistair23-linux
redonkable/alistair23-linux
1
0
Fork 0
Code Releases Activity

vmwgfx: memory leaks caused by double allocation

Browse source 
These variables get allocated twice so the first allocation is a
memory leak.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Thomas Hellstrom <thellstrom@vmware.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
This commit is contained in:
Dan Carpenter 2011-10-18 09:09:45 +03:00 committed by Dave Airlie
parent d2c184fb10
commit 0c5d37033b
2 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
drivers/gpu/drm/vmwgfx/vmwgfx_fence.c
View file

@ -953,8 +953,7 @@ int vmw_event_fence_action_create(struct drm_file *file_priv,
uint32_t *tv_usec, uint32_t *tv_usec,
bool interruptible) bool interruptible)
{ {
struct vmw_event_fence_action *eaction = struct vmw_event_fence_action *eaction;
kzalloc(sizeof(*eaction), GFP_KERNEL);
struct ttm_mem_global *mem_glob = struct ttm_mem_global *mem_glob =
vmw_mem_glob(fence->fman->dev_priv); vmw_mem_glob(fence->fman->dev_priv);
struct vmw_fence_manager *fman = fence->fman; struct vmw_fence_manager *fman = fence->fman;

3
drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
View file

@ -1255,8 +1255,7 @@ int vmw_surface_define_ioctl(struct drm_device *dev, void *data,
struct drm_file *file_priv) struct drm_file *file_priv)
{ {
struct vmw_private *dev_priv = vmw_priv(dev); struct vmw_private *dev_priv = vmw_priv(dev);
struct vmw_user_surface *user_srf = struct vmw_user_surface *user_srf;
kmalloc(sizeof(*user_srf), GFP_KERNEL);
struct vmw_surface *srf; struct vmw_surface *srf;
struct vmw_resource *res; struct vmw_resource *res;
struct vmw_resource *tmp; struct vmw_resource *tmp;