From: JunsuChoi <jsuya.choi@samsung.com>
Date: Wed, 11 Mar 2020 12:42:55 +0000 (+0900)
Subject: vdrawable: Don't strdup for name data
X-Git-Tag: submit/tizen/20200322.220136~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F87%2F227887%2F1;p=platform%2Fcore%2Fuifw%2Flottie-player.git

vdrawable: Don't strdup for name data

The name information passed in is allocated and managed by the parser.
There is no need to duplicate the same data.

Change-Id: I9c488e654c01bdfe7c48388c29fad350cd261deb
---

diff --git a/src/vector/vdrawable.cpp b/src/vector/vdrawable.cpp
index 2102e82..7245d5f 100644
--- a/src/vector/vdrawable.cpp
+++ b/src/vector/vdrawable.cpp
@@ -34,7 +34,6 @@ VDrawable::~VDrawable()
             delete mStrokeInfo;
         }
     }
-    if (mName) free(mName);
 }
 
 void VDrawable::setType(VDrawable::Type type)
diff --git a/src/vector/vdrawable.h b/src/vector/vdrawable.h
index 56aefa7..13dfdf3 100644
--- a/src/vector/vdrawable.h
+++ b/src/vector/vdrawable.h
@@ -57,9 +57,9 @@ public:
     VRle rle();
     void setName(const char *name)
     {
-        if (name) mName = strdup(name);
+        mName = name;
     }
-    char* name() const { return mName; }
+    const char* name() const { return mName; }
 
 public:
     struct StrokeInfo {
@@ -83,7 +83,7 @@ public:
     FillRule                 mFillRule{FillRule::Winding};
     VDrawable::Type          mType{Type::Fill};
 
-    char                     *mName{nullptr};
+    const char              *mName{nullptr};
 };
 
 #endif  // VDRAWABLE_H