diff options
| author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-11 09:30:56 (GMT) |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2015-11-03 05:02:18 (GMT) |
| commit | bad4274a69328256eb3eaad4baaefd1b33491f7e (patch) | |
| tree | 935ccd7cd54b2570688b08d1530c336f3350f761 | |
| parent | 4458c5639ba827d494ab06a81d1bb129ca5aea91 (diff) | |
| download | linux-bad4274a69328256eb3eaad4baaefd1b33491f7e.tar.xz | |
drm/nouveau: 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-- | drivers/gpu/drm/nouveau/nouveau_gem.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c b/drivers/gpu/drm/nouveau/nouveau_gem.c index 6bf1a78..a108cc3 100644 --- a/drivers/gpu/drm/nouveau/nouveau_gem.c +++ b/drivers/gpu/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) |