Bug#726912: libsdl-mixer1.2: Music with TiMidity backend plays too fast

[Sylvain]

Hi,

On Sun, Oct 20, 2013 at 06:10:24PM +0100, Manuel A. Fernandez Montecelo wrote:
> It would be great if you could recompile the deb package with that
> patch disabled and test it if plays the file, and other files of
> freedink, and see if it doesn't segfault (some modification of the
> current patch would be needed, I guess, that's why the patch was
> devised in the first place).

As I'm such a great guy I ran a few tests ;)

SDL_FORCE_SOUNDFONTS=1 ./playmus /usr/share/games/dink/dink/Sound/104.mid
SDL_FORCE_SOUNDFONTS=1 ./playmus /tmp/midibug/sound/10.mid 


Results:

upstream + bare ./configure
- 104.mid: OK
- minibug: OK

upstream + bare ./configure + bug-694260-freedink_stack_corruption.patch:
- 104.mid: faster tempo
- minibug: slower tempo

current debian:
- 104.mid: faster tempo
- minibug: still crashes! #694260 ain't fixed!

current debian - all patches:
- 104.mid: OK
- minibug: still crashes!

current debian - all patches - -fstatck-protector:
- 104.mid: OK
- minibug: OK


Conclusions:
- bug-694260-freedink_stack_corruption.patch doesn't fix the crash
- bug-694260-freedink_stack_corruption.patch introduces a new bug
Sorry pals :)

Cheers!
Sylvain