diff options
| author | Radoslaw Czerski <r.czerski@samsung.com> | 2016-04-12 15:15:25 +0200 |
|---|---|---|
| committer | Radoslaw Czerski <r.czerski@samsung.com> | 2016-04-12 15:15:25 +0200 |
| commit | da256e674871c24b818c81c89a1bfb53dcba0d62 (patch) | |
| tree | 976c05ba3f67f8bce49dbaa3c3f73daa16c5afab | |
| parent | 357a4c7c5323ad1ef926e7f6a747eee71c3ec234 (diff) | |
| download | indicator-win-da256e674871c24b818c81c89a1bfb53dcba0d62.tar.gz indicator-win-da256e674871c24b818c81c89a1bfb53dcba0d62.tar.bz2 indicator-win-da256e674871c24b818c81c89a1bfb53dcba0d62.zip | |
modules/information/mp3:Handle leak fixed.
WGID 6978
Change-Id: I7e169fabed0d4b6fce11fcc3972220c7cfd4d5bb
Signed-off-by: Radoslaw Czerski <r.czerski@samsung.com>
| -rw-r--r-- | src/modules/information/mp3_playing.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/modules/information/mp3_playing.c b/src/modules/information/mp3_playing.c index fe059a6..586b0f2 100644 --- a/src/modules/information/mp3_playing.c +++ b/src/modules/information/mp3_playing.c @@ -101,9 +101,17 @@ static void show_mp_icon(void* data) FILE* fp = fopen(util_get_data_file_path(MUSIC_STATUS_FILE_PATH), "r"); char line[MAX_NAM_LEN+1]; - retm_if(data == NULL, "Invalid parameter!"); + if (data == NULL) { + _E("Invalid parameter!"); + fclose(fp); + return; + } - retm_if(fp == NULL, "Invalid file path !!"); + if (fp == NULL) { + _E("Invalid file path!!"); + fclose(fp); + return; + } if(icon_get_update_flag() == 0) { updated_while_lcd_off = 1; |