Bug 4923 - mplayer-gui fails to open
Summary: mplayer-gui fails to open
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-13 09:49 CET by Ben Bullard
Modified: 2012-03-26 16:26 CEST (History)
6 users (show)

See Also:
Source RPM: mplayer
CVE:
Status comment:


Attachments
Contents of /usr/share/mplayer/skins/Blue/skin (5.83 KB, text/plain)
2012-03-13 09:51 CET, Ben Bullard
Details
Contents /usr/share/mplayer/skins/default/exit (5.83 KB, text/plain)
2012-03-13 10:03 CET, Ben Bullard
Details

Description Ben Bullard 2012-03-13 09:49:52 CET
Description of problem:clicking on mplayer in application Launcher results in 'Fatal error'


Version-Release number of selected component (if applicable):
mplayer-1.0-1.rc4.0.r34578.3.mga2.tainted
mplayer-gui-1.0-1.rc4.0.r34578.3.mga2.tainted

How reproducible:
Happens 100% of the time.

Steps to Reproduce:
1.Go to >Application Launcher>Applications>Sound and Video>More
2.Select 'Media Player MPlayer'
3.Window opens with "Fatal Error!"
"Error in skin config file on line 11: PNG read error in /usr/share/mplayer/skins/default/main" when I select "OK" another window opens with "Fatal error!" "Config file processing error with skin 'default'"

Commenting out line 11 in '/usr/share/mplayer/skins/default/main' merely changes the error to line 19 and so on.
Comment 1 Ben Bullard 2012-03-13 09:51:16 CET
Created attachment 1746 [details]
Contents of /usr/share/mplayer/skins/Blue/skin
Comment 2 Ben Bullard 2012-03-13 10:03:53 CET
Created attachment 1747 [details]
Contents /usr/share/mplayer/skins/default/exit

This is the actual file quoted in the 'Fatal error!' message.
Comment 3 Manuel Hiebel 2012-03-13 12:01:50 CET
Hi, thanks for reporting this bug.
As there is no maintainer for this package I added the lasts committers in CC.

(Please set the status to 'assigned' if you are working on it)

CC: (none) => anssi.hannula, fundawang, lists.jjorge, luigiwalser
Source RPM: mplayer-gui-1.0-1.rc4.0.r34578.3.mga2.tainted => mplayer

Comment 4 David Walser 2012-03-15 02:06:56 CET
Mandriva noticed this too:
https://qa.mandriva.com/show_bug.cgi?id=65376

They fixed it by just updating the default skin tarball to Blue-1.8.

I committed the same to Cauldron, but it won't build now, even though no changes to the code have been made.  Guess something got broken/changed in our compiler or default compiler options.  It looks like some of the build errors might be fixed by adding -fpermissive, but the other errors don't make sense considering it used to build for us and still does for mdv.
Comment 5 David Walser 2012-03-17 21:54:39 CET
I found out that it's not building now because the "live" package was updated (used for building RTSP streaming support).  I found a patch upstream that should have fixed it, but it still won't build.
Comment 6 David Walser 2012-03-17 22:14:46 CET
Funda, you updated "live", can you help with this?

According to these pages, the patch I added should have worked :o(
http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/2012-March/043743.html
http://lists.live555.com/pipermail/live-devel/2012-February/014700.html
Comment 7 David Walser 2012-03-18 15:50:57 CET
Funda fixed the "live" package and got mplayer to build (thanks Funda).

This should be fixed now.

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 8 Richard Walker 2012-03-19 14:41:32 CET
https://forums.mageia.org/en/viewtopic.php?f=15&t=2109&p=15589#p15589

I am still getting the "Error in skin config file..." error when I start Mplayer from the menu.
It works for me from the command line after a few bitchy complaints about sound and incompatible video_out device (!)
It works from the file manager context menu with custom command "mplayer"
It doesn't work with context menu Mplayer option.

So, mplayer OK for me with no gui but broken because of some default skin error'

Richard

CC: (none) => richard.j.walker

Comment 9 Richard Walker 2012-03-19 14:42:26 CET
I guess that means it ain't fixed yit.

Status: RESOLVED => REOPENED
Resolution: FIXED => (none)

Comment 10 Wolfgang Bornath 2012-03-19 14:54:52 CET
Bug still valid in Beta2 with updates as of 10 minutes ago (last mirror sync
with tier 1 at 13:01 Paris time).

Latest relevant updates (this morning):
mplayer-gui-1.0-1.rc4.0.r34578.4.mga2.tainted
mplayer-1.0-1.rc4.0.r34578.4.mga2.tainted

Copied from the forum entry as of today (only last 3 lines are relevant):
-----------------
$ gmplayer
MPlayer SVN-1.rc4.0.r34578.4.mga2.tainted-4.6.3 (C) 2000-2012 MPlayer Team
mplayer: could not connect to socket
mplayer: No such file or directory
Failed to open LIRC support. You will not be able to use your remote control
Gtk-Message: Failed to load module "canberra-gtk-module"
Error in skin config file on line 11: PNG read error in /usr/share/mplayer/skins/default/main
Config file processing error with skin 'default'
Exiting... (Fatal error)
----------------------------

Running mplayer without gui (from the CLI) works ok.

CC: (none) => molch.b

Comment 11 David Walser 2012-03-19 15:49:18 CET
That's really strange.  I just tested it and it works fine for me.

ls -l /usr/share/mplayer/skins/default/main.png
-rw-r--r-- 1 root root 30945 Mar 19 10:41 main.png

md5sum /usr/share/mplayer/skins/default/main.png
ea762e5eec6a9c64bc19785553d6c163  /usr/share/mplayer/skins/default/main.png
Comment 12 Richard Walker 2012-03-19 17:05:23 CET
I have a Beta2 which has never had mplayer on it - I'll try it now on this one. By the way, the problem was not with the existence of main.png. It was always there. It just broke the gui startup. If you change it for main-silver.png in the config file then it would get past that and break at exit.png (I think - I'll have to check the original discussion on Mageia forum).

