diff options
author | Maarten Lankhorst <maarten.lankhorst@canonical.com> | 2014-09-01 14:47:35 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-08 13:42:00 -0700 |
commit | 3ea411c56ef58a1827cf06fdcdeb14fa9265be09 (patch) | |
tree | bfcb718d9cc134bcd780d93ed4c90c361c5b5979 /drivers/staging/android | |
parent | d9fdb9fba7ec08769594abede8f78523ed3f025a (diff) | |
download | linux-3ea411c56ef58a1827cf06fdcdeb14fa9265be09.tar.bz2 |
android: fix reference leak in sync_fence_create
According to the documentation sync_fence_create takes ownership of the point,
not a reference on the point.
This fixes a memory leak introduced in 3.17's android fence rework.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Cc: Colin Cross <ccross@google.com>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/android')
-rw-r--r-- | drivers/staging/android/sync.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e0234196..69139ce7420d 100644 --- a/drivers/staging/android/sync.c +++ b/drivers/staging/android/sync.c @@ -199,7 +199,6 @@ struct sync_fence *sync_fence_create(const char *name, struct sync_pt *pt) fence->num_fences = 1; atomic_set(&fence->status, 1); - fence_get(&pt->base); fence->cbs[0].sync_pt = &pt->base; fence->cbs[0].fence = fence; if (fence_add_callback(&pt->base, &fence->cbs[0].cb, |