Re: [PATCH 1/4] ddraw: Don't write out if a ddraw1 surface of size 0 is requested.

[Henri Verbeet]

On 6 January 2017 at 08:07, Alex Henrie  wrote:
> Fixes https://bugs.winehq.org/show_bug.cgi?id=42159

This isn't a regression, right?
> @@ -2975,10 +2975,7 @@ static HRESULT WINAPI ddraw1_CreateSurface(IDirectDraw *iface,
>      hr = ddraw_surface_create(ddraw, &surface_desc2, &impl, outer_unknown, 1);
>      wined3d_mutex_unlock();
>      if (FAILED(hr))
> - {
> -
>    *surface = NULL;
>    return hr;
> - }

This does a bit more than the commit message says it does, this
doesn't write to *surface on any error from ddraw_surface_create().
That may very well be correct, but that's not what the test tests. The
test only tests that CreateSurface() doesn't crash on error with an
invalid "surface" pointer, but that could also be because of e.g. an
exception handler. If you want to prove that it doesn't (try to) write
to the "surface" pointer, you should pass a valid pointer to
CreateSurface() and test that it isn't modified after the call.