[Mesa-stable] [Mesa-dev] [PATCH] Check for NULL native_window into eglCreateWindowSurface()

Ian Romanick idr at freedesktop.org
Tue Jun 3 17:23:38 PDT 2014


On 06/03/2014 03:47 PM, Chad Versace wrote:
> On Tue, Jun 03, 2014 at 03:42:32PM -0700, Chad Versace wrote:
>> On Tue, Jun 03, 2014 at 02:00:13PM -0700, Sinclair Yeh wrote:
>>> v2:
>>> Incorporated feedback from idr - moved the check to a higher level
>>> function.
>>>
>>> v1:
>>> We have customers using NULL as a way to test the robustness of
>>> the API.  Without this check, EGL will segfault trying to
>>> dereference dri2_surf->wl_win->private because wl_win is
>>> NULL.
>>>
>>> This fix adds a check and sets EGL_BAD_NATIVE_WINDOW
>>>
>>> Cc: "10.1 10.2" <mesa-stable at lists.freedesktop.org>
>>> ---
>>>  src/egl/main/eglapi.c | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>
>> Looks good to me.
>> Reviewed-by: Chad Versace <chad.versace at linux.intel.com>
>>
>> Ian, you gave the initial feedback, so you should commit.
> 
> Sinclair, one more thing. In the future, please prefix the commit
> subject with Mesa component name touched by the patch. For example:
> 
>     [PATCH] egl: Check for NULL native_window into eglCreateWindowSurface()
> 
> Ian, if this patch gets your r-b, please add the prefix before committing.

Roger that.  I'm also going to adjust the commmit message slightly.

> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-stable mailing list