Media Foundation plugin on Windows 7 fails to load M4A

Bug #885049 reported by RJ Skerry-Ryan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
William Good

Bug Description

From mixxx.log after trying to load a test M4A I found on the Internet:

Debug: [Main]: Dynamically loaded "C:/Program Files (x86)/Mixxx/plugins/soundsource/soundsourcemediafoundation.dll"
Debug: [Main]: m4a
Debug: [Main]: mp4
Debug: [Main]: Plugin supports: m4a
Debug: [Main]: Plugin supports: mp4
Debug: [Main]: Failed to dynamically load "C:/Program Files (x86)/Mixxx/plugins/soundsource/soundsourcemediafoundation.exp" "Cannot load library C:/Program Files (x86)/Mixxx/plugins/soundsource/soundsourcemediafoundation.exp: "
Debug: [Main]: Failed to dynamically load "C:/Program Files (x86)/Mixxx/plugins/soundsource/soundsourcemediafoundation.lib" "Cannot load library C:/Program Files (x86)/Mixxx/plugins/soundsource/soundsourcemediafoundation.lib: "

.....

Debug: []: Getting SoundSource plugin object for "m4a"
Warning: []: SSMF: failed to set media type
Debug: []: SSMF: Duration: 37
Fatal: []: ASSERT: "bufferLength * kNumChannels <= m_leftoverBufferSize" in file plugins\soundsourcemediafoundation\soundsourcemediafoundation.cpp, line 264
Warning: []: QObject::killTimer: timers cannot be stopped from another thread
Warning: []: QObject::killTimers: timers cannot be stopped from another thread

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :
Changed in mixxx:
milestone: none → 1.10.0
assignee: nobody → Bill Good (bkgood)
importance: Undecided → High
status: New → Confirmed
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

And the plugin DLL I built (for 32-bit) is attached.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

This file seems to trigger an assertion now:

Fatal: []: ASSERT: "m_nextFrame >= bufferPosition" in file plugins\soundsourcemediafoundation\soundsourcemediafoundation.cpp, line 248

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

(It happens on track load)

Revision history for this message
William Good (bkgood) wrote :

WMP 12 uses Media Foundation to play AAC and it's unable to play this Eagles sample, so I think we're SOL on that one (unless we figure out how to use MF better than Microsoft). I'll try to figure out how to get it to fail more peacefully, though.

I committed a change that keeps the failure to load thing from triggering an assert, you now just get a "track could not be loaded" dialog as normal. Media Foundation is able to decode the Nintendo file though, so I'm going to work on that yet.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Semi-broken files are now handled in a semi-reasonable way.

Changed in mixxx:
status: Confirmed → Fix Committed
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/6067

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.