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 741894 - video-frame: might need to revise its assertions
video-frame: might need to revise its assertions
Status: RESOLVED DUPLICATE of bug 741030
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: High blocker
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-12-23 05:45 UTC by Mathieu Duponchelle
Modified: 2014-12-23 05:58 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
video-frame: take crop meta into account. (2.01 KB, patch)
2014-12-23 05:47 UTC, Mathieu Duponchelle 		none Details | Review

Description Mathieu Duponchelle 2014-12-23 05:45:29 UTC 
wget https://people.internal.collabora.co.uk/~meh/Videos/basic.ogg

gst-launch-1.0 playbin uri=file://$PWD/basic.ogg

What should happen:

The file plays back without problems.

What happens :

Critical error gst_video_frame_map_id: assertion 'info->height == meta->height' failed

afaiu, this happens because the buffer also has a crop video meta, which should be the one whose dimensions are checked, tentative patch is attached next.
Comment 1 Mathieu Duponchelle 2014-12-23 05:47:06 UTC 
Created attachment 293222 [details] [review]
video-frame: take crop meta into account.

+ Prevents furious asserts.
Comment 2 Nicolas Dufresne (ndufresne) 2014-12-23 05:58:29 UTC 
I already filed this bug,

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

And am waiting on feedback on what should be done. It's not just a matter of checks imho.

*** This bug has been marked as a duplicate of bug 741030 ***