Bug 32532 - Guayadeque crashes inside Mageia9 when "enable volume and fader" is activated in settings
Summary: Guayadeque crashes inside Mageia9 when "enable volume and fader" is activated...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 9
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords:
: 32231 (view as bug list)
Depends on:
Blocks:
 
Reported: 2023-11-18 17:56 CET by Philippe Didier
Modified: 2023-11-25 18:38 CET (History)
7 users (show)

See Also:
Source RPM: guayadeque-0.4.7-1.mga9.src.rpm
CVE:
Status comment:


Attachments
spec file for guayadeque using a new source (2.54 KB, text/plain)
2023-11-18 17:58 CET, Philippe Didier
Details
corrected spec file for gauayadeque (2.57 KB, text/plain)
2023-11-20 15:21 CET, Philippe Didier
Details
diff for this newly corrected spec file (1006 bytes, patch)
2023-11-20 15:22 CET, Philippe Didier
Details | Diff
corrected spec file for Mageia9 and Cauldron (2.60 KB, text/x-matlab)
2023-11-20 16:02 CET, Philippe Didier
Details
diff with the actual spec from svn (1006 bytes, text/plain)
2023-11-20 16:11 CET, Philippe Didier
Details
diff with the actual spec from svn (640 bytes, patch)
2023-11-20 16:20 CET, Philippe Didier
Details | Diff

Description Philippe Didier 2023-11-18 17:56:21 CET
Guayadeque crashes in Mageia9 and in Cauldron when "enable volume and fader" and "enable equalizer" are activated

besides this lots of patches are needed to build it with wxgtk3.2.0

Changing the source to a fork https://github.com/openmonk/guayadeque
instead of https://github.com/anonbeat/guayadeque

allows now to build with the last versions of wxgtk3.2.0 and gstreamer1.22.3
without any patch
and might allow to get no more crash
Comment 1 Philippe Didier 2023-11-18 17:57:41 CET
This bug may replace 
https://bugs.mageia.org/show_bug.cgi?id=32231
which begins to be unreadable
Comment 2 Philippe Didier 2023-11-18 17:58:46 CET
Created attachment 14167 [details]
spec file for guayadeque using a new source
Comment 3 Philippe Didier 2023-11-18 18:11:52 CET
NB
I built this rpm with mock and then with Mageia BS and it's OK


BUT This updated rpm doesn't work for everyone
Some people get an immediate crash when launching it :
https://bugs.mageia.org/show_bug.cgi?id=32231#c54
https://bugs.mageia.org/attachment.cgi?id=14160
https://bugs.mageia.org/show_bug.cgi?id=32231#c72

and we need to investigate further

That seems caused for them by something interacting badly with 
/lib64/libwx_gtk3u_wxsqlite3-3.2.so.0"
katnatek 2023-11-18 18:32:27 CET

CC: (none) => j.alberto.vc, jani.valimaa, ngompa13

Comment 4 katnatek 2023-11-18 18:43:55 CET
I test %global optflags %{optflags} -fuse-ld=gold , to force the use of gold as ld tool and keep as-needed ldflags but that not works

The only that allows to me to use this in both arches is 

%global _disable_ld_as_needed 1

It's not an issue of missing require, I have all the wx requires in both systems
Maybe I'm wrong but also don't believe is an issue with missing symbols, I don't think in missing symbols case that disable as-needed fix the application in both arches

I think the current and testing version crash for me and others that never before use this application because both "enable volume and fader" and "enable equalizer" are default settings, I see that in my custom build that don't crash

The suggested change makes the testing version works even on Mageia 8 i586
Comment 5 Marja Van Waes 2023-11-18 18:46:28 CET
*** Bug 32231 has been marked as a duplicate of this bug. ***
Jani Välimaa 2023-11-18 18:59:58 CET

Summary: correct guayadeque crashes inside Mageia9 when "enable volume and fader" is activated in settings => Guayadeque crashes inside Mageia9 when "enable volume and fader" is activated in settings

Comment 6 Philippe Didier 2023-11-18 19:30:10 CET
@ Jani
I really don't know what to do with the corrected spec file since its version is now lower that the one in SVN
Is it possible nevertheless to use mgarepo to upload it ?


