diff options
| author | Minje Ahn <minje.ahn@samsung.com> | 2015-07-14 19:34:14 +0900 |
|---|---|---|
| committer | Minje Ahn <minje.ahn@samsung.com> | 2015-07-14 19:34:14 +0900 |
| commit | 8c59dd38edf2bfa6f7af1249e6df868cbc7b45cf (patch) | |
| tree | b50faa9b85f0ecd9a13d03b1d7893b5fed77ef70 | |
| parent | 51e31533b89929b6aed1a1727c0500cb01ce2d9f (diff) | |
| download | libmedia-thumbnail-8c59dd38edf2bfa6f7af1249e6df868cbc7b45cf.tar.gz libmedia-thumbnail-8c59dd38edf2bfa6f7af1249e6df868cbc7b45cf.tar.bz2 libmedia-thumbnail-8c59dd38edf2bfa6f7af1249e6df868cbc7b45cf.zip | |
Fix memory leak
Change-Id: I496932086d15704878ba0754a5b6d8ba6b7281df
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>
| -rwxr-xr-x | src/ipc/media-thumb-ipc.c | 5 | ||||
| -rwxr-xr-x | src/util/media-thumb-util.c | 6 |
2 files changed, 9 insertions, 2 deletions
diff --git a/src/ipc/media-thumb-ipc.c b/src/ipc/media-thumb-ipc.c index add4ba3..53f3ba2 100755 --- a/src/ipc/media-thumb-ipc.c +++ b/src/ipc/media-thumb-ipc.c @@ -581,6 +581,8 @@ static char* _media_thumb_get_default_path(uid_t uid) grpinfo = getgrnam("users"); if(grpinfo == NULL) { thumb_err("getgrnam(users) returns NULL !"); + if(result_psswd) + free(result_psswd); return NULL; } } @@ -682,8 +684,7 @@ int _media_thumb_process(thumbMsg *req_msg, thumbMsg *res_msg, uid_t uid) thumb_path[strlen(thumb_path)] = '\0'; } - if (g_file_test(thumb_path, - G_FILE_TEST_EXISTS | G_FILE_TEST_IS_REGULAR)) { + if (g_file_test(thumb_path, G_FILE_TEST_EXISTS | G_FILE_TEST_IS_REGULAR)) { thumb_warn("thumb path already exists in file system.. remove the existed file"); _media_thumb_remove_file(thumb_path); } diff --git a/src/util/media-thumb-util.c b/src/util/media-thumb-util.c index 485d13a..387a941 100755 --- a/src/util/media-thumb-util.c +++ b/src/util/media-thumb-util.c @@ -185,6 +185,8 @@ static char* _media_thumb_mmc_get_path(uid_t uid) grpinfo = getgrnam("users"); if(grpinfo == NULL) { thumb_err("getgrnam(users) returns NULL !"); + if(result_psswd) + free(result_psswd); return NULL; } } @@ -223,6 +225,8 @@ static char* _media_thumb_phone_get_path(uid_t uid) grpinfo = getgrnam("users"); if(grpinfo == NULL) { thumb_err("getgrnam(users) returns NULL !"); + if(result_psswd) + free(result_psswd); return NULL; } } @@ -283,6 +287,8 @@ int _media_thumb_get_hash_name(const char *file_full_path, ret_len = snprintf(thumb_hash_path, max_thumb_path - 1, "%s/.%s-%s.jpg", thumb_dir, file_ext, hash_name); + if(thumb_dir) + free(thumb_dir); if (ret_len < 0) { thumb_err("Error when snprintf"); return -1; |