Wine Traffic #288 For 19 Aug 2005

I am trying(once again) to get Empire Earth running with wine. After solving a few problems, I've run into a strange access violation crash. It looks like a buffer overrun, followed by a return to an invalid adress.

The crash messages look a little bit different every time, with illegal instructions / access violations or even a Segmentation Fault without starting winedbg at various addreses(Attached file crashes). On very rare occasions, this crash doesn't occur, and the game continues to load and crashes later in some ddraw function. Instead of crashing it complains about a corrupted heap:

err:heap:HEAP_ValidateInUseArena Heap 0x7fd80000: prev arena 0x7fe01640 is not prev for in-use 0x7fe01cb0

I've looked at a +heap trace, but I couldn't find anything usefull. (See ee-nocrash for a log). I've also attached a normal log without any special debug flags set(ee-normal.log.gz). I've added a few ERR statements for testing in some functions.

So my questions are:

• Am I right with my suspection that the problems are caused by a incorrect return?
• How can I get a disassembly of Low-Level Engine.?Deactivate_at_GERasterizer@@UAEJXZ or simmilar functions. I didn't find this symbol.

I was lucky with setting a breakpoint in the wine code. The crash happens in the DDraw implementation. The return from Main_DirectDraw_Release(ddraw_main.c:154) leads to a random adress. The call which leads to this is "HeapFree(GetProcessHeap(), 0, This);" in Main_DirectDrawSurface_Destroy, surface_main.c:154. If I comment out this call, Empire Earth continues loading and crashes more or less randomly at some later points.

I've edited the IDirectDrawSurfaceImpl structure and added a 2048 byte block at the beginning and the end. This makes the crashes reliable: With the HeapFree call, the ret jumps to NULL, and without the call Empire Earth crashes little later.

This looks like a really nasty heap corruption to me, and I'm afraid it's beyond my knowledge. Can anyone of the ddraw/d3d people help me?

This is interesting: Setting the +heap trace flag sets the bad address realiably to 0xaaaaaaaa (without my changes to DDraw). Does this say anything?

dlls/ntdll/heap.c:#define ARENA_FREE_FILLER 0xaa

I'd guess this is an address in an area that's actually gotten freed.

Using Wine 20050725 and Lotus Notes 6.51 when I move my mouse to the database tabs or over the replication window the mouse disappears while I move over the tab or replicated databases and then reappears when I leave the list. The databases are highlighted, but having the mouse completely disappear is a little annoying. I don't remember it doing this with 20041202.

I checked this on my XP machine and on that machine the cursor changes from an arrow to a hand pointer. So maybe the cursor just did not load correctly. So can a patch be done that if the cursor being selected is bogus that it stays the arrow pointer.

It should improve things over the original code that was there in cursoricon.c revision 1.9. That code first checked if there was only one cursor, then returned it, but if there was more than on cursor, it would choose one a 1bpp cursor. That scheme would miss out picking a cursor at all if there was two coloured cursors.

The new scheme makes sure that we always start with one cursor and improve over that.

I think the problem is that standard X cannot support coloured cursors. The Xcursor extension appears to, but we don't support that. Any volunteers?

I'm no longer able to run programs that require the native comctl32 and commctrl DLLs, since updating to today's CVS. Could it be related to these theme patches?

It seems that now, if you override comctl32 for any app (or globally), even WineCfg itself will error out with the error.

err:thunk:_loadthunk (commctrl.dll, Cctl1632_ThunkData16, comctl32.dll): Unable to load 'commctrl.dll', error 2
err:module:LdrInitializeThunk "comctl32.dll" failed to initialize, aborting
err:module:LdrInitializeThunk Main exe initialization for L"c:\\windows\\system32\\winecfg.exe" failed, status c0000142

The native commctrl.dll and comctl32.dll both exist in my windows/system directory - the same versions that always worked before. Overriding commctrl doesn't have any effect on this error.

You shouldn't need to use the native comctl32 any longer. Please update your dll overrides to "comctl32" = "builtin".

Wine's comctl32 implementation is almost complete. If there's bugs with the builtin comctl32 then please help us by reporting them rather than sweeping the problem under the carpet and using the native dll.

OK, there is a patch that is in CrossOver that has not yet been committed to WineHQ for some reason, that fixes this problem.

Could you please try the attached patch?

I fear that Wine doesn't go forward as much as it should for lack of testing :/

The patch worked great! At least, it appears to work a lot better than before.

There are still some graphical glitches, for example: If you switch server technology between yes and no when defining a site, the combo box doesn't completely reappear... but it's actually useable now!

Using CVS version I get following calls in the output very frequently:

000b:Call kernel32.97(408b8880) ret=40891d49
000b:Ret kernel32.97() retval=40428f60 ret=40891d49
000b:Call kernel32.98(408b8880) ret=40891e27
000b:Ret kernel32.98() retval=00000000 ret=40891e27

I want to exclude this stuff. Adding "97;98" to HKEY_CURRENT_USER/Software/Wine/debug/RelayExclude string value doesn't help me.

$ grep -Ir [^0-9]9[78][^0-9] dlls/kernel/*.spec | dlls/kernel/kernel32.spec: 97 stdcall -noname _EnterSysLevel(ptr) | dlls/kernel/kernel32.spec: 98 stdcall -noname _LeaveSysLevel(ptr)

Adding "_EnterSysLevel;_LeaveSysLevel;97;98" does not help either. Suggestions?

Try "kernel32.97;kernel32.98".

I am in the process of updating our "Acknowledgement" & "Who's Who" pages and if anyone here wants to be included and your currently not just send me a mail and ask for inclusion.

If you want inclusion on the "Acknowledgement" page let me know what areas you have worked on and you will be added in on my next patch.

If you believe you will be around for some time to come (a year) and want to be included to the "Who's Who" page let me know this as well. For the "Who's Who" I will need a short description of who you are, where you're from, when you started working on wine, and what areas you work on.

You can look at other entries for a good example..

• http://www.winehq.org/site/who
• http://www.winehq.org/site/acknowledgement

This is what I have thus far for my next update:

• ADD - Jonathan Ernst (Acknowledgement page)
• ADD - Krzysztof Foltman (Acknowledgement page)
• ADD - Phil Krylov (Acknowledgement page)
• Update - Gerald Pfeifer to (2005)
• Update - Michael Jung to (2005)
• Update - James Hawkins to (2005)
• Update - Vitaliy Margolen to (2005)
• Update - Mike Hearn to (2005)