From eb7bca1edcdd229b92c5d013847492a882194e1f Mon Sep 17 00:00:00 2001 From: Andy Shaw Date: Tue, 23 Feb 2021 09:28:05 +0100 Subject: Only create the GLXYSeriesData if there is no existing data This ensures that there is no memory leak due to unnecessary creation of the GLXYSeriesData when setting the series data. Change-Id: Iec7966bbbf53e595ead780462c89bb3e1205854e Reviewed-by: Miikka Heikkinen (cherry picked from commit eaf63689403df7f122591795d80ef126b8fb5eaa) Reviewed-by: Qt Cherry-pick Bot --- src/chartsqml2/declarativeopenglrendernode.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/chartsqml2/declarativeopenglrendernode.cpp b/src/chartsqml2/declarativeopenglrendernode.cpp index 6d2baf20..013cbe4f 100644 --- a/src/chartsqml2/declarativeopenglrendernode.cpp +++ b/src/chartsqml2/declarativeopenglrendernode.cpp @@ -229,7 +229,8 @@ void DeclarativeOpenGLRenderNode::setSeriesData(bool mapDirty, const GLXYDataMap GLXYSeriesData *data = oldMap.take(i.key()); const GLXYSeriesData *newData = i.value(); if (!data || newData->dirty) { - data = new GLXYSeriesData; + if (!data) + data = new GLXYSeriesData; *data = *newData; } m_xyDataMap.insert(i.key(), data); -- cgit v1.2.3