From 426c73a2b2717997bc3a77644ca101b328a879ae Mon Sep 17 00:00:00 2001
From: Ming Rui Zhang <mingrui.zhang@savoirfairelinux.com>
Date: Mon, 8 Feb 2021 12:51:42 -0500
Subject: [PATCH] directrenderer: prevent multiple potential video crashes

Avoid deleting the frame buffer while it's being
accessed by daemon or another DirectRenderer wrapper.

Change-Id: I8c905d0fe274c9b546a7b3438b8a0cbddf67f456
---
 src/directrenderer.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/directrenderer.cpp b/src/directrenderer.cpp
index f02d73b3..36960531 100644
--- a/src/directrenderer.cpp
+++ b/src/directrenderer.cpp
@@ -1,5 +1,5 @@
-/****************************************************************************
- *    Copyright (C) 2012-2021 Savoir-faire Linux Inc.                          *
+/****************************************************************************
+ *   Copyright (C) 2012-2021 Savoir-faire Linux Inc.                        *
  *   Author : Alexandre Lision <alexandre.lision@savoirfairelinux.com>      *
  *   Author : Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>      *
  *                                                                          *
@@ -89,6 +89,7 @@ Video::DirectRenderer::DirectRenderer(const QString& id, const QSize& res, bool
 /// Destructor
 Video::DirectRenderer::~DirectRenderer()
 {
+    QMutexLocker lk(mutex());
     stopRendering();
 
     d_ptr.reset();
-- 
GitLab