elm_map: Fixed a leak when ecore_file_download_full fails

Summary:
The url variable used for ecore_file_download_full api was not being freed when the api failed

@fix

Test Plan: elementary_test->map

Reviewers: seoz, daniel, raster

Reviewed By: raster

CC: govi
Differential Revision: https://phab.enlightenment.org/D692

diff --git a/AUTHORS b/AUTHORS
index 9bd2402..d291cb8 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -152,3 +152,4 @@ suxia li <suxia.li@samsung.com>
 yan.wang <yan.wang@linux.intel.com>
 Anand <anand.km@samsung.com>
 Subhransu Sekhar Mohanty <sub.mohanty@samsung.com>
+Rajesh P S <rajeshps@samsung.com>

diff --git a/src/lib/elm_authors.h b/src/lib/elm_authors.h
index f17abaa..a7aee5f 100644 (file)
--- a/src/lib/elm_authors.h
+++ b/src/lib/elm_authors.h
@@ -154,6 +154,7 @@
  * @author yan.wang <yan.wang@@linux.intel.com>
  * @author Anand <anand.km@@samsung.com>
  * @author Subhransu Sekhar Mohanty <sub.mohanty@@samsung.com>
+ * @author Rajesh P S <rajeshps@@samsung.com>
  *
  * Please contact <enlightenment-devel@lists.sourceforge.net> to get in
  * contact with the developers and maintainers.

diff --git a/src/lib/elm_map.c b/src/lib/elm_map.c
index 59ade2d..d0fe739 100644 (file)
--- a/src/lib/elm_map.c
+++ b/src/lib/elm_map.c
@@ -3121,6 +3121,7 @@ _name_request(const Evas_Object *obj,
         free(name->fname);
         free(name);
         free(fname);
+        free(url);
         return NULL;
      }
    INF("Name requested from %s to %s", url, name->fname);
@@ -3174,6 +3175,7 @@ _name_list_request(const Evas_Object *obj,
         ERR("Can't request Name from %s to %s", url, name_list->fname);
         free(name_list->fname);
         free(name_list);
+        free(url);
         free(fname);
         return NULL;
      }
@@ -4393,6 +4395,7 @@ _elm_map_route_add(Eo *obj, Elm_Map_Data *sd, Elm_Map_Route_Type type, Elm_Map_R
         ERR("Can't request Route from %s to %s", url, route->fname);
         free(route->fname);
         free(route);
+        free(url);
         return ret;
      }
    INF("Route requested from %s to %s", url, route->fname);