From: Hermet Park <chuneon.park@samsung.com>
Date: Tue, 7 Sep 2021 09:54:42 +0000 (+0900)
Subject: tvg_saver: fix a wrong logic by mistake.
X-Git-Tag: accepted/tizen/unified/20210910.004734~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ac18c7ce17cb982e40fba4da7bcacec8dcdb7534;p=platform%2Fcore%2Fgraphics%2Ftizenvg.git

tvg_saver: fix a wrong logic by mistake.

an incorrect logic was introduced by mistake.

that breaks the tvg memory chunks.

now recover it.

@Issues: https://github.com/Samsung/thorvg/issues/774
---

diff --git a/src/examples/images/test.tvg b/src/examples/images/test.tvg
index 4c628e3..78bb48f 100644
Binary files a/src/examples/images/test.tvg and b/src/examples/images/test.tvg differ
diff --git a/src/savers/tvg/tvgTvgSaver.cpp b/src/savers/tvg/tvgTvgSaver.cpp
index 21d3f12..bba2b9b 100644
--- a/src/savers/tvg/tvgTvgSaver.cpp
+++ b/src/savers/tvg/tvgTvgSaver.cpp
@@ -542,9 +542,9 @@ TvgBinCounter TvgSaver::serializeShape(const Shape* shape, const Matrix* pTransf
     if (pTransform) transform = _multiply(pTransform, &transform);
 
     //fill rule
-    if (auto flag = static_cast<TvgBinFlag>(shape->fillRule()))
+    if (auto flag = static_cast<TvgBinFlag>(shape->fillRule())) {
         cnt = writeTagProperty(TVG_TAG_SHAPE_FILLRULE, SIZE(TvgBinFlag), &flag);
-
+    }
 
     bool preTransform = true;
 
@@ -602,9 +602,9 @@ TvgBinCounter TvgSaver::serializePicture(const Picture* picture, const Matrix* p
             reserveCount();
             auto cnt = serializeChildren(it, &transform, true) + serializePaint(picture, pTransform);
             writeReservedCount(cnt);
+            delete(it);
+            return SERIAL_DONE(cnt);
         }
-        delete(it);
-        return SERIAL_DONE(cnt);
     }
     delete(it);