BatchRenderer: fix potential memory leak

if the same LottieAnimation object is registered twice,
without calling deregisterAnimator() first,
we are leaking the BM objects tree and the pre-rendered frames

Change-Id: I07ac06b5f0b7ba9cacf7b0b883fc3cdb9782a349
Reviewed-by: Rebecca Worledge <rebecca.worledge@theqtcompany.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/src/imports/rasterrenderer/batchrenderer.cpp b/src/imports/rasterrenderer/batchrenderer.cpp
index e1ad3c5..0cd2cc6 100644
--- a/src/imports/rasterrenderer/batchrenderer.cpp
+++ b/src/imports/rasterrenderer/batchrenderer.cpp
@@ -95,7 +95,9 @@ void BatchRenderer::registerAnimator(LottieAnimation *animator)
 
     Entry *&entry = m_animData[animator];
     if (entry) {
-        delete (entry);
+        qDeleteAll(entry->frameCache);
+        delete entry->bmTreeBlueprint;
+        delete entry;
         entry = nullptr;
     }
     Q_ASSERT(entry == nullptr);