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

[wine-bugs at winehq.org]

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





--- Comment #3 from Austin English <austinenglish at gmail.com>  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 :-).


-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.