[FFmpeg-devel] [PATCH] asfenc: too many payloads in a packet

Michael Niedermayer michaelni at gmx.at
Tue May 13 01:05:54 CEST 2014


On Mon, May 12, 2014 at 03:26:11PM -0700, Aaron Graham wrote:
> In the ASF specification:
> http://www.microsoft.com/en-us/download/details.aspx?id=14995
> 
> the section on Multiple payloads (5.2.3.3) describes the bit fields used to
> specify payload length type and number of payloads. The number of payloads
> field is a 6-bit field, meaning a packet can hold a maximum of 63 payloads.
> 
> asfenc.c was not honoring this, and if more than 63 payloads would fit in a
> packet, the "number of payloads" field would overflow into the "payload
> length type" field, causing parsing errors in ffplay.
> 
> You can reproduce this with any AAC file that has several seconds of
> silence in the beginning (lots of very small AAC frames). Use this ffmpeg
> command:
> 
> $ ffmpeg -i silence.m4a -acodec copy silence.asf
> 
> Note that ffplay will get lots of errors.
> 
> Anyway, here's a patch to fix the problem:

patch applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140513/30b9424c/attachment.asc>


More information about the ffmpeg-devel mailing list