dma-buf/sync_file: hold reference to fence when creating sync_file
authorGustavo Padovan <gustavo.padovan@collabora.co.uk>
Wed, 19 Oct 2016 17:48:32 +0000 (15:48 -0200)
committerSean Paul <seanpaul@chromium.org>
Fri, 21 Oct 2016 14:49:06 +0000 (10:49 -0400)
fence referencing was out of balance. It was not taking any ref to the
fence at creating time, but it was putting a reference when freeing the
sync file.

This patch fixes the balancing issue by getting a reference for the fence
when creating the sync_file.

Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: http://patchwork.freedesktop.org/patch/msgid/1476899313-22241-1-git-send-email-gustavo@padovan.org
drivers/dma-buf/sync_file.c

index b29a9e817320a844ce0a0f8a89711266a17211a0..235f8ac113cce7617f6e78d2c9a2c8de75637665 100644 (file)
@@ -79,7 +79,7 @@ struct sync_file *sync_file_create(struct fence *fence)
        if (!sync_file)
                return NULL;
 
-       sync_file->fence = fence;
+       sync_file->fence = fence_get(fence);
 
        snprintf(sync_file->name, sizeof(sync_file->name), "%s-%s%llu-%d",
                 fence->ops->get_driver_name(fence),