[vlc-commits] [Git][videolan/vlc][master] egl_gbm: set surface and its buffers to null pointer after destroy

[Steve Lhomme (@robUx4)]

Steve Lhomme pushed to branch master at VideoLAN / VLC


Commits:
b60480ff by Fatih Uzunoglu at 2024-10-20T07:50:32+00:00
egl_gbm: set surface and its buffers to null pointer after destroy

Currently, the surface and its buffers are destroyed but the buffer
objects are not set to null pointer. Because of this, it is tried to
call `gbm_surface_release_buffer()` with a new surface and buffer
of an already gone surface.

This causes crash on Wayland with Nvidia.

- - - - -


1 changed file:

- modules/video_output/opengl/egl_gbm.c


Changes:

=====================================
modules/video_output/opengl/egl_gbm.c
=====================================
@@ -172,6 +172,10 @@ static void Resize(vlc_gl_t* gl, unsigned width, unsigned height)
     if (sys->gbm.surface)
         gbm_surface_destroy(sys->gbm.surface);
 
+    sys->gbm.surface = NULL;
+    sys->gbm.bo_prev = NULL;
+    sys->gbm.bo_next = NULL;
+
     sys->egl.surface =  CreateSurface(gl, width, height);
 }
 



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/b60480ff6d4829f30d10fe6c601aaa64ca45572d

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/b60480ff6d4829f30d10fe6c601aaa64ca45572d
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance