Opened 16 years ago

Closed 13 years ago

Last modified 13 years ago

#1056 closed defect (fixed)

ogg with large, multi-packet header does not work

Reported by: hendrik@… Owned by: reimar
Priority: normal Component: demuxer
Version: 1.0rc2 Severity: normal
Keywords: Cc: jason52lh@…, lucidlytwisted@…
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

When playing an ogg file with big embedded cover art (in the COVERART tag) it crashes.

I tried a few coversizes with easytag: 30kB still worked, but 70kB didn't.

I didn't know the component, so I guessed one.

Change History (10)

comment:1 by compn, 16 years ago

Component: addemuxer

if you could provide a sample or way of creating such file
so we can reproduce the problem...

comment:2 by hendrik@…, 16 years ago

An example file can be downloaded from:
http://wdz5.xs4all.nl/~hendrik/mmw-deadzy.ogg

comment:3 by reimar, 16 years ago

Summary: Crash when playing Oggfile with big embedded coverogg with large, multi-packet header does not work

The crash is analyzed and (I expect) will be fixed soon, but that is not enough to make the file actually play.
I adjusted the summary a bit.

comment:4 by jason52lh@…, 13 years ago

Cc: jason52lh@… added

Seems this related to METADATA_BLOCK_PICTURE filed in ogg tags.

See more: http://wiki.xiph.org/VorbisComment#METADATA_BLOCK_PICTURE

VLC has the same problem: http://trac.videolan.org/vlc/ticket/1616

"VLC and mplayer are the only programs/libraries I've found that won't play these files. Xine, gstreamer, ffmpeg, ogg123 and the Xiph DirectShow filters will all play them without a problem."

i also found if ogg embedded cover art larger than 40kb, mplayer won't play, meanwhile other player works well.

i tired mplayer svn 32492 and git version, all have the same problem.

comment:5 by compn, 13 years ago

Owner: changed from r_togni@… to reimar

comment:6 by reimar, 13 years ago

Partially an FFmpeg issue: it both extracts the cover art as metadata and exports it in extradata, overflowing the 64 kB limit of extradata in wav/AVI/..., which unfortunately MPlayer currently shares.
Possible fix on ffmpeg-devel list.

comment:7 by lucidlytwisted@…, 13 years ago

Cc: lucidlytwisted@… added

Is there any movement on this bug? I note that it's still an issue with MPlayer 1.0rc4-4.4.5.

comment:8 by reimar, 13 years ago

Resolution: fixed
Status: newclosed

The fix has been applied to FFmpeg on the 31st of January and it works just fine with latest SVN MPlayer + FFmpeg.
rc4 is much, much older than the fix.

comment:9 by lucidlytwisted@…, 13 years ago

Thanks - guess the fix hasn't landed in my distro yet (Ubuntu). I'll see what I can do about updating my system.

comment:10 by lucidlytwisted@…, 13 years ago

I installed the MOTU Daily PPA and that fixed the issue.
Thanks very much!

Note: See TracTickets for help on using tickets.