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 667559 - h264parse: wrong durationg calculation of outgoing buffers
h264parse: wrong durationg calculation of outgoing buffers
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 659489
Blocks:
 
 
Reported: 2012-01-09 14:30 UTC by Sebastian Dröge (slomo)
Modified: 2018-11-03 13:10 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Sebastian Dröge (slomo) 2012-01-09 14:30:50 UTC
I only get a single, weird frame with this clip:
http://samples.mplayerhq.hu/V-codecs/h264/last_samurai.ts

Using a videosink with sync=false gives the correct output
Comment 1 Edward Hervey 2012-05-24 08:48:05 UTC
Hmm.... the file doesn't contain PTS/DTS in all PES headers, but only once every few frames.

As a result it sets the previously known valid pts on the outgoing frame.

This should be handled more gracefully. Ideally by not setting a buffer timestamp when we don't have a pts
Comment 2 Edward Hervey 2012-07-24 14:27:27 UTC
Even if we only set PTS/DTS on tsdemux output buffers when it's valid, it will still fail downstream.

Either h264parse, or GstVideoDecoder, needs to fill in missing PTS/DTS at some point
Comment 3 Edward Hervey 2013-04-26 14:34:19 UTC
the file plays back with current master. Only remaining issue is some stuttering at playback. This is related to h264parse setting "wrong" durations on the outgoing parsed buffers.

Changing bug title accordingly
Comment 4 Edward Hervey 2013-07-03 17:18:42 UTC
Actually ... there's another way to fix this file issue.

The mpeg-ts file contains these two descriptors for the video stream:
          [descriptor 0x02 (video-stream)]
          [descriptor 0x07 (target-background-grid)]

The first one gives information about framerate and so forth, and ... surprise surprise ... it states that it's 25/1 fps, meaning we could set that on the demuxer outgoing pad caps and the problem would be solved.

Note that the other descriptor would also solve the cropping issue.
Comment 5 Edward Hervey 2013-07-24 08:17:06 UTC
Making this block against #659489 since this issue should theoretically be fixed once PTS/DTS handling is fixed.
Comment 6 Olivier Crête 2018-05-04 12:28:37 UTC
FYI, this plays equally badly with VLC and ffplay
Comment 7 GStreamer system administrator 2018-11-03 13:10:19 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/57.