[FFmpeg-devel] MP3 file that crashes ffmpeg

[Michael Niedermayer]

Hi

On Mon, May 07, 2007 at 01:35:26PM +0200, Michael Niedermayer wrote:
> Hi
> 
> On Mon, May 07, 2007 at 09:54:28AM +0200, Reimar Doeffinger wrote:
> > Hello,
> > On Mon, May 07, 2007 at 02:17:50AM +0200, Diego Biurrun wrote:
> > > On Sun, May 06, 2007 at 06:28:36PM +0200, Michael Niedermayer wrote:
> > > > On Sun, May 06, 2007 at 04:20:22PM +0200, Reimar D?ffinger wrote:
> > > > > On Sun, May 06, 2007 at 04:13:41PM +0200, Reimar D?ffinger wrote:
> > > > > [...]
> > > > > > Attached (obviously wrong) patch fixes the crash for me.
> > > > > 
> > > > > And this patch will at least not break anything that was working before,
> > > > > but the code is probably wrong anyway.
> > > > 
> > > > fixed differently (num_sprite_wraping points isnt allowed to be >3 in mpeg4
> > > > with gmc)
> > > 
> > > Note that the file still does not play even though the crash is gone ..
> > 
> > Yes, MP3 files that do not start with MP3-frames only get a quarter of
> > the max. autodetection score, whereas anything that looks even only
> > remotely like raw MPEG4 gets one half, thus ffmpeg it as MPEG4 -
> > for some reason it does not give a bonus to mp3 for the extension (at
> > least that's where I think the difference to mplayer -demuxer lavf comes
> > from)??
> 
> yes there are several bugs related to this report
> 1. the file extension if its recognized should always give a minimum score
> 2. the mpeg4 probe should be improved
> 
> ill look at 2. and leave 1. for a volunteer ...

done, its no longer detected as mpeg4, but as mpeg-ps now :)
mans?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070507/97732c84/attachment.pgp>