ntdll: Do not execute callbacks past DeleteTimer(INVALID_HANDLE_VALUE).

[Joerg-Cyril.Hoehle at t-systems.com]

Hi,

If you've run my CreateTimerQueue tests found on testbot, you'll have noticed that callbacks
are running in Wine even after DeleteTimerQueue(INVALID_HANDLE_VALUE).
That is not supposed to happen.

This is no good and can cause mysterious and hard to reproduce crashes, as apps are
going to free resources after destruction of the timer queue.

I investigated the issue and here's a patch.
This bug has been in Wine since DeleteTimerQueue was added in 2008.

Perhaps that's why Andrew Eikum saw an improvement in wineoss after moving
away from using INVALID_HANDLE_VALUE.

Even though I don't have a bug number at hand, getting rid of time bombs should be considered for 1.4.0.

Note that this affects EXECUTEINTIMERTHREAD above all. I've not investigated
native's event signaling behaviour when independent worker threads are used.

Regards,
 Jörg Höhle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ntdll-Do-not-execute-callbacks-past-DeleteTimer-INVA.patch
Type: application/octet-stream
Size: 968 bytes
Desc: 0001-ntdll-Do-not-execute-callbacks-past-DeleteTimer-INVA.patch
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20120307/f738122c/attachment.obj>