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 423283 - [qtdemux] timestamps of outgoing h264 packets are wrong
[qtdemux] timestamps of outgoing h264 packets are wrong
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.5
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-27 09:16 UTC by Edward Hervey
Modified: 2007-03-28 15:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Process ctts atom and apply pts on outgoing buffers' timestamp (5.97 KB, patch)
2007-03-27 09:16 UTC, Edward Hervey
committed Details | Review

Description Edward Hervey 2007-03-27 09:16:00 UTC
Currently, qtdemux doesn't process/handle the 'ctts' atom present in h264 mov/iso files. This means that the outgoing h264 packets are timestamped with dts timestamps instead of pts timestamps.

The outgoing buffers of demuxers should have:
* TIMESTAMP == pts
and if needed:
* OFFSET == dts

Patch follows to fix that behaviour
Comment 1 Edward Hervey 2007-03-27 09:16:40 UTC
Created attachment 85374 [details] [review]
Process ctts atom and apply pts on outgoing buffers' timestamp
Comment 2 Edward Hervey 2007-03-28 15:19:11 UTC
2007-03-28  Edward Hervey  <edward@fluendo.com>

	* gst/qtdemux/qtdemux.c: (gst_qtdemux_prepare_current_sample),
	(gst_qtdemux_chain), (qtdemux_parse_samples):
	* gst/qtdemux/qtdemux_dump.c: (qtdemux_dump_ctts):
	* gst/qtdemux/qtdemux_dump.h:
	* gst/qtdemux/qtdemux_fourcc.h:
	* gst/qtdemux/qtdemux_types.c:
	Process 'ctts' atoms, which are present in AVC ISO files (.mov files
	with h264 video).
	Use the offset present in 'ctts' to calculate the PTS for each packet
	and set the PTS on outgoing buffers.
	Fixes #423283