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-sta...@lists.freedesktop.org>
> ---
>  src/egl/main/eglapi.c | 4 ++++
>  1 file changed, 4 insertions(+)

Looks good to me.
Reviewed-by: Chad Versace <chad.vers...@linux.intel.com>

Ian, you gave the initial feedback, so you should commit.

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

Reply via email to