ddraw fix

Lionel Ulmer lionel.ulmer at free.fr
Sun Nov 10 07:55:33 CST 2002


> Probably application bug: it called DIB_DirectDrawSurface_Blt
> with .bottom > 36000, seems like DirectDraw sanitized the values.

Well, I would prefer the fix to be different... For example, it seems to be
valid to have negative top values as long as 'bottom - top' is valid (as it
seems that DDraw will clip the blit for you).

Problem is that if negative top is valid, then bottom being bigger than
dwHeight should be valid too. And your patch will thus prevent this case to
work.

Did you try to simply return an error message (instead of doing the assert)
to see if you see any graphical glitches ?

Anyway, I am installing Mingw + the DirectX SDKs to do some test apps on a
Win2K laptop. I will try this case to see if I manage to see what real
Windows does.

                        Lionel

-- 
		 Lionel Ulmer - http://www.bbrox.org/



More information about the wine-devel mailing list