mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-26 03:31:05 +00:00
Code Issues 1 Projects Releases Wiki Activity

allocators: drm-dumb: Fix non-thread safe dmabuf allocator creation

Browse source 
If the allocation function get called from multiple threads at the same time,
multiple allocators may get created but only one get saved. Leading to other
allocators to be leaked. Simply create it once in the instance initialization.

Fixes: #2456
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6052>
This commit is contained in:
Nicolas Dufresne 2024-02-04 10:12:22 +01:00 committed by GStreamer Marge Bot
parent 39f2afbd45
commit 0fd76c2848
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
subprojects/gst-plugins-base/gst-libs/gst/allocators/gstdrmdumb.c
View file

@ -156,9 +156,6 @@ gst_drm_dumb_memory_export_dmabuf (GstMemory * mem)
if (ret)
goto export_fd_failed;
if (G_UNLIKELY (alloc->dmabuf_alloc == NULL))
alloc->dmabuf_alloc = gst_dmabuf_allocator_new ();
dmamem = gst_dmabuf_allocator_alloc (alloc->dmabuf_alloc, prime_fd,
gst_memory_get_sizes (mem, NULL, NULL));
@ -452,8 +449,7 @@ gst_drm_dumb_allocator_finalize (GObject * obj)
{
GstDRMDumbAllocator *alloc = GST_DRM_DUMB_ALLOCATOR (obj);
if (alloc->dmabuf_alloc)
gst_object_unref (alloc->dmabuf_alloc);
gst_object_unref (alloc->dmabuf_alloc);
g_free (alloc->drm_device_path);
alloc->drm_device_path = NULL;
@ -581,6 +577,7 @@ gst_drm_dumb_allocator_init (GstDRMDumbAllocator * alloc)
alloc->drm_fd = -1;
alloc->drm_device_path = NULL;
alloc->dmabuf_alloc = gst_dmabuf_allocator_new ();
base_alloc->mem_type = GST_DRM_DUMB_MEMORY_TYPE;
base_alloc->mem_map = gst_drm_dumb_memory_map;