Open Wohlstand opened 7 years ago
Original comment by Michael Pyne (Bitbucket: mpyne, GitHub: mpyne).
I can confirm some kind of problem. I get this error message when running under libgme current git master compiled with gcc 5.4 and UBSan:
/home/kde-svn/projects/bitbucket/game-music-emu/gme/Ay_Apu.cpp:136:33:
runtime error: index 117 out of bounds for type 'unsigned char [8][48]'
Looking at the code, it doesn't really appear to have the relatively careful usage of types that much of the rest of the code has (e.g. int
everywhere as opposed to unsigned
or fixed-width types). Until I can add improved sanity checking to keep the code from accessing memory illegally I think I'll have to at least remove it from the default build.
Original report by Hanno Böck (Bitbucket: [Hanno Böck](https://bitbucket.org/Hanno Böck), ).
The file PMD_4.ay from the Project AY [1], downloadable in this archive [2], causes an invalid memory read in the function Ay_Apu::run_until(), detectable with address sanitizer. I'll attach the file.
[1] http://www.worldofspectrum.org/projectay/gdmusic.htm [2] http://projectay.dl.spilsby.net/Update-06March2008.zip
ASAN stack trace:
PMD_4.ay.zip