mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-02-18 20:25:25 +00:00
Code Issues 1 Projects Releases Wiki Activity

mfvideosrc: Use only the first video stream per device

Browse source 
Non-first video stream might not be working with current
implementation. It could be non-video (e.g., photo source) and then
ReadSample() might be blocked forever.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1661>
This commit is contained in:
Seungha Yang 2020-10-07 05:05:25 +09:00 committed by GStreamer Merge Bot
parent 9279326d8a
commit 20d9283e3d
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
sys/mediafoundation/gstmfsourcereader.cpp
View file

@ -180,7 +180,14 @@ gst_mf_enum_media_type_from_source_reader (IMFSourceReader * source_reader,
g_return_val_if_fail (source_reader != NULL, FALSE); g_return_val_if_fail (source_reader != NULL, FALSE);
g_return_val_if_fail (media_types != NULL, FALSE); g_return_val_if_fail (media_types != NULL, FALSE);
for (i = 0;; i++) { {
/* Retrive only the first video stream. non-first video stream might be
* photo stream which doesn't seem to be working propertly in this implementation.
*
* Note: Chromium seems to be using the first video stream
* https://github.com/chromium/chromium/blob/ccd149af47315e4c6f2fc45d55be1b271f39062c/media/capture/video/win/video_capture_device_factory_win.cc
*/
i = MF_SOURCE_READER_FIRST_VIDEO_STREAM;
for (j = 0;; j++) { for (j = 0;; j++) {
ComPtr<IMFMediaType> media_type; ComPtr<IMFMediaType> media_type;