Crash (segfault) when using empty directory as resourcePath

Bug #1934560 reported by Michael
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Daniel Schürmann

Bug Description

Mixxx crashes when 'resourcePath' points to either an empty, or a non existing directory.
("mixxx" received signal SIGSEGV, Segmentation fault).

The 'Mixxx - Critical error' popup window appears correctly, stating that the skin directory is missing. After clicking 'OK' Mixxx tries to open, but crashes.
Maybe a better default behaviour would be to stop Mixxx when a critical error appears?

Revision history for this message
Michael (ehmic) wrote :

Here is the gdb output.

Revision history for this message
Jan Holthuis (holthuis-jan) wrote :

What Mixxx version does this affect? I cannot tell from the log (we should add an Info message).

Please copy the version (and git version string) from the about dialog.

Revision history for this message
Michael (ehmic) wrote :

Ah sorry, forgot to mention...

Version is: 2.3.0-8-gd3052f4fc5

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

main terminates regularly without a crash after a bunch of warnings and debug assertions:

warning [Main] Failed to load default skin "LateNight"
critical [Main] DEBUG ASSERT: "m_pSkin != nullptr" in function DlgPrefInterface::DlgPrefInterface(QWidget*, std::shared_ptr<mixxx::ScreensaverManager>, std::shared_ptr<mixxx::skin::SkinLoader>, UserSettingsPointer) at ../src/preferences/dialog/dlgprefinterface.cpp:48

critical [Main] "Skin directory does not exist: /tmp/mixxx_empty_res/skins"
critical [Main] DEBUG ASSERT: "pSkin && pSkin->isValid()" in function mixxx::skin::SkinPointer mixxx::skin::SkinLoader::getConfiguredSkin() const at ../src/skin/skinloader.cpp:122

warning [Main] Failed to load default skin "LateNight"
critical [Main] DEBUG ASSERT: "pSkin != nullptr && pSkin->isValid()" in function QWidget* mixxx::skin::SkinLoader::loadConfiguredSkin(QWidget*, QSet<ControlObject*>*, mixxx::CoreServices*) at ../src/skin/skinloader.cpp:145

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

2.3 crashes after an assertion:

Debug [Main]: =====================================
Debug [Main]: 0 ms stopping pending Library tasks
Critical [Main]: DEBUG ASSERT: "m_pScanner" in function void TrackCollectionManager::stopLibraryScan() at ../src/library/trackcollectionmanager.cpp:172

Thread 1 "mixxx" received signal SIGSEGV, Segmentation fault.

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Low
milestone: none → 2.3.1
Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

Probably difficult to fix for 2.3 and not worth the effort.

ronso0 (ronso0)
Changed in mixxx:
milestone: 2.3.1 → 2.3.2
Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
assignee: nobody → Daniel Schürmann (daschuer)
status: Confirmed → In Progress
Changed in mixxx:
status: In Progress → Fix Committed
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/10461

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.