Add mutex guards for AbstractDeclarative::updateWindowParameters

Adds a mutex to guard that pointer to controller is not deleted while
doing updateWindowParameters. The render thread doesn't have control to
the node mutex, so that won't apply since it may be deleted while requesting
a mutex.

Change-Id: I8897954415c1bd7ab61ef99b420c9ac2aaa89411
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

1 files changed, 1 insertions, 0 deletions

diff --git a/src/datavisualizationqml2/declarativescatter.cpp b/src/datavisualizationqml2/declarativescatter.cpp
index 0b7efdb4..29e56a79 100644
--- a/src/datavisualizationqml2/declarativescatter.cpp
+++ b/src/datavisualizationqml2/declarativescatter.cpp
@@ -49,6 +49,7 @@ DeclarativeScatter::DeclarativeScatter(QQuickItem *parent)
 DeclarativeScatter::~DeclarativeScatter()
 {
     QMutexLocker locker(m_nodeMutex.data());
+    const QMutexLocker locker2(mutex());
     delete m_scatterController;
 }