enlightenment/efl
enlightenment
/
efl
8
7
Fork 14
Code Issues 13 Pull Requests 3 Projects Releases Wiki Activity

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
This commit is contained in:
Rajesh PS 2014-04-03 19:16:14 +09:00 committed by Carsten Haitzler (Rasterman)
parent 73b16ef210
commit 69bdc47081
3 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
legacy/elementary/AUTHORS
View File

@ -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>

1
legacy/elementary/src/lib/elm_authors.h
View File

@ -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.

3
legacy/elementary/src/lib/elm_map.c
View File

@ -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);