[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: ddraw/device.c: Fixed Thunk_IDirect3DDeviceImpl_1_AddViewport
From:       Stefan =?iso-8859-15?q?D=F6singer?= <stefandoesinger () gmx ! at>
Date:       2006-11-28 9:29:50
Message-ID: 200611281029.53213.stefandoesinger () gmx ! at
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Am Montag 27 November 2006 11:23 schrieb Christoph Bumiller:
> This is my first "patch" - I was trying an old game with wine
> (Starfleet Academy), which crashed when using D3D, and I got the
> following trace messages:
>
> trace:ddraw_thunk:Thunk_IDirect3DDeviceImpl_1_AddViewport
> (0x1b440c)->(0x1f44f0) thunking to IDirect3DDevice3 interface.
> trace:ddraw_thunk:Thunk_IDirect3DDeviceImpl_2_GetStats
> (0x1b4408)->(0x1f44f0) thunking to IDirect3DDevice3 interface.
> fixme:d3d7:IDirect3DDeviceImpl_3_GetStats (0x1b4408)->(0x1f44f0): Stub!
>
> This looks like Thunk_IDirect3DDeviceImpl_1_AddViewport called the
> GetStats thunk
> which called GetStats with the viewport as argument, where it set some
> fields to
> 0, which resulted in a segfault in IDirect3DViewportImpl_SetBackground
> because IDirect3DViewportImpl->ddraw was NULL.
Good catch, yeah, your patch is correct. I blamed native d3drm.dll of calling 
GetStats with a viewport, but it turns out that it was my fault in the 
end :-)

[Attachment #5 (application/pgp-signature)]



[prev in list] [next in list] [prev in thread] [next in thread]