Do not free the header while unmapping the xxxx_UNPREPARE messages
Eric Pouech
eric.pouech at wanadoo.fr
Wed Dec 25 10:19:03 CST 2002
Dmitry Timoshkov wrote:
> Hello,
>
> this patch is aimed to fix crashes in the old 16-bit game, which
> calls waveOutUnprepareHeader16 for busy headers. MMDRV_Message
> in that case returns WAVERR_STILLPLAYING, but it's not stopping
> MMDRV_WaveOut_UnMap16To32A() from freeing the header. Slightly
> later game calls waveOutReset16() which crashes. In that case
> the duty of freeing the header should be moved to a higher level.
> I have generalized this approach to all similar cases.
I'd rather see the fix being made completely in message16.c by adding
the return value to the unmapping functions
IMO, it better encapsulates all the 32<=>16 message mappings, and it's
bad to spread this across several compilation unit
A+
--
Eric Pouech
More information about the wine-devel
mailing list