Opened 13 months ago

Closed 3 months ago

Last modified 3 months ago

#15220 closed bug (fixed)

MediaPlayer crashes at opening mp3 with cover art

Reported by: miqlas Owned by: leavengood
Priority: normal Milestone: R1/beta2
Component: Audio & Video/Codecs Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

No other ffmpeg based media player crashes with the same file, not even ffplay, so this is a bug in MediaPlayer.

Probably happens since the switch from ffmpeg 0.1x. Unlike ffmpeg 0.1, the current ffmpeg 4.x provides the cover art as video stream, but MediaPlayer can't handle it well, and mostly just crashes. After stripping the cover art like ffmpeg -i tagged.mp3 -vn -codec:a copy -map_metadata -1 out.mp3 no crash happens, so it is definetely the cover art what triggers the bug.

I will attach a debug report about a crash happens reproduceably, 100%. If needed i can provide the file too.

ffmpeg devs said the bug could be somewhere in _HandleNewVideoFrameAndUpdateSystemState

Attachments (1)

MediaPlayer-10489-debug-01-08-2019-20-24-22.txt (32.0 KB ) - added by miqlas 13 months ago.
Crashlog

Download all attachments as: .zip

Change History (8)

comment:1 by AlienSoldier, 13 months ago

I never experienced a crash but i experienced very high cpu load (leading to some sound skiping).

comment:2 by leavengood, 12 months ago

Owner: changed from stippi to leavengood
Status: newassigned

comment:3 by miqlas, 10 months ago

I don't experience crashes anymore.

Version 0, edited 10 months ago by miqlas (next)

comment:4 by pulkomandy, 3 months ago

Component: Applications/MediaPlayerAudio & Video/Codecs
Milestone: UnscheduledR1/beta2

comment:5 by waddlesplash, 3 months ago

Resolution: fixed
Status: assignedclosed

Fix merged in hrev54121.

comment:6 by AlienSoldier, 3 months ago

Taking this file as an exemple:

https://remix.kwed.org/download.php/6124/Ctrl-Z%20-%20Spy%20vs%20Spy.mp3

There is still problem in hrev54121 for mp3 with cover picture. The time counter does not change and when the end of the file is reached, the tune start again.

That prevent those file to be in playlist as they never proceed to the next track as they just loop even if no loop option is enabled.

comment:7 by AlienSoldier, 3 months ago

Skipping and high cpu load don't appear anymore.

Note: See TracTickets for help on using tickets.