[FFmpeg-devel] [PATCH] Fix timestamp calculation when demuxing MP3 frames in a .avi container.

Mike Scheutzow mike.scheutzow at alcatel-lucent.com
Thu Jun 28 18:08:34 CEST 2012


Michael Niedermayer wrote:
> On Wed, Jun 27, 2012 at 11:51:10AM -0400, Mike Scheutzow wrote:
>> The attached patch fixes trac 1432.
>>
>> The bad code was using the frame byte length to calculate a timestamp,
>> which is completely broken. Also set AVPacket.duration, since demux is
>> already calculating it. The existing avi muxer already properly handles
>> the muxing and index creation for MP3 audio frames.  Fixes the sample
>> file in trac 1432.
> 
> code breaks make fate
> also please test your patch against cbr & vbr style muxed avi


I see the problem, and will investigate.

Thanks for looking at the patch,

Mike Scheutzow


More information about the ffmpeg-devel mailing list