Mixxx 1.6.0beta2 Segfaults while "Scratching" about mixxx HOT 7 CLOSED

Commented by: asantoni
Date: 2008-02-17T16:50:15Z

Just out of curiosity, what is the jog wheel control set to in your MIDI mapping? (This probably has nothing to do with the problem...)

from mixxx.

Commented by: alex-malexmedia
Date: 2008-02-17T18:23:40Z

My jog wheels are set like so:

<control>
	<group>[Channel1]</group>
	<key>jog</key>
	<miditype>Ctrl</miditype>
	<midino>25</midino>
	<sensitivity>10</sensitivity>
	<options><diff/></options>
</control>

However, my codebase has a few lines changed in the configobject.cpp which make diff a little saner for my setup.

from mixxx.

Commented by: alex-malexmedia
Date: 2008-02-17T18:24:50Z
Attachments: [Backtrace of a locked-up mixxx.](https://bugs.launchpad.net/mixxx/+bug/192663/+attachment/209334/+files/Backtrace of a locked-up mixxx.)

As you requested, I tried to cause the same problem in pitch-independent stretch mode. Mixxx locked up instead of segfaulting, but here is the backtrace.

from mixxx.

Commented by: alex-malexmedia
Date: 2008-02-17T18:44:16Z
Attachments: [Lots and Lots of Valgrind](https://bugs.launchpad.net/mixxx/+bug/192663/+attachment/209338/+files/Lots and Lots of Valgrind)

Caused the same segfault while running under valgrind.

from mixxx.

Commented by: alex-malexmedia
Date: 2008-02-17T22:34:43Z
Attachments: [Patch for issue.](https://bugs.launchpad.net/mixxx/+bug/192663/+attachment/209437/+files/Patch for issue.)

Fixed!

When the seek speed is too low, we essentially ask SoundTouch to expand our 100-sample buffer into literally gigabytes of RAM. (Which causes SoundTouch to overflow an int and segfault.)

This patch adds MIN_SEEK_SPEED, which successfully prevents such nonsense from ever happening.

from mixxx.

Commented by: alex-malexmedia
Date: 2008-02-18T01:30:30Z
Attachments: [A fix for the fix.](https://bugs.launchpad.net/mixxx/+bug/192663/+attachment/209498/+files/A fix for the fix.)

Discovered another case where soundtouch->setRate() could be called with a super-small number.

sheepish Guess I need to do my homework better...

from mixxx.

Issue closed with status Fix Released.

from mixxx.