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 651625 - [ffdec] Check for AV_NOPTS_VALUE in PTS
[ffdec] Check for AV_NOPTS_VALUE in PTS
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other All
: Normal normal
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-06-01 10:14 UTC by Raimo Järvi
Modified: 2011-06-06 13:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Check for AV_NOPTS_VALUE in PTS value (934 bytes, patch)
2011-06-01 10:16 UTC, Raimo Järvi
committed Details | Review

Description Raimo Järvi 2011-06-01 10:14:38 UTC
I've tried decoding some AAC LATM streams with ffdec_aac_latm and the result is poor. Timestamps in the decoded data are incorrect and after some investigation it seems that gst-ffmpeg decoder doesn't check for AV_NOPTS_VALUE in PTS. Using AV_NOPTS_VALUE like a valid PTS causes the incorrect output timestamps, the attached patch should fix this.
Comment 1 Raimo Järvi 2011-06-01 10:16:31 UTC
Created attachment 188981 [details] [review]
Check for AV_NOPTS_VALUE in PTS value
Comment 2 Edward Hervey 2011-06-01 14:19:27 UTC
commit 4a66ecdf2577c3f9e7cffb143f3907edea01e00f
Author: Raimo Järvi <raimo.jarvi@gmail.com>
Date:   Tue May 31 13:16:26 2011 +0300

    ffdec: Check for AV_NOPTS_VALUE in PTS value.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=651625
Comment 3 Raimo Järvi 2011-06-06 13:52:24 UTC
Shouldn't target milestone be 0.10.12?
Comment 4 Tim-Philipp Müller 2011-06-06 13:58:43 UTC
> Shouldn't target milestone be 0.10.12?

It should, thanks.