From af4d463cc8f4f404ffe406f77a01fac5b4be49b7 Mon Sep 17 00:00:00 2001 From: Donald Carr Date: Tue, 13 Jun 2017 11:19:30 -0700 Subject: Avoid double deletion of mBuffer Both WaylandBrcmBuffer and WaylandBuffer were calling wl_buffer_destroy(mBuffer) without the derived class nulling the pointer on deletion Change-Id: I66a553a3d11a1ef90bbfe6ad0fb49f40fe1a0bdb Reviewed-by: Johan Helsing --- src/hardwareintegration/client/brcm-egl/qwaylandbrcmeglwindow.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/hardwareintegration/client/brcm-egl/qwaylandbrcmeglwindow.cpp b/src/hardwareintegration/client/brcm-egl/qwaylandbrcmeglwindow.cpp index 9842f61c9..f2709983e 100644 --- a/src/hardwareintegration/client/brcm-egl/qwaylandbrcmeglwindow.cpp +++ b/src/hardwareintegration/client/brcm-egl/qwaylandbrcmeglwindow.cpp @@ -91,6 +91,7 @@ public: { wl_array_release(&m_array); wl_buffer_destroy(mBuffer); + mBuffer = nullptr; } QSize size() const { return m_size; } -- cgit v1.2.3