[Bug 727498] videodecoder: deactivates downstream bufferpool

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=727498
  GStreamer | gst-plugins-base | git

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.3.1

--- Comment #3 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2014-04-04 17:25:20 UTC ---
commit 05e957106f7c23255e6cb908a547e7214a9f8540
Author: Thiago Santos <ts.santos at sisa.samsung.com>
Date:   Wed Apr 2 07:20:43 2014 -0300

    videodecoder: do not deactivate the bufferpool, just unref

    Videodecoder does late renegotiation, it will wait for the next
    buffer before renegotiating its caps and bufferpool. It might happen
    that downstream element switched from passthrough to non-passthrough
    and sent a reconfigure upstream (that caused this renegotiation).
    This downstream element will ask the video sink below for the bufferpool
    with an allocation query and will get the same bufferpool that
    videodecoder is holding, too.

    When renegotiating, if videodecoder deactivates its bufferpool it
    might be deactivating the bufferpool that some element downstream
    is using and cause the pipeline to fail.

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

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.