mixxxdj / mixxx

Mixxx is Free DJ software that gives you everything you need to perform live mixes.
http://mixxx.org
Other
4.45k stars 1.27k forks source link

Mixxx crashes on start with another Qt program running in Linux #8304

Closed mixxxbot closed 2 years ago

mixxxbot commented 2 years ago

Reported by: Pegasus-RPG Date: 2015-11-19T04:21:06Z Status: Invalid Importance: High Launchpad Issue: lp1517723 Tags: crash, jack Attachments: [Last few log lines and backtrace](https://bugs.launchpad.net/bugs/1517723/+attachment/4521678/+files/Last few log lines and backtrace)


Steps to reproduce: 1) Start qjackctl as an example (JACK server doesn't actually need to be running) 2) Start Mixxx master (r5490) 3) Observe segfault.

Backtrace attached.

mixxxbot commented 2 years ago

Commented by: Pegasus-RPG Date: 2015-11-19T04:21:06Z Attachments: [Last few log lines and backtrace](https://bugs.launchpad.net/mixxx/+bug/1517723/+attachment/4521678/+files/Last few log lines and backtrace)

mixxxbot commented 2 years ago

Commented by: Pegasus-RPG Date: 2015-11-19T04:29:07Z


Apparently it's related to the QString::fromUcs4() and QString::fromWCharArray() stuff you guys have been hacking on, but it only happens when another Qt program (like qjackctl) is running.

mixxxbot commented 2 years ago

Commented by: daschuer Date: 2015-11-19T10:12:59Z


I cannot confirm this on Ubuntu Trusty using latest Mixxx 1.12

However, you log contains: 
Warning [Main]: DEBUG ASSERT: "false" in file src/sounddevicenetwork.cpp, line 76
This is a different pending issue if no sound device is configured.   

Can you reproduce the crash reliable?

Is it fixed with after reverting one of these commits 67d56f51430b556c1329875f2ab5390178fee51d 67d56f51430b556c1329875f2ab5390178fee51d

mixxxbot commented 2 years ago

Commented by: ywwg Date: 2015-11-19T14:39:14Z


the actual crash appears to be here:

#0  0x00007ffff4a61d15 in ?? () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
mixxxdj/mixxx#4910  0x00007ffff496667d in QString::fromUcs4(unsigned int const*, int) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
mixxxdj/mixxx#4911  0x00007ffff49666d9 in QString::fromWCharArray(wchar_t const*, int) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
mixxxdj/mixxx#4912  0x00000000005d1688 in safeDecodeWideString (max_length=512, pStr=0x7fff84028a80 L"Kensington Eagle Trackball")
    at src/controllers/hid/hidcontroller.cpp:59

is this code absolutely up to date? RJ and Daniel have been applying some fixes to the safeDecodeWideString function

mixxxbot commented 2 years ago

Commented by: Pegasus-RPG Date: 2015-11-19T15:44:57Z


This is happening on master. Have those fixes been merged to master?

mixxxbot commented 2 years ago

Commented by: ywwg Date: 2015-11-19T15:53:32Z


Yes as of yesterday https://github.com/mixxxdj/mixxx/commit/67d56f51430b556c1329875f2ab5390178fee51d

mixxxbot commented 2 years ago

Commented by: daschuer Date: 2015-11-19T15:54:04Z


Yes, the backtrace indicates that you are on a recent version including the commit from my post #⁠3

mixxxbot commented 2 years ago

Commented by: ywwg Date: 2015-11-19T15:54:31Z


based on the line number of the crash, it looks like your version might be out of date. Line 59 is a comment now

mixxxbot commented 2 years ago

Commented by: Pegasus-RPG Date: 2015-11-19T16:13:54Z


Yup, I just updated again and it's fine now. Sorry for the false alarm.

mixxxbot commented 2 years ago

Issue closed with status Invalid.