On 4 August 2016 at 03:13, Nicolas Boichat <[email protected]> wrote: > Thanks! See also related series here, which fixes the other platforms: > https://lists.freedesktop.org/archives/mesa-dev/2016-August/125147.html > > On Sun, Jul 31, 2016 at 9:44 PM, Jan Ziak <[email protected]> wrote: >> Found with valgrind: >> >> ==4841== Invalid read of size 4 >> ==4841== at 0x56BDC80: dri2_initialize (egl_dri2.c:783) >> ==4841== by 0x56BAFE5: _eglMatchAndInitialize (egldriver.c:261) >> ==4841== by 0x56BB15E: _eglMatchDriver (egldriver.c:295) >> ==4841== by 0x56B58C9: eglInitialize (eglapi.c:480) >> ==4841== by 0x4F537DC: _glfwInitEGL (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F4BEFB: _glfwPlatformInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F46F40: glfwInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x402E59: main >> ==4841== Address 0x6a05824 is 148 bytes inside a block of size 480 free'd >> ==4841== at 0x4C2B680: free (in >> /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) >> ==4841== by 0x56C2AAE: dri2_initialize_x11_swrast (platform_x11.c:1233) >> ==4841== by 0x56C2AAE: dri2_initialize_x11 (platform_x11.c:1493) >> ==4841== by 0x56BDCEB: dri2_initialize (egl_dri2.c:805) >> ==4841== by 0x56BAFAF: _eglMatchAndInitialize (egldriver.c:261) >> ==4841== by 0x56BB0C9: _eglMatchDriver (egldriver.c:292) >> ==4841== by 0x56B58C9: eglInitialize (eglapi.c:480) >> ==4841== by 0x4F537DC: _glfwInitEGL (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F4BEFB: _glfwPlatformInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F46F40: glfwInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x402E59: main >> ==4841== Block was alloc'd at >> ==4841== at 0x4C2A868: calloc (in >> /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) >> ==4841== by 0x56C2A47: dri2_initialize_x11_swrast (platform_x11.c:1171) >> ==4841== by 0x56C2A47: dri2_initialize_x11 (platform_x11.c:1493) >> ==4841== by 0x56BDCEB: dri2_initialize (egl_dri2.c:805) >> ==4841== by 0x56BAFAF: _eglMatchAndInitialize (egldriver.c:261) >> ==4841== by 0x56BB0C9: _eglMatchDriver (egldriver.c:292) >> ==4841== by 0x56B58C9: eglInitialize (eglapi.c:480) >> ==4841== by 0x4F537DC: _glfwInitEGL (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F4BEFB: _glfwPlatformInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x4F46F40: glfwInit (in /usr/lib64/libglfw.so.3.2) >> ==4841== by 0x402E59: main >> >> Signed-off-by: Jan Ziak (http://atom-symbol.net) <[email protected]> > > Fixes: 9ee683f877 (egl/dri2: Add reference count for dri2_egl_display) > Cc: "12.0" <[email protected]> > Reviewed-by: Nicolas Boichat <[email protected]> > This and there remaining DriverData patches are in master now.
Jan, I believe you're ok/don't mind Patchwork. Can you please check/update things if needed. Patchwork seems unhappy whenever I look up for you. Thanks gents, Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
