On 18 May 2017 at 08:21, Tapani Pälli <tapani.pa...@intel.com> wrote: > Function droid_swap_buffers may get called without dri2_surf->buffer set, > in these cases we don't have a back buffer set either. Patch fixes segfault > seen with 3DMark that uses android.opengl.GLSurfaceView for rendering it's UI. > > backtrace: > #00 pc 00013f88 /system/lib/egl/libGLES_mesa.so (droid_swap_buffers+104) > #01 pc 000117b2 /system/lib/egl/libGLES_mesa.so (dri2_swap_buffers+50) > #02 pc 000058b2 /system/lib/egl/libGLES_mesa.so (eglSwapBuffers+386) > #03 pc 00011329 /system/lib/libEGL.so (eglSwapBuffersWithDamageKHR+553) > #04 pc 000118e7 /system/lib/libEGL.so (eglSwapBuffers+55) > #05 pc 000754dc /system/lib/libandroid_runtime.so > > Note, this is v1 as v2 caused dEQP regressions. > > Fixes: 2acc69d ("EGL/Android: Add EGL_EXT_buffer_age extension") > Signed-off-by: Tapani Pälli <tapani.pa...@intel.com> > --- > src/egl/drivers/dri2/platform_android.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/egl/drivers/dri2/platform_android.c > b/src/egl/drivers/dri2/platform_android.c > index 49cbeb4..88f6af8 100644 > --- a/src/egl/drivers/dri2/platform_android.c > +++ b/src/egl/drivers/dri2/platform_android.c > @@ -628,7 +628,9 @@ droid_swap_buffers(_EGLDriver *drv, _EGLDisplay *disp, > _EGLSurface *draw) > if (dri2_surf->color_buffers[i].age > 0) > dri2_surf->color_buffers[i].age++; > } > - dri2_surf->back->age = 1; > + > + if (dri2_surf->back) > + dri2_surf->back->age = 1;
Doesn't seem quite right to use get_back_bo() elsewhere and a NULL check here. With a inline comment vaguely like "XXX: we don't use get_back_bo() since it causes regressions in $app/test." Acked-by: Emil Velikov <emil.veli...@collabora.com> -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev