Possible restore bug in su_load_gmdls

[mrange]

Hi.

Big fan of the sound and size of sointu. I thought I noticed one possible issue in su_load_gmdls.

In my version it does

mov     ebx, su_sample_table

but AFAICT it doesn't restore ebx.

In certain configs this fails for me as calling code don't expect rbx to be modified.

When I removed that row and instead

push    ebx

does this:

push    su_sample_table

it seems to resolve the issue for me.

So I have a workaround so it's not a big issue for me.

[vsariola]

Oops. I need to read more carefully what the 32-bit calling convention says. Thanks for reporting!

[vsariola]

Pushad / Popad is also only two bytes (uncompressed) so that might also work

[vsariola]

Should be fixed. Please test and report if this continues to be an issue.

[vsariola]

I push most of the arguments to ReadFile first so we can use only eax (which is free to trash) for the su_sample_table pointer; this way I don't have to trust OpenFile not touching anything if its implementation changes. Only the return value of OpenFile is pushed after its called.

[mrange]

Hi. I will make a try to see if it works. Thanks.