wined3d: Better validation of BltFast destination

Michael Stefaniuc mstefani at redhat.com
Wed Jun 23 04:48:37 CDT 2010


Iain Arnell wrote:
> On Wed, Jun 23, 2010 at 10:47 AM, Stefan Dösinger
> <stefandoesinger at gmx.at> wrote:
>> Am Mittwoch 23 Juni 2010 10:40:30 schrieb Stefan Dösinger:
>>> You're catching only the GDI surface case here since opengl surfaces don't
>>>  use IWineD3DBaseSurfaceImpl_BltFast. I recommend to put this check into
>>>  ddraw, like in your previous attempts(but keep checking for < 0, don't do
>>>  any bit testing magic like in your old patches)
> 
> I'd hope that the opengl implementation also has some checking of its own.
> 
> I did the nasty bit twiddling to avoid making any assumptions about
> equivalence of types (is it always safe to assume that DWORD can be
> cast as a LONG?). And gcc optimizes dstx < 0 away as "comparison of
> unsigned expression < 0 is always false".
In the Windows world:
- DWORD == unsigned int32
- LONG  == signed int32

bye
	michael



More information about the wine-devel mailing list