[PATCH] ddraw/tests: Disable a test that causes heap corruption on some Nvidia.

Henri Verbeet hverbeet at gmail.com
Mon Jan 6 07:58:33 CST 2020


On Sat, 4 Jan 2020 at 00:20, Stefan Dösinger <stefandoesinger at gmail.com> wrote:
> > Am 03.01.2020 um 13:00 schrieb Henri Verbeet <hverbeet at gmail.com>:
> > On Fri, 3 Jan 2020 at 02:25, Stefan Dösinger <stefan at codeweavers.com> wrote:
> >> +    if (0)
> > Should that then be "if (!ddraw_is_nvidia(ddraw))"?
> I can make it so if you want, but I wouldn't put my hand in a fire that it really is an Nvidia issue and not a random corruption that can affect other systems too, but doesn't in my computer zoo by luck. The part that Deus Ex cares about is that the surface destruction does not crash.

Yeah, understood. I'm inclined to go for the narrower option and
expand it if that turns out to be necessary, instead of disabling the
test right away.



More information about the wine-devel mailing list