msvcrt: Don't use default process heap

Piotr Caban piotr at codeweavers.com
Tue Apr 22 07:38:47 CDT 2014


This patch fixes following bugs:
http://bugs.winehq.org/show_bug.cgi?id=34698
http://bugs.winehq.org/show_bug.cgi?id=36043
in both cases the applications are broken. Battlefield 1942 accesses 
already freed memory and expects that it was not overwritten. QQGame2011 
uses LocalAlloc incorrectly and overwrites data on heap.

I'm not sure how we should deal with these bugs. I've decided to write 
this patch because native msvcrt is not using default process heap, it 
also happens to hide/fix the bugs.

---
  dlls/msvcrt/heap.c   | 40 ++++++++++++++++++++++++++--------------
  dlls/msvcrt/main.c   |  8 +++++++-
  dlls/msvcrt/msvcrt.h |  2 ++
  3 files changed, 35 insertions(+), 15 deletions(-)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-msvcrt-Don-t-use-defalut-process-heap.txt
Type: text/x-patch
Size: 6242 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20140422/aad7e275/attachment.bin>


More information about the wine-patches mailing list