system/pipewire
system/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire synced 2024-07-03 07:58:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

gst: Check for video/ caps before parsing for info

Browse Source 
The gst_video_info_from_caps() API isn't really intended to be used as a
check-for-videoness function (it generates an error-level GStreamer
debug message when used this way).

We check the caps for a video/ name for this reason, which is
functionally equivalent.
This commit is contained in:
Arun Raghavan 2024-05-27 15:59:25 +03:00
parent 7eeb657d37
commit f400ff2050
2 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/gst/gstpipewirepool.c
View File

@ -224,7 +224,14 @@ set_config (GstBufferPool * pool, GstStructure * config)
return FALSE;
}
has_video = gst_video_info_from_caps (&p->video_info, caps);
if (g_str_has_prefix (gst_structure_get_name (
gst_caps_get_structure (caps, 0)),
"video/")) {
has_video = TRUE;
gst_video_info_from_caps (&p->video_info, caps);
} else {
has_video = FALSE;
}
p->add_metavideo = has_video && gst_buffer_pool_config_has_option (config,
GST_BUFFER_POOL_OPTION_VIDEO_META);

12
src/gst/gstpipewiresrc.c
View File

@ -1028,6 +1028,11 @@ handle_format_change (GstPipeWireSrc *pwsrc,
if (pwsrc->caps && gst_caps_is_fixed (pwsrc->caps)) {
pwsrc->negotiated = TRUE;
if (g_str_has_prefix (gst_structure_get_name (
gst_caps_get_structure (pwsrc->caps, 0)),
"video/")) {
pwsrc->is_video = TRUE;
#ifdef HAVE_GSTREAMER_DMA_DRM
if (gst_video_is_dma_drm_caps (pwsrc->caps)) {
if (!gst_video_info_dma_drm_from_caps (&pwsrc->drm_info, pwsrc->caps)) {
@ -1042,13 +1047,12 @@ handle_format_change (GstPipeWireSrc *pwsrc,
pw_stream_set_error (pwsrc->stream, -EINVAL, "internal error");
return;
}
pwsrc->is_video = TRUE;
} else {
gst_video_info_dma_drm_init (&pwsrc->drm_info);
#endif
pwsrc->is_video = gst_video_info_from_caps (&pwsrc->video_info,
pwsrc->caps);
gst_video_info_from_caps (&pwsrc->video_info,
pwsrc->caps);
}
#ifdef HAVE_GSTREAMER_DMA_DRM
}
#endif