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 449420 - [fixed upstream] [FLV] Incorrect FLV frame sizes
[fixed upstream] [FLV] Incorrect FLV frame sizes
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other All
: Normal normal
: 0.10.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-06-20 10:02 UTC by Tim Angus
Modified: 2007-12-17 12:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
FLV exhibiting problem (577.36 KB, application/x-flash-video)
2007-06-20 10:05 UTC, Tim Angus
  Details
Patch from upstream FFMPEG SVN which fixes the problem (4.93 KB, patch)
2007-06-20 10:09 UTC, Tim Angus
none Details | Review

Description Tim Angus 2007-06-20 10:02:58 UTC
Please describe the problem:
The version of FFMPEG last mirrored (Sept 06) has a bug whereby some FLV files' frame dimensions are incorrectly read. FFMPEG pads the difference using the pixels on the bottom and right edges of the frame, resulting in a "smearing" effect.

Steps to reproduce:
1. Play an affected FLV file.



Actual results:
Padding on bottom and right hand side of output

Expected results:
No padding and thus no smearing

Does this happen every time?
Yes

Other information:
Comment 1 Tim Angus 2007-06-20 10:05:16 UTC
Created attachment 90327 [details]
FLV exhibiting problem
Comment 2 Tim Angus 2007-06-20 10:09:29 UTC
Created attachment 90328 [details] [review]
Patch from upstream FFMPEG SVN which fixes the problem

This is generated via "svn diff -r6769:6773" on FFMPEG SVN. Some minor casting is added to correct typing.
Comment 3 Tim Angus 2007-06-20 10:15:34 UTC
(Obviously this bug would be fixed by a new FFMPEG merge and the patch here obviated; I'm just adding this in case anybody else runs into it in the meantime).
Comment 4 Edward Hervey 2007-08-14 14:53:43 UTC
changing title so we can close it once we update mirror.
Comment 5 Edward Hervey 2007-12-17 12:47:26 UTC
fixed thanks to new build system.