Crash in AnalyzerEbur128

Bug #1573904 reported by Uwe Klotz
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Undecided
Unassigned

Bug Description

Build: master (WriteAudioTags) using version 1.0.2 of libebur128 that comes with Fedora 23

The following crash happened only once. It's not reproducible and probably caused by a race condition:

Debug [CachingReaderWorker 2]: Retry decoding MP3 frame @ 2964096
Debug [AnalyzerQueue 1]: AnalysisDAO fetched 2 analyses, 1384987 bytes for track 172160 in 32 ms
Debug [AnalyzerQueue 1]: Reading waveform from byte array: allSignalSize 148886 visualSampleRate 441 audioVisualRatio 100
Debug [AnalyzerQueue 1]: Reading waveform from byte array: allSignalSize 3840 visualSampleRate 11.3742 audioVisualRatio 3877.2
Debug [AnalyzerQueue 1]: AnalyzerWaveform::loadStored - Stored waveform loaded
Debug [AnalyzerQueue 1]: Keys version/sub-version unchanged since previous analysis. Not analyzing.
Debug [AnalyzerQueue 1]: Interrupting analysis to give preference to a loaded track.

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fff91703700 (LWP 5226)]
0x00007ffff7681c47 in ebur128_destroy () from /usr/lib64/libebur128.so.1
(gdb) bt
#0 0x00007ffff7681c47 in ebur128_destroy () at /usr/lib64/libebur128.so.1
#1 0x00000000004b3bdd in AnalyzerEbur128::cleanup(QSharedPointer<TrackInfoObject>) (this=0x5d97140, tio=...) at src/analyzer/analyzerebur128.cpp:44
#2 0x00000000004bfde5 in AnalyzerQueue::run() (this=0x5dbd610)
    at src/analyzer/analyzerqueue.cpp:352
#3 0x00007ffff431838c in QThreadPrivate::start(void*) ()
    at /usr/lib64/libQtCore.so.4
#4 0x00007ffff1d1e60a in start_thread () at /usr/lib64/libpthread.so.0
#5 0x00007fffeeeb7a4d in clone () at /usr/lib64/libc.so.6

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

The initialization code in AnalyzerEbur128 is not really safe even in the single-threaded case due to some redundancies. I will fix this.

Changed in mixxx:
assignee: nobody → Uwe Klotz (uklotzde)
Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :
Changed in mixxx:
status: New → In Progress
Changed in mixxx:
status: In Progress → Fix Committed
Changed in mixxx:
milestone: none → 2.1.0
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/8520

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

Remote bug watches

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