[v3 PATCH 2/2] mfreadwrite/reader: Unset sample allocator callbacks while destroying the source reader.
Nikolay Sivov
nsivov at codeweavers.com
Sat Apr 9 01:16:18 CDT 2022
From: Giovanni Mascellani <gmascellani at codeweavers.com>
The sample allocator callbacks own a reference to the source reader,
which creates a reference loop and prevents the source reader from
being properly destroyed.
Signed-off-by: Giovanni Mascellani <gmascellani at codeweavers.com>
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
dlls/mfreadwrite/reader.c | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/dlls/mfreadwrite/reader.c b/dlls/mfreadwrite/reader.c
index 2d3da19db10..e7b7b555e68 100644
--- a/dlls/mfreadwrite/reader.c
+++ b/dlls/mfreadwrite/reader.c
@@ -1412,6 +1412,7 @@ static ULONG WINAPI src_reader_Release(IMFSourceReader *iface)
{
struct source_reader *reader = impl_from_IMFSourceReader(iface);
ULONG refcount = InterlockedDecrement(&reader->public_refcount);
+ unsigned int i;
TRACE("%p, refcount %lu.\n", iface, refcount);
@@ -1430,6 +1431,23 @@ static ULONG WINAPI src_reader_Release(IMFSourceReader *iface)
LeaveCriticalSection(&reader->cs);
}
+
+ for (i = 0; i < reader->stream_count; ++i)
+ {
+ struct media_stream *stream = &reader->streams[i];
+ IMFVideoSampleAllocatorCallback *callback;
+
+ if (!stream->allocator)
+ continue;
+
+ if (SUCCEEDED(IMFVideoSampleAllocatorEx_QueryInterface(stream->allocator, &IID_IMFVideoSampleAllocatorCallback,
+ (void **)&callback)))
+ {
+ IMFVideoSampleAllocatorCallback_SetCallback(callback, NULL);
+ IMFVideoSampleAllocatorCallback_Release(callback);
+ }
+ }
+
source_reader_release(reader);
}
--
2.35.1
More information about the wine-devel
mailing list