SCUMM: COMI: core dumped after a while

[mcanonic]

Hi,
in my linux fedora 35 and ScummVM 2.0.6 I have no problems since the end of Chapter 2 when I got this error from the console:

[code]scummvm: engines/scumm/imuse_digi/dimuse_sndmgr.cpp:245: void Scumm::ImuseDigiSndMgr::scheduleSoundForDeallocation(int): Assertion `checkForProperHandle(soundDesc)' failed.
Aborted (core dumped)
code

Before that error, when I load the game I get these warning but they do not really have some problem with the game
[code]User picked target 'comi-it' (engine ID 'scumm', game ID 'comi')...

Looking for a plugin supporting this target... SCUMM [all games]

WARNING: BundleMgr::open() Can't open bundle file: voxdisk2.bun!
WARNING: BundleMgr::open() Can't open bundle file: musdisk2.bun!
WARNING: BundleMgr::open() Can't open bundle file: musdisk2.bun!
WARNING: BundleMgr::open() Can't open bundle file: musdisk2.bun!
code

To reproduce the bug, nothing particular has to be done: I've just finished the second part.

Before that, the game works perfectly so I suppose that the source code and CDs imported are ok.

[mcanonic]

The problem was related to that warning.
The files were missing, so scummvm should rise an error not a warning.

I was missing that file due to a problem on the CD's files extraction.

You can close the ticket, but I suggest to think about these "warning".

[AndywinXp]

Grazie!
We try to provide a smooth gaming experience even when not-strictly-necessary files (music and voices) are not used (e.g. you might want to copy the files from Steam to your tablet/smartphone but you don't want to occupy all that space).

This is the reason why that's a warning instead of an error. But indeed, it shouldn't behave like this and it certainly shouldn't throw the assertion error in this case: I'll see what I can do!

Ciao!

[AndywinXp]

Fixed in ​https://github.com/scummvm/scummvm/commit/c1502271b417b0438f19af58c8f1646a81773baa