[Ffmpeg-devel] [PATCH] allow ffmpeg to read mp3s beginning with partial frames

Andreas Öman andreas
Sat Sep 9 07:51:04 CEST 2006

Michael Niedermayer wrote:

>>> yes thats why i am so much against doing it in lavf/mp3.c, it belongs
>>> to the mp3 parser (libavcodec/parser.c)
>> Turns out im wrong it does. libavcodec/parser.c handles initial partial
>> frame and files work fine with just the change to mp3_read_probe as
>> supplied. 
> patch looks ok

I have to disagree. With this new patch ffmpeg still thinks some of my 
files are mp3-files (especially .ts-files)

However, having bumped into the same problem I've had a patch that fixes 
the problem (without generating any false positives). It searches the 
first 4k of the file for two consecutive mp3-frames.

Due to some recent mail-malfunctioning I could not respond to the thread 
earlier. Sorry 'bout that. And I haven't gotten around to send the patch 
either, shame on me...

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mp3.diff
Type: text/x-patch
Size: 1633 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060909/2f056d6e/attachment.bin>

More information about the ffmpeg-devel mailing list