svg_loader: memleak prevention 97/288897/1
authorMira Grudzinska <m.grudzinska@samsung.com>
Tue, 18 Jan 2022 17:13:28 +0000 (18:13 +0100)
committerMichal Szczecinski <mihashco89@gmail.com>
Fri, 24 Feb 2023 07:09:34 +0000 (08:09 +0100)
If image href set more than once, the memory was not freed.

Change-Id: Iff8edacb94d4da93dfbd81c7505c535ea675d28c

src/loaders/svg/tvgSvgLoader.cpp

index 9201a2d..eb7a0e0 100644 (file)
@@ -1628,6 +1628,7 @@ static bool _attrParseImageNode(void* data, const char* key, const char* value)
     }
 
     if (!strcmp(key, "href") || !strcmp(key, "xlink:href")) {
+        if (image->href && value) free(image->href);
         image->href = _idFromHref(value);
     } else if (!strcmp(key, "id")) {
         if (node->id && value) free(node->id);