#2411 closed defect (fixed)
mplayer (+Xwayland) crashes with a floating-point math exception
| Reported by: | plorenzo | Owned by: | beastd |
|---|---|---|---|
| Priority: | normal | Component: | undetermined |
| Version: | unspecified | Severity: | blocker |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Reproduced by developer: | no | Analyzed by developer: | no |
Description
Version: SVN-38408
I'm forwarding from Debian, as I don't use wayland.
Using the 'nokeepaspect' option under Xwayland crashes mplayer with a floating-point math exception;
also, when the 'wid' option is used under Xwayland mplayer complains with "X11 error: BadWindow (invalid Window parameter)" and only audio is played.
This was reported in Debian [1] as bug against SMplayer, but the reporter shows that he can reproduce with mplayer.
There is also an open issue at SMplayer upstream[2], where there are more details and upstream claims to have patched version of both mplayer and mpv that hs's releasing as flatpack
Lorenzo
[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1034867
[2] https://github.com/smplayer-dev/smplayer/issues/668
Change History (6)
comment:1 by , 19 months ago
comment:3 by , 19 months ago
I'm still a bit confused about the linked reports. Maybe I'm misunderstanding things.
Not sure if this could possibly be fixed for good.
Guess I will need to try it out myself and see what happens and what would be possible.
Thanks for the reporting in any case!
comment:4 by , 19 months ago
I fixed the crashes in r38428, but that doesn't change that MPlayer can only work in SMPlayer if SMPlayer's main window is a X11 window - that is by design, even if we had wayland support, I believe sharing windows like that across processes is just not supported by wayland.
comment:5 by , 19 months ago
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
The crash should be fixed by Reimar in r38428 .
Closing as invalid under the assumption that this kind of sharing between processes doesn't work in Walyand.
Just re-open if you think that's not the case or have other ideas.
comment:6 by , 19 months ago
| Resolution: | invalid → fixed |
|---|
Sorry. Got a bit confused :(
This ticket was actually about the crashes only and thus it should be closed as fixed.
Anyway feel free to re-open if something seems wrong or the fix was incomplete.
As per upstream bug, this is a case of an invalid -wid parameter.
So the window that MPlayer is supposed to render into does not exist.
The error handling clearly should be fixed to not crash, but it will not really fix the main issue, so not very high priority to look into this.
(not sure about the patched version of mplayer statement, it sounds like the flatpak simply has SMPlayer run via X11 instead of wayland as well, which makes it work).