[gstreamer-bugs] [Bug 616128] New: oggdemux: bad calculated granulepos

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=616128
  GStreamer | gst-plugins-base | git

           Summary: oggdemux: bad calculated granulepos
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: ds at schleef.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=159033)
 --> (https://bugzilla.gnome.org/attachment.cgi?id=159033)
patch

oggdemux calculates the granulepos values incorrectly for this stream:

http://ia331209.us.archive.org/0/items/Patent_Absurdity/Patent_Absurdity_HQ_768kbit.ogv

Instead of the expected values: 64, 65, 66, 67, 68, 69, 70, 71, the current
oggdemux outputs 1, 2, 3, 4, 5, 6, 7, 71.  These coincidentally translate to
the same timestamps, so playback is correct, however, theoraparse and oggmux
get confused if you attempt to remux the stream.  If the stream didn't start at
time=0, the first few frames would have the wrong timestamp, and likely be
dropped since the incorrect timestamps would be outside the segment.  So I
don't think anything *bad* happens during playback.

Patch attached.  It's a bit invasive, so I question whether it's worth breaking
a freeze for it.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.