AddMusicKFF icon indicating copy to clipboard operation
AddMusicKFF copied to clipboard

Infinite loop when "Addmusic_sample groups.txt" includes invalid data

Open spooonsss opened this issue 2 years ago • 1 comments

I assumed Addmusic_sample groups.txt would support comments, but it puts the parser into an infinite loop instead.

Try Addmusic_sample groups.txt:


;f

https://github.com/KungFuFurby/AddMusicKFF/blob/312f3454b87808baac4001e762f634e89cac2280/src/AddmusicK/AddmusicK.cpp#L609 needs an else that throws an error.

(I'd suggest it support comments too)

spooonsss avatar Aug 17 '23 19:08 spooonsss

Yup. I can see how this can occur: simply by not even finding a sample group name to begin with and not having gotten any samples either and running into any non-alphanumeric character. For whatever reason, I may have other suspicions on the ones that do output errors...

KungFuFurby avatar Aug 17 '23 20:08 KungFuFurby