Richard
Comment 13 Richard Walker 2012-03-19 17:11:08 CET
Sorry can't test it yet. I have this to fix first:

[richard@Tupperware ~]$ mplayer 
mplayer: relocation error: /usr/lib64/libavformat.so.53: symbol avpriv_ac3_channel_layout_tab, version LIBAVCODEC_53 not defined in file libavcodec.so.53 with link time reference
Comment 14 Richard Walker 2012-03-19 17:39:52 CET
OK, I had half a dozen pending updates. One of them must have fixed the library problem.

Situation as before, still fails trying to load main.png. The config file I referred to is /usr/share/mplayer/skins/default/skin where you see:

[...]
;  base = main-silver, -2, -2
  base = main, -2, -2
[...]

I change that to:

[...]
  base = main-silver, -2, -2
;  base = main, -2, -2
[...]

and the error report moves to the next load which is exit.png
Comment 15 Wolfgang Bornath 2012-03-19 18:22:22 CET
(In reply to comment #11)
> That's really strange.  I just tested it and it works fine for me.
> 
> ls -l /usr/share/mplayer/skins/default/main.png
> -rw-r--r-- 1 root root 30945 Mar 19 10:41 main.png
> 
> md5sum /usr/share/mplayer/skins/default/main.png
> ea762e5eec6a9c64bc19785553d6c163  /usr/share/mplayer/skins/default/main.png

My main.png is dated Mar 19 16:02 but it has the same md5sum as yours. Still I have the same error as described in Comment #10
Comment 16 Wolfgang Bornath 2012-03-19 18:42:18 CET
Just did a fresh installation of mplayer-gui on another Beta2 where mplayer-gui was not installed before: same error.
Comment 17 Ben Bullard 2012-03-20 13:36:35 CET
Just updated my Beta 2/Cauldron. Current output of 'rpm -qa | grep mplayer'.

kmplayer-0.11.3b-1.mga2
gnome-mplayer-1.0.5-4.mga2
mplayer-1.0-1.rc4.0.r34578.5.mga2.tainted
mplayer-gui-1.0-1.rc4.0.r34578.5.mga2.tainted
smplayer-0.7.1-3.mga2
smplayer-themes-20120131-1.mga2

Error still exists in mplayer. Same files will open with kmplayer, gnome-mplayer, or smplayer.
Comment 18 David Walser 2012-03-25 17:57:18 CEST
Reproduced in a new Cauldron VM at work.  A couple things I noticed:

In the strace output when it's reading the main.png file, right before it closes it, it looks like the one that works seeks to the end, and the other one doesn't, right before closing it.  Missing from the one where it isn't working is:
_llseek(9, 30945, [30945], SEEK_SET) = 0

In the ltrace output, it's basically exactly the same until after reading the main.png file.  Then in the working one it does a calloc call and then continues on reading the skin file:
calloc(1, 6600) = 0x0a364500

In the one that isn't working, it does some different calls and then starts printing out the error message about the skin config file.  Is there possibly an ASCII/Unicode problem here?
__vsnprintf_chk(0xbf95fa4c, 512, 1, 512, 0x82ed885) = 56
__vsnprintf_chk(0xbf95f80c, 512, 1, 512, 0x82ed9c4) = 94
__vsnprintf_chk(0xbf95df9c, 3072, 1, 3072, 0x8438190) = 94
strcasecmp("UTF-8", "noconv") = 7
strcmp("UTF-8", "UTF-8") = 0
iconv(0x9133fd0, 0xbf95df94, 0xbf95df8c, 0xbf95df98, 0xbf95df90) = 0
Comment 22 Richard Walker 2012-03-25 23:12:55 CEST
Problem is still evident here.
 Currently installed version: 1.0-1.rc4.0.r34578.7.mga2.tainted
Comment 23 David Walser 2012-03-26 00:35:16 CEST
OK, found two more commits from another poster on the thread Funda linked.  These definitely look needed as well:
http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/2012-January/043539.html
http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/2012-January/043540.html
Comment 24 David Walser 2012-03-26 00:50:58 CEST
mplayer-1.0-1.rc4.0.r34578.8.mga2 is submitted.  Fingers crossed :o)
Comment 25 Richard Walker 2012-03-26 01:15:41 CEST
Well, that works for me. What can I say? Thank you will have to do.

Richard
Comment 26 David Walser 2012-03-26 01:41:50 CEST
Thank you Ben and Richard for reporting and testing this, and for your patience.

Closing.

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED

Comment 27 Wolfgang Bornath 2012-03-26 16:26:03 CEST
Good job, solution confirmed.

Note You need to log in before you can comment on or make changes to this bug.