[Mesa-stable] [Mesa-dev] [PATCH 4/8] egl: rework input validation order in _eglCreateWindowSurfaceCommon

[Emil Velikov]

On 8 August 2017 at 08:21, Tapani Pälli <tapani.palli at intel.com> wrote:
>
>
> On 08/05/2017 02:25 AM, Emil Velikov wrote:
>>
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> As mentioned in previous commit the negative tests in dEQP expect the
>> arguments to be evaluated in particular order.
>>
>> Namely - first the dpy, then the config, followed by the surface/window.
>>
>> Move the check further down or executing the test below will produce
>> the following error.
>>
>>     dEQP-EGL.functional.negative_api.create_pbuffer_surface
>>
>>
>>     <Section Name="Test2" Description="EGL_BAD_CONFIG is generated if
>> config is not an EGL frame buffer configuration">
>>        <Text>eglCreateWindowSurface(0x9bfff0f150, 0xffffffffffffffff,
>> 0x0000000000000000, { EGL_NONE });</Text>
>>        <Text>// 0x0000000000000000 returned</Text>
>>        <Text>// ERROR expected: EGL_BAD_CONFIG, Got:
>> EGL_BAD_NATIVE_WINDOW</Text>
>>     </Section>
>>
>> Cc: <mesa-stable at lists.freedesktop.org>
>> Cc: Mark Janes <mark.a.janes at intel.com>
>> Cc: Chad Versace <chadversary at chromium.org>
>> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
>> ---
>> Mark,
>>
>> IMHO the CI does the impossible and passes the test. Perhaps it's worth
>> looking into how/why it does so - I don't know.
>
>
> For me the above test is passing fine on x11 (x11_egl).
>
Barring local changes i cannot see how that's possible.

The test itself is as below, while Mesa does the window (NULL) check first.

expectNoSurface(eglCreateWindowSurface(display, (EGLConfig)-1,
DE_NULL, s_emptyAttribList));
expectError(EGL_BAD_CONFIG);

Then again, I've spotted a few extra bits that we're not handling
correctly. More patches coming shortly.

-Emil