FaSTTUBe/GST-Tensordecoder-ov_ep
35
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

waylandsink: Do not pre-configure proposed pool

Browse Source 
Upstream must configure the pool before using it. Pre-configuring
the proposed pool could hide bugs in upstream elements.
This commit is contained in:
Nicolas Dufresne 2016-09-22 15:37:41 -04:00
parent ecf88d0d43
commit 0859e201f6
1 changed files with 15 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
ext/wayland/gstwaylandsink.c
View File

@ -436,7 +436,7 @@ gst_wayland_create_pool (GstWaylandSink * sink, GstCaps * caps)
pool = gst_video_buffer_pool_new (); pool = gst_video_buffer_pool_new ();
structure = gst_buffer_pool_get_config (pool); structure = gst_buffer_pool_get_config (pool);
gst_buffer_pool_config_set_params (structure, caps, size, 2, 0); gst_buffer_pool_config_set_params (structure, caps, size, 0, 0);
gst_buffer_pool_config_set_allocator (structure, gst_wl_shm_allocator_get (), gst_buffer_pool_config_set_allocator (structure, gst_wl_shm_allocator_get (),
NULL); NULL);
@ -520,8 +520,6 @@ gst_wayland_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query)
GstWaylandSink *sink = GST_WAYLAND_SINK (bsink); GstWaylandSink *sink = GST_WAYLAND_SINK (bsink);
GstCaps *caps; GstCaps *caps;
GstBufferPool *pool = NULL; GstBufferPool *pool = NULL;
GstStructure *config;
guint size, min_bufs, max_bufs;
gboolean need_pool; gboolean need_pool;
gst_query_parse_allocation (query, &caps, &need_pool); gst_query_parse_allocation (query, &caps, &need_pool);
@ -530,12 +528,7 @@ gst_wayland_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query)
pool = gst_wayland_create_pool (sink, caps); pool = gst_wayland_create_pool (sink, caps);
if (pool) { if (pool) {
config = gst_buffer_pool_get_config (pool); gst_query_add_allocation_pool (query, pool, sink->video_info.size, 2, 0);
gst_buffer_pool_config_get_params (config, NULL, &size, &min_bufs,
&max_bufs);
gst_query_add_allocation_pool (query, pool, size, min_bufs, max_bufs);
gst_structure_free (config);
g_object_unref (pool); g_object_unref (pool);
} }
@ -666,9 +659,19 @@ gst_wayland_sink_show_frame (GstVideoSink * vsink, GstBuffer * buffer)
/* sink->pool always exists (created in set_caps), but it may not /* sink->pool always exists (created in set_caps), but it may not
* be active if upstream is not using it */ * be active if upstream is not using it */
if (!gst_buffer_pool_is_active (sink->pool) && if (!gst_buffer_pool_is_active (sink->pool)) {
!gst_buffer_pool_set_active (sink->pool, TRUE)) GstStructure *config;
goto activate_failed; GstCaps *caps;
guint size = sink->video_info.size;
config = gst_buffer_pool_get_config (sink->pool);
gst_buffer_pool_config_get_params (config, &caps, &size, NULL, NULL);
gst_buffer_pool_config_set_params (config, caps, size, 0, 0);
/* This is a video pool, it should not fail with basic setings */
if (!gst_buffer_pool_set_active (sink->pool, TRUE))
goto activate_failed;
}
ret = gst_buffer_pool_acquire_buffer (sink->pool, &to_render, NULL); ret = gst_buffer_pool_acquire_buffer (sink->pool, &to_render, NULL);
if (ret != GST_FLOW_OK) if (ret != GST_FLOW_OK)