After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 725948 - videomixer2: crash after renegotiating with different resolution
videomixer2: crash after renegotiating with different resolution
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.2.3
Other Linux
: Normal normal
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-03-08 18:01 UTC by Snark
Modified: 2014-03-11 16:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Small test triggering the crash (2.36 KB, text/x-c++src)
2014-03-08 18:01 UTC, Snark
  Details
videomixer2: store video info with buffers to keep it in sync (3.05 KB, patch)
2014-03-11 03:53 UTC, Thiago Sousa Santos
committed Details | Review

Description Snark 2014-03-08 18:01:42 UTC
Created attachment 271327 [details]
Small test triggering the crash

The attached small test program crashes quite fast as-is, but works when the line using the ximagesrc element is commented out.
Comment 1 Thiago Sousa Santos 2014-03-10 04:55:03 UTC
It seems that an internally queued buffer from an old caps is being used as if it had the new caps from ximagesrc that is usually different from the others.
Comment 2 Thiago Sousa Santos 2014-03-11 03:53:41 UTC
Created attachment 271502 [details] [review]
videomixer2: store video info with buffers to keep it in sync

Instead the queued buffer might have an old caps while the pad
is already storing the information for a new caps. Mixing those
while handling buffers will often lead to issues
Comment 3 Thiago Sousa Santos 2014-03-11 16:02:49 UTC
commit 373eceef7c6f5e2e113ef8e2703535c2a2280dce
Author: Thiago Santos <ts.santos@sisa.samsung.com>
Date:   Tue Mar 11 00:46:06 2014 -0300

    videomixer2: store video info with buffers to keep it in sync
    
    Instead the queued buffer might have an old caps while the pad
    is already storing the information for a new caps. Mixing those
    while handling buffers will often lead to issues
    
    https://bugzilla.gnome.org/show_bug.cgi?id=725948