[Mesa-stable] [Mesa-dev] [PATCH] Check for NULL native_window into eglCreateWindowSurface()
Chad Versace
chad.versace at intel.com
Tue Jun 3 15:42:32 PDT 2014
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.
More information about the mesa-stable
mailing list