diff options
| author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-11 15:00:56 +0530 |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2015-10-12 13:08:20 +1000 |
| commit | fdc5d93347be4b2dd4c9b62668cea4cfa7aa72ef (patch) | |
| tree | 7d568ec3a007199cf64b989d5682f2d0d28639a2 | |
| parent | c60cd4ec9e07e5da5e914a465b6bc3d660beb614 (diff) | |
| download | nouveau-fdc5d93347be4b2dd4c9b62668cea4cfa7aa72ef.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 1497a1462..ce74ab12e 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) |