Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 209521 - media-video/smplayer crashes mplayer when mplayer has USE="-png"
Summary: media-video/smplayer crashes mplayer when mplayer has USE="-png"
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-10 13:35 UTC by Bernd Steinhauser
Modified: 2008-02-19 00:25 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bernd Steinhauser 2008-02-10 13:35:25 UTC
If one compiles mplayer with USE="-png" and runs it via
"mplayer -vf-add screenshot"
as smplayer does in _default_ configuration, then mplayer will crash.
There should be either a built check

if ! built_with_use media-video/mplayer png; then
 die "Smplayer needs mplayer to be built with png support, to prevent crashes."
fi

Maybe ewarn instead of die is sufficient. If ewarn is used, there should be a comment, that one can clear the "Folder for storing screenshots" field to prevent smplayer from adding -vf-add screenshot to the command line it passes to mplayer.

See also:
http://smplayer.sourceforge.net/forums/viewtopic.php?id=234

Reproducible: Always

Steps to Reproduce:
1. USE="-png" emerge -v1 mplayer smplayer
2. mv ~/.smplayer ~/.smplayer.old
3. smplayer some_video_file

Actual Results:  
Debug: MplayerProcess::parseLine: 'Opening video filter: [screenshot]'
Debug: MplayerProcess::parseLine: ''
Debug: MplayerProcess::parseLine: ''
Debug: MplayerProcess::parseLine: 'MPlayer interrupted by signal 11 in module: init_video_filters'
Debug: MplayerProcess::parseLine: 'ID_SIGNAL=11'
Debug: MplayerProcess::parseLine: '- MPlayer crashed by bad usage of CPU/FPU/RAM.'
Comment 1 Bernd Steinhauser 2008-02-10 14:17:40 UTC
Forgot to add, this is of course the case for both versions in portage.
Comment 2 Ben de Groot (RETIRED) gentoo-dev 2008-02-19 00:25:49 UTC
Applied in cvs.