[Bug 17518] winmm/mci tests fail with +heap enabled

http://bugs.winehq.org/show_bug.cgi?id=17518

--- Comment #3 from Austin English   2009-03-02 18:33:59 ---
(In reply to comment #1)
> Created an attachment (id=19744)

--> (http://bugs.winehq.org/attachment.cgi?id=19744) [details]
> Zero lpwstrRet, before trying to do anything with it.
>
> Austin, 
> could you try this patch?
> Another solution is to check the return value ( if (lpwstrRet && ret==0) ), but
> that assumes that in failure case there is never written anything useful to the
> lpwstrRet.

Works fine. Problem was that I was piping the output away and checking the exit
status. The crash is gone, but a new 'failure' is introduced:

mci.c:56: Test succeeded inside todo block: mciSendString(close all notify,
buf, sizeof(buf) , NULL) changed output buffer:

Remove that todo and the test passes with/without +heap :-).