From: Younghwan Ahn Date: Tue, 8 Jan 2013 12:25:35 +0000 (+0900) Subject: fix prevent defects X-Git-Tag: submit/tizen_2.1/20130424.232047~6^2~18 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=eb74575c2a627edfb8efef1e967fa196af353a67;p=platform%2Fcore%2Fmultimedia%2Flibmm-player.git fix prevent defects Change-Id: Iafb18e9f6a472f72b60f56e86593da85541404d2 --- 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; }