Regression with 'explorer' starting automatically by 'Process Explorer'

[James Liggett]

On Wed, 2006-08-23 at 20:39 +0200, Paul Vriens wrote:
> Hi,
> 
> normally when I start Process Explorer it automatically launches the
> 'explorer' process.
> 
> With the latest git 'explorer' crashes. Regression testing showed that
> this is most likely due to the XEmbed patch:
> 
> http://source.winehq.org/git/?p=wine.git;a=commit;h=60a97505a63e5102f239d0d4c2a2ea7fd775c34e
> 
Yep, that's the one. I'm looking into it now. BTW, thanks for the debug
output. 

Here's the most interesting part:

> 000b:trace:x11drv:X11DRV_set_window_pos mapping win 0x10052
> 000b:trace:x11drv:systray_dock_window Docking tray icon 0x10052
> 000b:trace:x11drv:X11DRV_SetWindowPos hwnd 0x10052, after (nil), swp 1025,769 0x0 flags 00000005
> 000b:err:syslevel:_CheckNotSysLevel Holding lock 0x603ef040 level 2
> 000b:trace:seh:raise_exception code=80000003 flags=0 addr=0x77a5fc40
> 000b:trace:seh:raise_exception  eax=00000044 ebx=77a83f64 ecx=602daa80 edx=00000044 esi=7ffdc044 edi=00000000
> 000b:trace:seh:raise_exception  ebp=0034e2b8 esp=0034e290 cs=0073 ds=007b es=007b fs=003b gs=0033 flags=00000206
> 000b:trace:seh:call_stack_handlers calling handler at 0x77a05ec8 code=80000003 flags=0
> wine: Unhandled exception 0x80000003 at address 0x77a5fc40 (thread 000b), starting debugger...
What's happening here is that we need to make sure the window isn't
mapped when we dock it, so the patch moves it offscreen. Looks like the
problem is with SetWindowPos. Seems like we're holding a lock and trying
to acquire another one, and _CheckNotSysLevel doesn't like that and thus
it throws a breakpoint. I'll check it out and see if I can get a fix.

Thanks,
James Liggett