NB I rebuilt guayadeque rpm with this corrected spec file inside mock
I uninstalled my last built rpm and installed this one
I renamed my /home/myusername/.guayadeque to /home/myusername/.guayadequebak not to pollute it
I launched guayadeque which created a new /home/myusername/.guayadeque
added a path to a music directory (guayadeque built its database and displayed the library
I created a playlist
Played the playlist (with fader and equalizer enabled)
changed the track
modified the equalizer

NO CRASH

stopped guayadeque

I then deleted the new created /home/myusername/.guayadeque
Renamed my backed up /home/myusername/.guayadequebak to/home/myusername/.guayadeque

Launched again Guayadeque
It used my whole previous library (verifying if the database is up to date)
I can play whatever I want without any crash !

Strange isn't it ?
Comment 7 Philippe Didier 2023-11-19 15:28:19 CET
This solution for this bug made an other bug appear for some testers
who tested both the current 0.4.7-1 version and the proposed update
https://bugs.mageia.org/show_bug.cgi?id=32535
For them Guayadeque crashes immediately for an other reason
Philippe Didier 2023-11-19 15:40:31 CET

CC: (none) => andrewsfarm, geiger.david68210

Philippe Didier 2023-11-19 15:40:54 CET

CC: (none) => marja11

Comment 8 Philippe Didier 2023-11-19 16:00:41 CET
the useful part of the initial crash report created by Guayadeque itself for this problem 
https://bugs.mageia.org/attachment.cgi?id=13965 
indicated that something went wrong with Gstreamer :

 <stack>
    <frame level="0" function="Guayadeque::guMainApp::OnFatalException()" offset="0" address="0x56e9ba"/>
    <frame level="1" offset="0x1d8570" address="0x7fbdfd5d8570" module="/lib64/libwx_baseu-3.2.so.0"/>
    <frame level="2" offset="0x36980" address="0x7fbdfbb88980" module="/lib64/libc.so.6"/>
    <frame level="3" function="Guayadeque::guGstStateToNullAndUnref(_GstElement*)" offset="0" address="0x58410b"/>
    <frame level="4" function="Guayadeque::guFaderPlaybin::~guFaderPlaybin()" offset="0" address="0x570ede"/>
    <frame level="5" function="Guayadeque::guFaderPlaybin::~guFaderPlaybin()" offset="0" address="0x571079"/>
    <frame level="6" function="Guayadeque::guMediaCtrl::DoCleanUp()" offset="0" address="0x58756f"/>
    <frame level="7" function="Guayadeque::guMediaCtrl::DoCleanUp()" offset="0" address="0x5878c9"/>
    <frame level="8" offset="0x59164" address="0x7fbdfc75a164" module="/lib64/libglib-2.0.so.0"/>
    <frame level="9" function="g_main_context_dispatch" offset="0x149" address="0x7fbdfc759689" module="/lib64/libglib-2.0.so.0"/>
    <frame level="10" offset="0x58a18" address="0x7fbdfc759a18" module="/lib64/libglib-2.0.so.0"/>
    <frame level="11" function="g_main_loop_run" offset="0x6f" address="0x7fbdfc759cbf" module="/lib64/libglib-2.0.so.0"/>
    <frame level="12" function="gtk_main" offset="0x75" address="0x7fbdfb3ecc45" module="/lib64/libgtk-3.so.0"/>
    <frame level="13" function="wxGUIEventLoop::DoRun()" offset="0x25" address="0x7fbdfcd723e5" module="/lib64/libwx_gtk3u_core-3.2.so.0"/>
    <frame level="14" function="wxEventLoopBase::Run()" offset="0x2d" address="0x7fbdfd4d72ad" module="/lib64/libwx_baseu-3.2.so.0"/>
    <frame level="15" function="wxAppConsoleBase::MainLoop()" offset="0x6b" address="0x7fbdfd4a4afb" module="/lib64/libwx_baseu-3.2.so.0"/>
    <frame level="16" function="wxEntry(int&amp;, wchar_t**)" offset="0x47" address="0x7fbdfd51ec67" module="/lib64/libwx_baseu-3.2.so.0"/>
    <frame level="17" function="main" offset="0" address="0x561a88"/>
    <frame level="18" offset="0x236b7" address="0x7fbdfbb756b7" module="/lib64/libc.so.6"/>
    <frame level="19" function="__libc_start_main" offset="0x85" address="0x7fbdfbb75775" module="/lib64/libc.so.6"/>
    <frame level="20" function="_start" offset="0" address="0x566fd1"/>
  </stack>

The first patch proposed by Jani to disable fader and equalizer was a workaround

The guayadeque dev brought minor corrections to comply with the lat version of gstreamer and wxgtk 
This kind of crash has totally disappeared
Comment 9 Philippe Didier 2023-11-20 15:21:57 CET
Created attachment 14172 [details]
corrected spec file for gauayadeque

This modified spec file has a correct version numbering
And an added Requires : binutils

This Requires is needed since binutils is not installed by default on a simple install of Mageia9
see :
https://bugs.mageia.org/show_bug.cgi?id=32535#c15
https://bugs.mageia.org/show_bug.cgi?id=32535#c38

That solves partly this other bug

Attachment 14167 is obsolete: 0 => 1

Comment 10 Philippe Didier 2023-11-20 15:22:54 CET
Created attachment 14173 [details]
diff for this newly corrected spec file
Comment 11 Philippe Didier 2023-11-20 15:25:29 CET
NB 
the corrected spec files has not yet been uploaded to the svn with mgarepo neither for cauldron nor for Mageia9/core/updates_testing
Comment 12 Philippe Didier 2023-11-20 16:02:11 CET
Created attachment 14174 [details]
corrected spec file for Mageia9 and Cauldron

Attachment 14172 is obsolete: 0 => 1

Comment 13 Philippe Didier 2023-11-20 16:11:15 CET
Created attachment 14175 [details]
diff with the actual spec from svn
Comment 14 Philippe Didier 2023-11-20 16:20:00 CET
Created attachment 14176 [details]
diff with the actual spec from svn

Attachment 14173 is obsolete: 0 => 1
Attachment 14175 is obsolete: 0 => 1

Comment 15 Philippe Didier 2023-11-20 16:30:34 CET
I think that when this will have been submitted to 
Cauldron and 
Mageia9/core/updates_testing
and validated for this problem
this bug might be closed as resolved

 two other bugs will remain 
https://bugs.mageia.org/show_bug.cgi?id=32535
https://bugs.mageia.org/show_bug.cgi?id=32536
They are caused by something else ... and prevent some users to test this solution which is valid for this particular bug !
Philippe Didier 2023-11-20 19:10:58 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=32535

Philippe Didier 2023-11-20 19:11:35 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=32536

Comment 16 Lewis Smith 2023-11-20 22:51:40 CET
https://bugs.mageia.org/show_bug.cgi?id=32535#c40
4) Update guayadeque from updates_testing
launch it
enable fader and equalizer
no more crash ok

