Reviewed-by: Mark Janes <mark.a.ja...@intel.com> Chad Versace <chadvers...@chromium.org> writes:
> Don't dereference a null EGLDisplay. > > Fixes tests > dEQP-EGL.functional.negative_api.create_pbuffer_surface > dEQP-EGL.functional.negative_api.create_pixmap_surface > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=99038 > --- > src/egl/main/eglapi.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c > index 33d562042f..a349992c46 100644 > --- a/src/egl/main/eglapi.c > +++ b/src/egl/main/eglapi.c > @@ -856,7 +856,7 @@ _eglCreateWindowSurfaceCommon(_EGLDisplay *disp, > EGLConfig config, > RETURN_EGL_ERROR(disp, EGL_BAD_NATIVE_WINDOW, EGL_NO_SURFACE); > > #ifdef HAVE_SURFACELESS_PLATFORM > - if (disp->Platform == _EGL_PLATFORM_SURFACELESS) { > + if (disp && disp->Platform == _EGL_PLATFORM_SURFACELESS) { > /* From the EGL_MESA_platform_surfaceless spec (v1): > * > * eglCreatePlatformWindowSurface fails when called with a <display> > @@ -977,7 +977,7 @@ _eglCreatePixmapSurfaceCommon(_EGLDisplay *disp, > EGLConfig config, > EGLSurface ret; > > #if HAVE_SURFACELESS_PLATFORM > - if (disp->Platform == _EGL_PLATFORM_SURFACELESS) { > + if (disp && disp->Platform == _EGL_PLATFORM_SURFACELESS) { > /* From the EGL_MESA_platform_surfaceless spec (v1): > * > * [Like eglCreatePlatformWindowSurface,] > eglCreatePlatformPixmapSurface > -- > 2.11.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
