[Bug 758344] avviddec: May set padded width/height in pool caps

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=758344

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #316545|0                           |1
        is obsolete|                            |

--- Comment #24 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Created attachment 316546
  --> https://bugzilla.gnome.org/attachment.cgi?id=316546&action=edit
avviddec: Make sure to use a buffer pool with the correct width/height
configured on it for pushing buffers downstream

If downstream does not provide a (usable) pool, we would use our internal
pool. But the internal pool might be configured with a different width/height
because of padding, which then will cause problems if we push buffers from it
directly downstream.
Instead create a new pool if the width/height is different.

This prevents crashes with vaapisink and d3dvideosink for example.

Based on the debugging results and discussions with
Nicolas Dufresne <nicolas.dufresne at collabora.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.