https://bugs.mageia.org/show_bug.cgi?id=32535#c41
4) Update guayadeque from updates_testing (0.4.8-4.git20231115.1)
launch it
enable fader and equalizer
no more crash           OK

Looks like the pending update fixes the problem, good for OK?
Except the new issue of a requires for binutils.

CC: (none) => lewyssmith

Comment 17 Philippe Didier 2023-11-20 23:15:34 CET
Hi Lewis
Indeed there will be a new rpm in updates_testing :
I made a mistake with the version numbering : 0.4.8-4.git20231115.1 is not right

Waiting for sysadmins to remove it before submitting a better number : that should have  been : 0.4.7-3.git20231115.1 

In the mean time
I modified again the spec to add Requires : binutils to correct partly https://bugs.mageia.org/show_bug.cgi?id=32535
and the version will be now 0.4.7-3.git20231115.2

It's not yet submitted !
Comment 18 Thomas Andrews 2023-11-21 00:54:11 CET
(In reply to Lewis Smith from comment #16)

> Except the new issue of a requires for binutils.

It does indeed require binutils to supply the addr2line command, but that, in and of itself, doesn't fix the new issue. 

I tried simply installing binutils, and while the complaint of no addr2line command went away, guayadeque still crashed right away.

It's my belief that there may be yet another missing dependency, which in turn is dependent on binutils. But, not being a developer myself, all I can do is speculate on that.
Comment 19 Philippe Didier 2023-11-21 01:35:21 CET
(In reply to Thomas Andrews from comment #18)
> (In reply to Lewis Smith from comment #16)
> 
> > Except the new issue of a requires for binutils.
> 
> It does indeed require binutils to supply the addr2line command, but that,
> in and of itself, doesn't fix the new issue. 
> 
> I tried simply installing binutils, and while the complaint of no addr2line
> command went away, guayadeque still crashed right away.
> 
> It's my belief that there may be yet another missing dependency, which in
> turn is dependent on binutils. But, not being a developer myself, all I can
> do is speculate on that.

Hi Thomas
I splitted the bug in two parts
this one is only for the fader and equalizer problem
for the crash that you encountered I created bug 32535
Where I will dig again since the binutils problem is solved
Comment 20 Philippe Didier 2023-11-21 01:39:42 CET
@ Thomas
look at this :
https://bugs.mageia.org/show_bug.cgi?id=32535#c41
that really make me think that some other hidden dependency is missing for you
Philippe Didier 2023-11-24 17:47:27 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=32550

Comment 21 Philippe Didier 2023-11-25 18:38:04 CET
resolved by bug 32550

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


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