From d6d4839f1ac77320644b12a8060f3f8c23c2ab26 Mon Sep 17 00:00:00 2001 From: Hermet Park Date: Thu, 11 Jul 2024 00:56:13 +0900 Subject: [PATCH] lottie: ++memory access stability Internal matrix can be null. This update prevents such cases by using a safer API. --- src/loaders/lottie/tvgLottieBuilder.cpp | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/loaders/lottie/tvgLottieBuilder.cpp b/src/loaders/lottie/tvgLottieBuilder.cpp index 03a211d1..493da9ba 100644 --- a/src/loaders/lottie/tvgLottieBuilder.cpp +++ b/src/loaders/lottie/tvgLottieBuilder.cpp @@ -1125,30 +1125,28 @@ static void _updateText(LottieLayer* layer, float frameNo) if (start > end) std::swap(start, end); if (idx < start || idx >= end) continue; - auto matrix = PP(shape.get())->transform(); + auto matrix = shape->transform(); shape->opacity((*s)->style.opacity(frameNo)); auto color = (*s)->style.fillColor(frameNo); shape->fill(color.rgb[0], color.rgb[1], color.rgb[2], (*s)->style.fillOpacity(frameNo)); - rotate(matrix, (*s)->style.rotation(frameNo)); + rotate(&matrix, (*s)->style.rotation(frameNo)); auto glyphScale = (*s)->style.scale(frameNo) * 0.01f; - tvg::scale(matrix, glyphScale.x, glyphScale.y); + tvg::scale(&matrix, glyphScale.x, glyphScale.y); auto position = (*s)->style.position(frameNo); - translate(matrix, position.x, position.y); + translate(&matrix, position.x, position.y); - shape->transform(*matrix); + shape->transform(matrix); if (doc.stroke.render) { auto strokeColor = (*s)->style.strokeColor(frameNo); - shape->strokeWidth((*s)->style.strokeWidth(frameNo) / scale); shape->strokeFill(strokeColor.rgb[0], strokeColor.rgb[1], strokeColor.rgb[2], (*s)->style.strokeOpacity(frameNo)); } - cursor.x += (*s)->style.letterSpacing(frameNo); }