[user32] Allocate buffer on heap (Bug 17076) (Try3)

Paul Vriens paul.vriens.wine at gmail.com
Fri Jan 23 11:16:29 CST 2009


Hi,

- Try2 with the needed HeapFree
- Try3 gets rid of local_buffer altogether and so the goto is not needed as well

This fixes a regression (commit 7804129e68a0b25549b729854c15706827140b46).

Some called functions assume that buffer is always on heap. That isn't the case
anymore.

I'm not sure if some logic is required to limit the number of HeapFree / 
HeapAlloc in the 'if (res)' branch. I ran the tests and no extra HeapAlloc was 
needed in that branch.

Changelog
     Allocate buffer on heap

-- 
Cheers,

Paul.





-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Allocate-buffer-on-heap.patch
Type: text/x-diff
Size: 2665 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20090123/790a097e/attachment-0001.patch 


More information about the wine-patches mailing list