[Bug 715137] New: avimux: padding granularity field not filled in correctly

[GStreamer (bugzilla.gnome.org)]

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

           Summary: avimux: padding granularity field not filled in
                    correctly
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: vhdirk at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Avimux never actually fills the padding granularity field of the AVI header.
This is important when working with non-default image sizes, since GStreamer
requires each row to be padded to the next 4-byte boundary.
Leaving this field zero results in ffmpeg and vlc interpreting the padding as
actual image data, wrapping it onto the next row. This introduces an
incremental spatial shift with each new row.

The variable avimux->avi_hdr.pad_gran is there, yet it is never assigned a
value.
This could default to 4. I assume there may be some corner cases where 4 is
incorrect, but non that I can think of right now.

-- 
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.