[gstreamer-bugs] [Bug 335231] New: Incorrect way to calculate seek position in wavparse

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=335231
 GStreamer | gst-plugins-good | Ver: 0.10.2

           Summary: Incorrect way to calculate seek position in wavparse
           Product: GStreamer
           Version: 0.10.2
          Platform: Other
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: ext-tommi.myohanen at nokia.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Please describe the problem:
This bug is related to bug #326864. For audio formats that use block aligning, 
seek position might be calculated incorrectly in wavparse element. The "wav->
offset" is added by "wav->datastart" value, and the block alignment modulo 
calculation is performed after this. in certain conditions this causes the 
offset to become zero.

I attached a minimalistic patch that fixes this by changing the order of these 
operations :)


Steps to reproduce:



Actual results:



Expected results:



Does this happen every time?



Other information:


-- 
Configure bugmail: http://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.