Dylan Smith <dylan.ah.sm...@gmail.com> writes:

> @@ -3123,6 +3123,8 @@ static HRESULT CreateSurface(IDirectDrawImpl *ddraw, 
> DDSURFACEDESC2 *DDSD,
>      memset(&desc2, 0, sizeof(desc2));
>      desc2.dwSize = sizeof(desc2);   /* For the struct copy */
>      DD_STRUCT_COPY_BYSIZE(&desc2, DDSD);
> +    if (DDSD->dwSize == sizeof(DDSURFACEDESC))
> +        CLEAR_STRUCT_END_PADDING((DDSURFACEDESC*)&desc2, ddsCaps);

I'd suggest to rethink the whole thing. We don't really need to set the
same structure fields 4 times.

-- 
Alexandre Julliard
julli...@winehq.org


Reply via email to