diff options
author | Younghwan Ahn <younghwan_.an@samsung.com> | 2013-01-08 21:25:35 +0900 |
---|---|---|
committer | Younghwan Ahn <younghwan_.an@samsung.com> | 2013-01-08 21:25:35 +0900 |
commit | eb74575c2a627edfb8efef1e967fa196af353a67 (patch) | |
tree | 03f96d09239c9658eda0e85eb61d328735bec430 | |
parent | ca36acc757328ac23dc816111e5e325fbbcef8f5 (diff) | |
download | libmm-player-eb74575c2a627edfb8efef1e967fa196af353a67.tar.gz libmm-player-eb74575c2a627edfb8efef1e967fa196af353a67.tar.bz2 libmm-player-eb74575c2a627edfb8efef1e967fa196af353a67.zip |
fix prevent defects
Change-Id: Iafb18e9f6a472f72b60f56e86593da85541404d2
-rwxr-xr-x | src/mm_player_priv.c | 2 | ||||
-rwxr-xr-x | src/mm_player_utils.c | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c index e7d17c5..bcae32d 100755 --- a/src/mm_player_priv.c +++ b/src/mm_player_priv.c @@ -1943,7 +1943,7 @@ __mmplayer_gst_rtp_no_more_pads (GstElement *element, gpointer data) // @ debug_log("it seems pad caps is directely used for autoplugging. removing fakesink now\n"); if ( ! __mmplayer_gst_remove_fakesink( player, - &player->pipeline->mainbin[MMPLAYER_M_SRC_FAKESINK]) ); + &player->pipeline->mainbin[MMPLAYER_M_SRC_FAKESINK]) ) { /* NOTE : __mmplayer_pipeline_complete() can be called several time. because * signaling mechanism ( pad-added, no-more-pad, new-decoded-pad ) from various diff --git a/src/mm_player_utils.c b/src/mm_player_utils.c index fd2c18e..261d7d4 100755 --- a/src/mm_player_utils.c +++ b/src/mm_player_utils.c @@ -81,7 +81,10 @@ bool util_remove_file_backup(const char *backup_path) int res = access(backup_path, R_OK); if (!res) - remove(backup_path); + { + if (remove(backup_path) == -1) + return FALSE; + } return TRUE; } |