crash when analyzing corrupt FLAC file

Bug #1406815 reported by Be
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Critical
Unassigned

Bug Description

Mixxx crashes when trying to analyze a corrupt FLAC file. This occurs in both 1.11 stable and a recent 1.12 build.

This is what happens when trying to convert the file to WAV:

[be@localhost Luminous EP]$ flac -d 03\ -\ Darkness\ \(Faded\ Mix\).flac

flac 1.3.1, Copyright (C) 2000-2009 Josh Coalson, 2011-2014 Xiph.Org Foundation
flac comes with ABSOLUTELY NO WARRANTY. This is free software, and you are
welcome to redistribute it under certain conditions. Type `flac' for details.

03 - Darkness (Faded Mix).flac: *** Got error code 0:FLAC__STREAM_DECODER_ERROR_STATUS_LOST_SYNC
*** Got error code 0:FLAC__STREAM_DECODER_ERROR_STATUS_LOST_SYNC

03 - Darkness (Faded Mix).flac: ERROR while decoding data
                                state = FLAC__STREAM_DECODER_READ_FRAME

Revision history for this message
Be (be.ing) wrote :
Revision history for this message
Be (be.ing) wrote :

crash report with Mixxx 1.11

Revision history for this message
Be (be.ing) wrote :

crash report with Mixxx 1.12 git

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

Confirmed: Crash caused by invalid blocksize in FLAC frame header that exceeds the maximum blocksize of the file.

Just fixed it in https://github.com/mixxxdj/mixxx/pull/411 that is scheduled for 1.13.

Happy new year, btw ;)

Changed in mixxx:
assignee: nobody → Uwe Klotz (uklotzde)
status: New → Confirmed
Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

I will prepare a backport of the fix for 1.12

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :
Changed in mixxx:
status: Confirmed → In Progress
Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
importance: Undecided → Medium
milestone: none → 1.12.0
status: In Progress → Fix Committed
Changed in mixxx:
importance: Medium → Critical
RJ Skerry-Ryan (rryan)
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/7783

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.