diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-11 15:00:56 +0530 |
---|---|---|
committer | Ben Skeggs <bskeggs@redhat.com> | 2015-10-22 09:11:27 +1000 |
commit | 1f7b091e9766c44ef0b421aee32891b10f2322d8 (patch) | |
tree | b724954c7df4b3d7a8bb650d513cb679c08bec96 | |
parent | 92ac5157be53ea1d1272ecbdbb37751ef7905cc0 (diff) | |
download | nouveau-1f7b091e9766c44ef0b421aee32891b10f2322d8.tar.gz |
drm: fix memory leak
If pm_runtime_get_sync() we were going to "out" but we missed freeing
vma.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
-rw-r--r-- | drm/nouveau/nouveau_gem.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drm/nouveau/nouveau_gem.c b/drm/nouveau/nouveau_gem.c index 6bf1a7895..a108cc375 100644 --- a/drm/nouveau/nouveau_gem.c +++ b/drm/nouveau/nouveau_gem.c @@ -84,8 +84,10 @@ nouveau_gem_object_open(struct drm_gem_object *gem, struct drm_file *file_priv) } ret = pm_runtime_get_sync(dev); - if (ret < 0 && ret != -EACCES) + if (ret < 0 && ret != -EACCES) { + kfree(vma); goto out; + } ret = nouveau_bo_vma_add(nvbo, cli->vm, vma); if (ret) |