[0.6.164] Apply tizen coding rule
[platform/core/multimedia/libmm-player.git] / src / mm_player_priv.c
index 86c8c29..ee65161 100644 (file)
@@ -129,71 +129,71 @@ static sound_stream_info_h stream_info;
 /*---------------------------------------------------------------------------
 |    LOCAL FUNCTION PROTOTYPES:                                                                                                |
 ---------------------------------------------------------------------------*/
-static int             __mmplayer_gst_create_pipeline(mm_player_tplayer);
-static int             __mmplayer_gst_destroy_pipeline(mm_player_tplayer);
-static int             __mmplayer_gst_create_text_pipeline(mm_player_tplayer);
-static int             __mmplayer_gst_create_video_sink_bin(mm_player_tplayer, GstCaps *caps, MMDisplaySurfaceType surface_type);
-static int             __mmplayer_gst_create_audio_sink_bin(mm_player_tplayer);
-static int             __mmplayer_gst_create_text_sink_bin(mm_player_tplayer);
-
-static GstPadProbeReturn       __mmplayer_gst_selector_blocked(GstPadpad, GstPadProbeInfo *info, gpointer data);
-static void            __mmplayer_gst_decode_no_more_pads(GstElementelem, gpointer data);
+static int             __mmplayer_gst_create_pipeline(mm_player_t *player);
+static int             __mmplayer_gst_destroy_pipeline(mm_player_t *player);
+static int             __mmplayer_gst_create_text_pipeline(mm_player_t *player);
+static int             __mmplayer_gst_create_video_sink_bin(mm_player_t *player, GstCaps *caps, MMDisplaySurfaceType surface_type);
+static int             __mmplayer_gst_create_audio_sink_bin(mm_player_t *player);
+static int             __mmplayer_gst_create_text_sink_bin(mm_player_t *player);
+
+static GstPadProbeReturn       __mmplayer_gst_selector_blocked(GstPad *pad, GstPadProbeInfo *info, gpointer data);
+static void            __mmplayer_gst_decode_no_more_pads(GstElement *elem, gpointer data);
 static void            __mmplayer_gst_create_sinkbin(GstElement *decodebin, GstPad *pad, gpointer data);
-static void            __mmplayer_gst_decode_unknown_type(GstElement *elem,  GstPadpad, GstCaps *caps, gpointer data);
-static gboolean __mmplayer_gst_decode_autoplug_continue(GstElement *bin,  GstPad* pad, GstCaps * caps,  gpointer data);
-static void __mmplayer_gst_decode_pad_removed(GstElement *elem,  GstPadnew_pad, gpointer data);
+static void            __mmplayer_gst_decode_unknown_type(GstElement *elem,  GstPad *pad, GstCaps *caps, gpointer data);
+static gboolean __mmplayer_gst_decode_autoplug_continue(GstElement *bin,  GstPad *pad, GstCaps *caps,  gpointer data);
+static void __mmplayer_gst_decode_pad_removed(GstElement *elem,  GstPad *new_pad, gpointer data);
 static void __mmplayer_gst_decode_drained(GstElement *bin, gpointer data);
 static void    __mmplayer_pipeline_complete(GstElement *decodebin,  gpointer data);
-static gboolean __mmplayer_is_midi_type(gcharstr_caps);
+static gboolean __mmplayer_is_midi_type(gchar *str_caps);
 static gboolean __mmplayer_is_only_mp3_type(gchar *str_caps);
-static void    __mmplayer_set_audio_attrs(mm_player_t* player, GstCaps* caps);
-
-static gboolean        __mmplayer_update_subtitle(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data);
-static void            __mmplayer_release_misc(mm_player_tplayer);
-static void            __mmplayer_release_misc_post(mm_player_tplayer);
-static gboolean        __mmplayer_init_gstreamer(mm_player_tplayer);
-static void __mmplayer_video_stream_decoded_preroll_cb(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data);
-static void __mmplayer_video_stream_decoded_render_cb(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data);
+static void    __mmplayer_set_audio_attrs(mm_player_t *player, GstCaps *caps);
+
+static gboolean        __mmplayer_update_subtitle(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data);
+static void            __mmplayer_release_misc(mm_player_t *player);
+static void            __mmplayer_release_misc_post(mm_player_t *player);
+static gboolean        __mmplayer_init_gstreamer(mm_player_t *player);
+static void __mmplayer_video_stream_decoded_preroll_cb(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data);
+static void __mmplayer_video_stream_decoded_render_cb(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data);
 static GstPadProbeReturn __mmplayer_subtitle_adjust_position_probe(GstPad *pad, GstPadProbeInfo *info, gpointer u_data);
-static int __mmplayer_change_selector_pad(mm_player_tplayer, MMPlayerTrackType type, int index);
-
-static gboolean __mmplayer_check_subtitle(mm_player_tplayer);
-static int             __mmplayer_handle_missed_plugin(mm_player_tplayer);
-static int             __mmplayer_check_not_supported_codec(mm_player_t* player, const gchar* factory_class, const gchar* mime);
-static void            __mmplayer_add_sink(mm_player_t* player, GstElement* sink);
-static void            __mmplayer_del_sink(mm_player_t* player, GstElement* sink);
-static void            __mmplayer_release_signal_connection(mm_player_tplayer, MMPlayerSignalType type);
+static int __mmplayer_change_selector_pad(mm_player_t *player, MMPlayerTrackType type, int index);
+
+static gboolean __mmplayer_check_subtitle(mm_player_t *player);
+static int             __mmplayer_handle_missed_plugin(mm_player_t *player);
+static int             __mmplayer_check_not_supported_codec(mm_player_t *player, const gchar *factory_class, const gchar *mime);
+static void            __mmplayer_add_sink(mm_player_t *player, GstElement *sink);
+static void            __mmplayer_del_sink(mm_player_t *player, GstElement *sink);
+static void            __mmplayer_release_signal_connection(mm_player_t *player, MMPlayerSignalType type);
 static gpointer __mmplayer_gapless_play_thread(gpointer data);
 static gboolean __mmplayer_add_dump_buffer_probe(mm_player_t *player, GstElement *element);
 static GstPadProbeReturn __mmplayer_dump_buffer_probe_cb(GstPad *pad,  GstPadProbeInfo *info, gpointer u_data);
 static void __mmplayer_release_dump_list(GList *dump_list);
-static int             __mmplayer_gst_realize(mm_player_tplayer);
-static int             __mmplayer_gst_unrealize(mm_player_tplayer);
-static int             __mmplayer_gst_adjust_subtitle_position(mm_player_tplayer, int format, int position);
-static int             __mmplayer_gst_set_message_callback(mm_player_tplayer, MMMessageCallback callback, gpointer user_param);
+static int             __mmplayer_gst_realize(mm_player_t *player);
+static int             __mmplayer_gst_unrealize(mm_player_t *player);
+static int             __mmplayer_gst_adjust_subtitle_position(mm_player_t *player, int format, int position);
+static int             __mmplayer_gst_set_message_callback(mm_player_t *player, MMMessageCallback callback, gpointer user_param);
 
 /* util */
 static gboolean __mmplayer_verify_gapless_play_path(mm_player_t *player);
 static void __mmplayer_activate_next_source(mm_player_t *player, GstState target);
-static void __mmplayer_check_pipeline(mm_player_tplayer);
+static void __mmplayer_check_pipeline(mm_player_t *player);
 static gboolean __mmplayer_deactivate_selector(mm_player_t *player, MMPlayerTrackType type);
 static void __mmplayer_deactivate_old_path(mm_player_t *player);
-static int __mmplayer_gst_create_plain_text_elements(mm_player_tplayer);
-static guint32 _mmplayer_convert_fourcc_string_to_value(const gcharformat_name);
-static void            __mmplayer_gst_caps_notify_cb(GstPad * pad, GParamSpec * unused, gpointer data);
-static void            __mmplayer_audio_stream_send_data(mm_player_tplayer, mm_player_audio_stream_buff_t *a_buffer);
-static void            __mmplayer_initialize_storage_info(mm_player_tplayer, MMPlayerPathType path_type);
+static int __mmplayer_gst_create_plain_text_elements(mm_player_t *player);
+static guint32 _mmplayer_convert_fourcc_string_to_value(const gchar *format_name);
+static void            __mmplayer_gst_caps_notify_cb(GstPad *pad, GParamSpec *unused, gpointer data);
+static void            __mmplayer_audio_stream_send_data(mm_player_t *player, mm_player_audio_stream_buff_t *a_buffer);
+static void            __mmplayer_initialize_storage_info(mm_player_t *player, MMPlayerPathType path_type);
 static int             __resource_release_cb(mm_resource_manager_h rm, mm_resource_manager_res_h res, void *user_data);
 static gboolean __mmplayer_update_duration_attrs(mm_player_t *player, MMHandleType attrs);
 static gboolean __mmplayer_update_audio_attrs(mm_player_t *player, MMHandleType attrs);
 static gboolean __mmplayer_update_video_attrs(mm_player_t *player, MMHandleType attrs);
 static gboolean __mmplayer_update_bitrate_attrs(mm_player_t *player, MMHandleType attrs);
 
-static void __mmplayer_copy_uri_and_set_type(MMPlayerParseProfiledata, const char *uri, int uri_type);
-static int __mmplayer_set_mem_uri(MMPlayerParseProfiledata, char *path, void *param);
-static int __mmplayer_set_file_uri(MMPlayerParseProfiledata, const char *uri);
+static void __mmplayer_copy_uri_and_set_type(MMPlayerParseProfile *data, const char *uri, int uri_type);
+static int __mmplayer_set_mem_uri(MMPlayerParseProfile *data, char *path, void *param);
+static int __mmplayer_set_file_uri(MMPlayerParseProfile *data, const char *uri);
 
-static MMPlayerVideoStreamDataType__mmplayer_create_stream_from_pad(GstPad *pad);
+static MMPlayerVideoStreamDataType *__mmplayer_create_stream_from_pad(GstPad *pad);
 static void __mmplayer_zerocopy_set_stride_elevation_bo(MMPlayerVideoStreamDataType *stream, GstMemory *mem);
 static gboolean __mmplayer_swcodec_set_stride_elevation(MMPlayerVideoStreamDataType *stream);
 static gboolean __mmplayer_swcodec_set_bo(mm_player_t *player, MMPlayerVideoStreamDataType *stream, GstMemory *mem);
@@ -208,7 +208,7 @@ static void __mmplayer_set_playing_state(mm_player_t *player);
 
 #if 0 //debug
 static void
-print_tag(const GstTagList * list, const gchar * tag, gpointer unused)
+print_tag(const GstTagList *list, const gchar *tag, gpointer unused)
 {
        gint i, count;
 
@@ -222,13 +222,14 @@ print_tag(const GstTagList * list, const gchar * tag, gpointer unused)
                if (gst_tag_get_type(tag) == G_TYPE_STRING) {
                        if (!gst_tag_list_get_string_index(list, tag, i, &str))
                                g_assert_not_reached();
-               } else
+               } else {
                        str = g_strdup_value_contents(gst_tag_list_get_value_index(list, tag, i));
+               }
 
                if (i == 0)
-                       g_print("  %15s: %s\n", gst_tag_get_nick(tag), str);
+                       g_print("  %15s: %s", gst_tag_get_nick(tag), str);
                else
-                       g_print("                 : %s\n", str);
+                       g_print("                 : %s", str);
 
                g_free(str);
        }
@@ -238,7 +239,7 @@ print_tag(const GstTagList * list, const gchar * tag, gpointer unused)
 /* This function should be called after the pipeline goes PAUSED or higher
 state. */
 gboolean
-__mmplayer_update_content_attrs(mm_player_tplayer, enum content_attr_flag flag)
+__mmplayer_update_content_attrs(mm_player_t *player, enum content_attr_flag flag)
 {
        static gboolean has_duration = FALSE;
        static gboolean has_video_attrs = FALSE;
@@ -302,7 +303,7 @@ __mmplayer_update_content_attrs(mm_player_t* player, enum content_attr_flag flag
 
        /* validate all */
        if (mm_attrs_commit_all(attrs)) {
-               LOGE("failed to update attributes\n");
+               LOGE("failed to update attributes");
                return FALSE;
        }
 
@@ -311,7 +312,8 @@ __mmplayer_update_content_attrs(mm_player_t* player, enum content_attr_flag flag
        return TRUE;
 }
 
-MMStreamingType __mmplayer_get_stream_service_type(mm_player_t* player)
+MMStreamingType
+__mmplayer_get_stream_service_type(mm_player_t *player)
 {
        MMStreamingType streaming_type = STREAMING_SERVICE_NONE;
 
@@ -347,7 +349,6 @@ MMStreamingType __mmplayer_get_stream_service_type(mm_player_t* player)
        return streaming_type;
 }
 
-
 /* this function sets the player state and also report
  * it to applicaton by calling callback function
  */
@@ -359,7 +360,7 @@ __mmplayer_set_state(mm_player_t *player, int state)
        MMPLAYER_RETURN_IF_FAIL(player);
 
        if (MMPLAYER_CURRENT_STATE(player) == state) {
-               LOGW("already same state(%s)\n", MMPLAYER_STATE_GET_NAME(state));
+               LOGW("already same state(%s)", MMPLAYER_STATE_GET_NAME(state));
                MMPLAYER_PENDING_STATE(player) = MM_PLAYER_STATE_NONE;
                return;
        }
@@ -386,7 +387,7 @@ __mmplayer_set_state(mm_player_t *player, int state)
                break;
        case MM_PLAYER_STATE_NONE:
        default:
-               LOGW("invalid target state, there is nothing to do.\n");
+               LOGW("invalid target state, there is nothing to do.");
                break;
        }
 
@@ -401,13 +402,13 @@ __mmplayer_set_state(mm_player_t *player, int state)
                LOGD("player reach the target state (%s)", MMPLAYER_STATE_GET_NAME(MMPLAYER_TARGET_STATE(player)));
 
                /* state changed by resource callback */
-               if (player->interrupted_by_resource) {
+               if (player->interrupted_by_resource)
                        MMPLAYER_POST_MSG(player, MM_MESSAGE_STATE_INTERRUPTED, &msg);
-               } else { /* state changed by usecase */
+               else /* state changed by usecase */
                        MMPLAYER_POST_MSG(player, MM_MESSAGE_STATE_CHANGED, &msg);
-               }
+
        } else {
-               LOGD("intermediate state, do nothing.\n");
+               LOGD("intermediate state, do nothing.");
                MMPLAYER_PRINT_STATE(player);
                return;
        }
@@ -422,14 +423,14 @@ __mmplayer_set_state(mm_player_t *player, int state)
 }
 
 int
-__mmplayer_check_state(mm_player_tplayer, enum PlayerCommandState command)
+__mmplayer_check_state(mm_player_t *player, enum PlayerCommandState command)
 {
        MMPlayerStateType current_state = MM_PLAYER_STATE_NUM;
        MMPlayerStateType pending_state = MM_PLAYER_STATE_NUM;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       //LOGD("incomming command : %d \n", command);
+       //LOGD("incomming command : %d ", command);
 
        current_state = MMPLAYER_CURRENT_STATE(player);
        pending_state = MMPLAYER_PENDING_STATE(player);
@@ -497,8 +498,9 @@ __mmplayer_check_state(mm_player_t* player, enum PlayerCommandState command)
                        goto ALREADY_GOING;
                } else if (pending_state == MM_PLAYER_STATE_PAUSED) {
                        LOGD("player is going to paused state, just change the pending state as playing");
-               } else
+               } else {
                        goto INVALID_STATE;
+               }
        }
        break;
 
@@ -558,8 +560,9 @@ __mmplayer_check_state(mm_player_t* player, enum PlayerCommandState command)
                        goto ALREADY_GOING;
                } else if (pending_state == MM_PLAYER_STATE_PAUSED) {
                        LOGD("player is going to paused state, just change the pending state as playing");
-               } else
+               } else {
                        goto INVALID_STATE;
+               }
        }
                break;
 
@@ -588,26 +591,27 @@ ALREADY_GOING:
        return MM_ERROR_PLAYER_NO_OP;
 }
 
-static gpointer __mmplayer_gapless_play_thread(gpointer data)
+static gpointer
+__mmplayer_gapless_play_thread(gpointer data)
 {
-       mm_player_t* player = (mm_player_t*) data;
+       mm_player_t *player = (mm_player_t *)data;
        MMPlayerGstElement *mainbin = NULL;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, NULL);
 
        MMPLAYER_GAPLESS_PLAY_THREAD_LOCK(player);
        while (!player->gapless_play_thread_exit) {
-               LOGD("gapless play thread started. waiting for signal.\n");
+               LOGD("gapless play thread started. waiting for signal.");
                MMPLAYER_GAPLESS_PLAY_THREAD_WAIT(player);
 
-               LOGD("reconfigure pipeline for gapless play.\n");
+               LOGD("reconfigure pipeline for gapless play.");
 
                if (player->gapless_play_thread_exit) {
                        if (player->gapless.reconfigure) {
                                player->gapless.reconfigure = false;
                                MMPLAYER_PLAYBACK_UNLOCK(player);
                        }
-                       LOGD("exiting gapless play thread\n");
+                       LOGD("exiting gapless play thread");
                        break;
                }
 
@@ -634,7 +638,6 @@ __mmplayer_remove_g_source_from_context(GMainContext *context, guint source_id)
        MMPLAYER_FENTER();
 
        source = g_main_context_find_source_by_id(context, source_id);
-
        if (source != NULL) {
                LOGW("context: %p, source id: %d, source: %p", context, source_id, source);
                g_source_destroy(source);
@@ -643,9 +646,10 @@ __mmplayer_remove_g_source_from_context(GMainContext *context, guint source_id)
        MMPLAYER_FLEAVE();
 }
 
-void __mmplayer_bus_msg_thread_destroy(MMHandleType hplayer)
+void
+__mmplayer_bus_msg_thread_destroy(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        GstMessage *msg = NULL;
        GQueue *queue = NULL;
 
@@ -689,9 +693,9 @@ void __mmplayer_bus_msg_thread_destroy(MMHandleType hplayer)
 }
 
 gboolean
-__mmplayer_gst_remove_fakesink(mm_player_t* player, MMPlayerGstElement* fakesink)
+__mmplayer_gst_remove_fakesink(mm_player_t *player, MMPlayerGstElement *fakesink)
 {
-       GstElementparent = NULL;
+       GstElement *parent = NULL;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, FALSE);
 
@@ -706,9 +710,9 @@ __mmplayer_gst_remove_fakesink(mm_player_t* player, MMPlayerGstElement* fakesink
                goto ERROR;
 
        /* get parent of fakesink */
-       parent = (GstElement*)gst_object_get_parent((GstObject*)fakesink->gst);
+       parent = (GstElement *)gst_object_get_parent((GstObject *)fakesink->gst);
        if (!parent) {
-               LOGD("fakesink already removed\n");
+               LOGD("fakesink already removed");
                goto ERROR;
        }
 
@@ -718,13 +722,13 @@ __mmplayer_gst_remove_fakesink(mm_player_t* player, MMPlayerGstElement* fakesink
         * so no need to wait for completion of state transiton
         */
        if (GST_STATE_CHANGE_FAILURE == gst_element_set_state(fakesink->gst, GST_STATE_NULL))
-               LOGE("fakesink state change failure!\n");
+               LOGE("fakesink state change failure!");
                /* FIXIT : should I return here? or try to proceed to next? */
                /* return FALSE; */
 
        /* remove fakesink from it's parent */
        if (!gst_bin_remove(GST_BIN(parent), fakesink->gst)) {
-               LOGE("failed to remove fakesink\n");
+               LOGE("failed to remove fakesink");
 
                gst_object_unref(parent);
 
@@ -733,7 +737,7 @@ __mmplayer_gst_remove_fakesink(mm_player_t* player, MMPlayerGstElement* fakesink
 
        gst_object_unref(parent);
 
-       LOGD("state-holder removed\n");
+       LOGD("state-holder removed");
 
        gst_element_set_locked_state(fakesink->gst, FALSE);
 
@@ -749,14 +753,14 @@ ERROR:
 }
 
 static GstPadProbeReturn
-__mmplayer_gst_selector_blocked(GstPadpad, GstPadProbeInfo *info, gpointer data)
+__mmplayer_gst_selector_blocked(GstPad *pad, GstPadProbeInfo *info, gpointer data)
 {
        LOGD("pad(%s:%s) is blocked", GST_DEBUG_PAD_NAME(pad));
        return GST_PAD_PROBE_OK;
 }
 
 static void
-__mmplayer_gst_selector_update_start_time(mm_player_tplayer, MMPlayerTrackType stream_type)
+__mmplayer_gst_selector_update_start_time(mm_player_t *player, MMPlayerTrackType stream_type)
 {
        gint64 stop_running_time = 0;
        gint64 position_running_time = 0;
@@ -820,7 +824,7 @@ __mmplayer_gst_selector_event_probe(GstPad *pad, GstPadProbeInfo *info, gpointer
 {
        GstPadProbeReturn ret = GST_PAD_PROBE_OK;
        GstEvent *event = GST_PAD_PROBE_INFO_DATA(info);
-       mm_player_t *player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
        GstCaps *caps = NULL;
        GstStructure *str = NULL;
        const gchar *name = NULL;
@@ -1019,7 +1023,7 @@ EXIT:
        return;
 }
 
-static GstElement*
+static GstElement *
 __mmplayer_gst_make_selector(mm_player_t *player, enum MainElementID elem_idx, MMPlayerTrackType stream_type)
 {
        GstElement *pipeline = NULL;
@@ -1061,7 +1065,7 @@ __mmplayer_gst_make_selector(mm_player_t *player, enum MainElementID elem_idx, M
 void
 __mmplayer_gst_decode_pad_added(GstElement *elem, GstPad *pad, gpointer data)
 {
-       mm_player_t *player = NULL;
+       mm_player_t *player = (mm_player_t *)data;
        GstElement *selector = NULL;
        GstCaps *caps = NULL;
        GstStructure *str = NULL;
@@ -1073,9 +1077,6 @@ __mmplayer_gst_decode_pad_added(GstElement *elem, GstPad *pad, gpointer data)
        enum MainElementID elem_idx = MMPLAYER_M_NUM;
        MMPlayerTrackType stream_type = MM_PLAYER_TRACK_TYPE_AUDIO;
 
-       /* check handles */
-       player = (mm_player_t*)data;
-
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(elem && pad);
        MMPLAYER_RETURN_IF_FAIL(player && player->pipeline && player->pipeline->mainbin);
@@ -1097,7 +1098,7 @@ __mmplayer_gst_decode_pad_added(GstElement *elem, GstPad *pad, gpointer data)
                caps_str = gst_caps_to_string(caps);
                if (caps_str && (strstr(caps_str, "ST12") || strstr(caps_str, "SN12") ||
                        strstr(caps_str, "SN21") || strstr(caps_str, "S420") || strstr(caps_str, "SR32")))
-                       player->set_mode.video_zc = TRUE;
+                       player->set_mode.video_zc = true;
 
                MMPLAYER_FREEIF(caps_str);
 
@@ -1193,9 +1194,10 @@ ERROR:
        return;
 }
 
-static gboolean __mmplayer_create_sink_path(mm_player_t* player, GstElement* selector, MMPlayerTrackType type)
+static gboolean
+__mmplayer_create_sink_path(mm_player_t *player, GstElement *selector, MMPlayerTrackType type)
 {
-       GstPadsrcpad = NULL;
+       GstPad *srcpad = NULL;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, FALSE);
@@ -1232,7 +1234,8 @@ static gboolean __mmplayer_create_sink_path(mm_player_t* player, GstElement* sel
        return TRUE;
 }
 
-static void __mmplayer_set_decode_track_info(mm_player_t* player, MMPlayerTrackType type)
+static void
+__mmplayer_set_decode_track_info(mm_player_t *player, MMPlayerTrackType type)
 {
        MMHandleType attrs = 0;
        gint active_index = 0;
@@ -1268,7 +1271,8 @@ static void __mmplayer_set_decode_track_info(mm_player_t* player, MMPlayerTrackT
        return;
 }
 
-static gboolean __mmplayer_create_audio_sink_path(mm_player_t* player, GstElement* audio_selector)
+static gboolean
+__mmplayer_create_audio_sink_path(mm_player_t *player, GstElement *audio_selector)
 {
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, FALSE);
@@ -1306,7 +1310,8 @@ static gboolean __mmplayer_create_audio_sink_path(mm_player_t* player, GstElemen
        return TRUE;
 }
 
-static gboolean __mmplayer_create_text_sink_path(mm_player_t* player, GstElement* text_selector)
+static gboolean
+__mmplayer_create_text_sink_path(mm_player_t *player, GstElement *text_selector)
 {
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player && text_selector, FALSE);
@@ -1337,7 +1342,7 @@ static gboolean __mmplayer_create_text_sink_path(mm_player_t* player, GstElement
 static gboolean
 __mmplayer_gst_set_queue2_buffering(mm_player_t *player)
 {
-#define ESTIMATED_BUFFER_UNIT (1*1024*1024)
+#define ESTIMATED_BUFFER_UNIT (1 * 1024 * 1024)
 
        gint init_buffering_time = 0;
        guint buffer_bytes = 0;
@@ -1348,12 +1353,12 @@ __mmplayer_gst_set_queue2_buffering(mm_player_t *player)
                player->pipeline->mainbin && player->streamer, FALSE);
 
        init_buffering_time = player->streamer->buffering_req.prebuffer_time;
-       buffer_bytes = (guint)(init_buffering_time/1000) * ESTIMATED_BUFFER_UNIT;
+       buffer_bytes = (guint)(init_buffering_time / 1000) * ESTIMATED_BUFFER_UNIT;
 
        buffer_bytes = MAX(buffer_bytes, player->streamer->buffer_handle[BUFFER_TYPE_MUXED].buffering_bytes);
        LOGD("pre buffer time: %d ms, buffer size : %d", init_buffering_time, buffer_bytes);
 
-       init_buffering_time = (init_buffering_time != 0) ? (init_buffering_time) : (player->ini.http_buffering_time);
+       init_buffering_time = (init_buffering_time != 0) ? init_buffering_time : player->ini.http_buffering_time;
 
        if (!gst_element_query_duration(player->pipeline->mainbin[MMPLAYER_M_SRC].gst, GST_FORMAT_BYTES, &dur_bytes))
                LOGE("fail to get duration.");
@@ -1382,7 +1387,7 @@ __mmplayer_gst_decode_no_more_pads(GstElement *elem, gpointer data)
        GstElement *text_selector = NULL;
 
        MMPLAYER_FENTER();
-       player = (mm_player_t*) data;
+       player = (mm_player_t *)data;
 
        LOGD("no-more-pad signal handling");
 
@@ -1492,6 +1497,7 @@ EXIT:
        MMPLAYER_FLEAVE();
        return ret;
 }
+
 static void
 __mmplayer_gst_create_sinkbin(GstElement *elem, GstPad *pad, gpointer data)
 {
@@ -1507,7 +1513,7 @@ __mmplayer_gst_create_sinkbin(GstElement *elem, GstPad *pad, gpointer data)
        gchar *sink_pad_name = "sink";
 
        /* check handles */
-       player = (mm_player_t*) data;
+       player = (mm_player_t *)data;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(elem && pad);
@@ -1637,7 +1643,7 @@ ERROR:
 }
 
 static gboolean
-__mmplayer_get_property_value_for_rotation(mm_player_tplayer, int display_angle, int orientation, int *value)
+__mmplayer_get_property_value_for_rotation(mm_player_t *player, int display_angle, int orientation, int *value)
 {
        int required_angle = 0; /* Angle required for straight view */
        int rotation_angle = 0;
@@ -1692,7 +1698,7 @@ __mmplayer_get_property_value_for_rotation(mm_player_t* player, int display_angl
 }
 
 int
-__mmplayer_video_param_check_video_sink_bin(mm_player_tplayer)
+__mmplayer_video_param_check_video_sink_bin(mm_player_t *player)
 {
        /* check video sinkbin is created */
        MMPLAYER_RETURN_VAL_IF_FAIL(player &&
@@ -1706,7 +1712,7 @@ __mmplayer_video_param_check_video_sink_bin(mm_player_t* player)
 }
 
 int
-__mmplayer_get_video_angle(mm_player_tplayer, int *display_angle, int *orientation)
+__mmplayer_get_video_angle(mm_player_t *player, int *display_angle, int *orientation)
 {
        int display_rotation = 0;
        gchar *org_orient = NULL;
@@ -1766,7 +1772,7 @@ __mmplayer_get_video_angle(mm_player_t* player, int *display_angle, int *orienta
 }
 
 void
-__mmplayer_video_param_set_display_rotation(mm_player_tplayer)
+__mmplayer_video_param_set_display_rotation(mm_player_t *player)
 {
        int rotation_value = 0;
        int orientations = 0; // current supported angle values are 0, 90, 180, 270
@@ -1786,7 +1792,7 @@ __mmplayer_video_param_set_display_rotation(mm_player_t* player)
 }
 
 void
-__mmplayer_video_param_set_display_visible(mm_player_tplayer)
+__mmplayer_video_param_set_display_visible(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        int visible = 0;
@@ -1805,7 +1811,7 @@ __mmplayer_video_param_set_display_visible(mm_player_t* player)
 }
 
 void
-__mmplayer_video_param_set_display_method(mm_player_tplayer)
+__mmplayer_video_param_set_display_method(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        int display_method = 0;
@@ -1822,8 +1828,9 @@ __mmplayer_video_param_set_display_method(mm_player_t* player)
        g_object_set(player->pipeline->videobin[MMPLAYER_V_SINK].gst, "display-geometry-method", display_method, NULL);
        LOGD("set video param : method %d", display_method);
 }
+
 void
-__mmplayer_video_param_set_video_roi_area(mm_player_tplayer)
+__mmplayer_video_param_set_video_roi_area(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        void *handle = NULL;
@@ -1848,7 +1855,7 @@ __mmplayer_video_param_set_video_roi_area(mm_player_t* player)
 }
 
 void
-__mmplayer_video_param_set_roi_area(mm_player_tplayer)
+__mmplayer_video_param_set_roi_area(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        void *handle = NULL;
@@ -1885,8 +1892,9 @@ __mmplayer_video_param_set_roi_area(mm_player_t* player)
                        win_roi_x, win_roi_y, win_roi_width, win_roi_height);
        }
 }
+
 void
-__mmplayer_video_param_set_display_overlay(mm_player_tplayer)
+__mmplayer_video_param_set_display_overlay(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        void *handle = NULL;
@@ -1904,21 +1912,21 @@ __mmplayer_video_param_set_display_overlay(mm_player_t* player)
        if (handle) {
                /* default is using wl_surface_id */
                unsigned int wl_surface_id      = 0;
-               wl_surface_id = *(int*)handle;
+               wl_surface_id = *(int *)handle;
                LOGD("set video param : wl_surface_id %d", wl_surface_id);
                gst_video_overlay_set_wl_window_wl_surface_id(
                                GST_VIDEO_OVERLAY(player->pipeline->videobin[MMPLAYER_V_SINK].gst),
-                               *(int*)handle);
-       } else
+                               *(int *)handle);
+       } else {
                /* FIXIT : is it error case? */
                LOGW("still we don't have a window handle on player attribute. create it's own surface.");
+       }
 }
 
-
 int
-__mmplayer_update_wayland_videosink_video_param(mm_player_tplayer, char *param_name)
+__mmplayer_update_wayland_videosink_video_param(mm_player_t *player, char *param_name)
 {
-       bool update_all_param = FALSE;
+       gboolean update_all_param = FALSE;
        MMPLAYER_FENTER();
 
        /* check video sinkbin is created */
@@ -1951,7 +1959,7 @@ __mmplayer_update_wayland_videosink_video_param(mm_player_t* player, char *param
 }
 
 int
-_mmplayer_update_video_param(mm_player_tplayer, char *param_name)
+_mmplayer_update_video_param(mm_player_t *player, char *param_name)
 {
        MMHandleType attrs = 0;
        int surface_type = 0;
@@ -1994,7 +2002,7 @@ int
 _mmplayer_set_audio_only(MMHandleType hplayer, bool audio_only)
 {
        gboolean disable_overlay = FALSE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -2024,7 +2032,7 @@ _mmplayer_set_audio_only(MMHandleType hplayer, bool audio_only)
                        ret = mm_resource_manager_mark_for_release(player->resource_manager,
                                        player->video_overlay_resource);
                        if (ret != MM_RESOURCE_MANAGER_ERROR_NONE) {
-                               LOGE("failed to mark overlay resource for release, ret(0x%x)\n", ret);
+                               LOGE("failed to mark overlay resource for release, ret(0x%x)", ret);
                                goto ERROR;
                        }
                        player->video_overlay_resource = NULL;
@@ -2032,7 +2040,7 @@ _mmplayer_set_audio_only(MMHandleType hplayer, bool audio_only)
 
                ret = mm_resource_manager_commit(player->resource_manager);
                if (ret != MM_RESOURCE_MANAGER_ERROR_NONE) {
-                       LOGE("failed to commit acquiring of overlay resource, ret(0x%x)\n", ret);
+                       LOGE("failed to commit acquiring of overlay resource, ret(0x%x)", ret);
                        goto ERROR;
                }
        } else {
@@ -2043,7 +2051,7 @@ _mmplayer_set_audio_only(MMHandleType hplayer, bool audio_only)
                                        MM_RESOURCE_MANAGER_RES_VOLUME_FULL,
                                        &player->video_overlay_resource);
                        if (ret != MM_RESOURCE_MANAGER_ERROR_NONE) {
-                               LOGE("could not prepare for video_overlay resource\n");
+                               LOGE("could not prepare for video_overlay resource");
                                goto ERROR;
                        }
                }
@@ -2052,7 +2060,7 @@ _mmplayer_set_audio_only(MMHandleType hplayer, bool audio_only)
                /* acquire resources for video overlay */
                ret = mm_resource_manager_commit(player->resource_manager);
                if (ret != MM_RESOURCE_MANAGER_ERROR_NONE) {
-                       LOGE("could not acquire resources for video playing\n");
+                       LOGE("could not acquire resources for video playing");
                        goto ERROR;
                }
 
@@ -2069,7 +2077,7 @@ ERROR:
 int
 _mmplayer_get_audio_only(MMHandleType hplayer, bool *paudio_only)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        gboolean disable_overlay = FALSE;
 
        MMPLAYER_FENTER();
@@ -2087,7 +2095,7 @@ _mmplayer_get_audio_only(MMHandleType hplayer, bool *paudio_only)
 
        g_object_get(player->pipeline->videobin[MMPLAYER_V_SINK].gst, "disable-overlay", &disable_overlay, NULL);
 
-       *paudio_only = (bool)(disable_overlay);
+       *paudio_only = (bool)disable_overlay;
 
        LOGD("audio_only : %d", *paudio_only);
 
@@ -2097,32 +2105,32 @@ _mmplayer_get_audio_only(MMHandleType hplayer, bool *paudio_only)
 }
 
 int
-__mmplayer_gst_element_link_bucket(GListelement_bucket)
+__mmplayer_gst_element_link_bucket(GList *element_bucket)
 {
-       GListbucket = element_bucket;
-       MMPlayerGstElementelement = NULL;
-       MMPlayerGstElementprv_element = NULL;
+       GList *bucket = element_bucket;
+       MMPlayerGstElement *element = NULL;
+       MMPlayerGstElement *prv_element = NULL;
        gint successful_link_count = 0;
 
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(element_bucket, -1);
 
-       prv_element = (MMPlayerGstElement*)bucket->data;
+       prv_element = (MMPlayerGstElement *)bucket->data;
        bucket = bucket->next;
 
        for (; bucket; bucket = bucket->next) {
-               element = (MMPlayerGstElement*)bucket->data;
+               element = (MMPlayerGstElement *)bucket->data;
 
                if (element && element->gst) {
                        if (prv_element && prv_element->gst) {
                                if (gst_element_link(GST_ELEMENT(prv_element->gst), GST_ELEMENT(element->gst))) {
-                                       LOGD("linking [%s] to [%s] success\n",
+                                       LOGD("linking [%s] to [%s] success",
                                                GST_ELEMENT_NAME(GST_ELEMENT(prv_element->gst)),
                                                GST_ELEMENT_NAME(GST_ELEMENT(element->gst)));
                                        successful_link_count++;
                                } else {
-                                       LOGD("linking [%s] to [%s] failed\n",
+                                       LOGD("linking [%s] to [%s] failed",
                                                GST_ELEMENT_NAME(GST_ELEMENT(prv_element->gst)),
                                                GST_ELEMENT_NAME(GST_ELEMENT(element->gst)));
                                        return -1;
@@ -2139,10 +2147,10 @@ __mmplayer_gst_element_link_bucket(GList* element_bucket)
 }
 
 int
-__mmplayer_gst_element_add_bucket_to_bin(GstBin* bin, GList* element_bucket)
+__mmplayer_gst_element_add_bucket_to_bin(GstBin *bin, GList *element_bucket)
 {
-       GListbucket = element_bucket;
-       MMPlayerGstElementelement = NULL;
+       GList *bucket = element_bucket;
+       MMPlayerGstElement *element = NULL;
        int successful_add_count = 0;
 
        MMPLAYER_FENTER();
@@ -2151,11 +2159,11 @@ __mmplayer_gst_element_add_bucket_to_bin(GstBin* bin, GList* element_bucket)
        MMPLAYER_RETURN_VAL_IF_FAIL(bin, 0);
 
        for (; bucket; bucket = bucket->next) {
-               element = (MMPlayerGstElement*)bucket->data;
+               element = (MMPlayerGstElement *)bucket->data;
 
                if (element && element->gst) {
                        if (!gst_bin_add(bin, GST_ELEMENT(element->gst))) {
-                               LOGD("__mmplayer_gst_element_link_bucket : Adding element [%s]  to bin [%s] failed\n",
+                               LOGD("__mmplayer_gst_element_link_bucket : Adding element [%s]  to bin [%s] failed",
                                        GST_ELEMENT_NAME(GST_ELEMENT(element->gst)),
                                        GST_ELEMENT_NAME(GST_ELEMENT(bin)));
                                return 0;
@@ -2169,9 +2177,10 @@ __mmplayer_gst_element_add_bucket_to_bin(GstBin* bin, GList* element_bucket)
        return successful_add_count;
 }
 
-static void __mmplayer_gst_caps_notify_cb(GstPad *pad, GParamSpec *unused, gpointer data)
+static void
+__mmplayer_gst_caps_notify_cb(GstPad *pad, GParamSpec *unused, gpointer data)
 {
-       mm_player_t *player = (mm_player_t*) data;
+       mm_player_t *player = (mm_player_t *)data;
        GstCaps *caps = NULL;
        GstStructure *str = NULL;
        const char *name;
@@ -2223,8 +2232,6 @@ ERROR:
        return;
 }
 
-
-
 /**
  * This function is to create audio pipeline for playing.
  *
@@ -2236,22 +2243,22 @@ ERROR:
  */
 /* macro for code readability. just for sinkbin-creation functions */
 #define MMPLAYER_CREATE_ELEMENT(x_bin, x_id, x_factory, x_name, x_add_bucket, x_player) \
-do {\
-       x_bin[x_id].id = x_id;\
-       x_bin[x_id].gst = gst_element_factory_make(x_factory, x_name);\
-       if (!x_bin[x_id].gst) {\
-               LOGE("failed to create %s \n", x_factory);\
-               goto ERROR;\
-       } else {\
-               if (x_player->ini.set_dump_element_flag)\
-                       __mmplayer_add_dump_buffer_probe(x_player, x_bin[x_id].gst);\
-       } \
-       if (x_add_bucket)\
-               element_bucket = g_list_append(element_bucket, &x_bin[x_id]);\
-} while (0);
+       do {\
+               x_bin[x_id].id = x_id;\
+               x_bin[x_id].gst = gst_element_factory_make(x_factory, x_name);\
+               if (!x_bin[x_id].gst) {\
+                       LOGE("failed to create %s", x_factory);\
+                       goto ERROR;\
+               } else {\
+                       if (x_player->ini.set_dump_element_flag)\
+                               __mmplayer_add_dump_buffer_probe(x_player, x_bin[x_id].gst);\
+               } \
+               if (x_add_bucket)\
+                       element_bucket = g_list_append(element_bucket, &x_bin[x_id]);\
+       } while (0);
 
 void
-__mmplayer_audio_stream_clear_buffer(mm_player_tplayer, gboolean send_all)
+__mmplayer_audio_stream_clear_buffer(mm_player_t *player, gboolean send_all)
 {
        GList *l = NULL;
 
@@ -2266,9 +2273,8 @@ __mmplayer_audio_stream_clear_buffer(mm_player_t* player, gboolean send_all)
                                        LOGD("[%"G_GUINT64_FORMAT"] send remained data.", tmp->channel_mask);
                                        __mmplayer_audio_stream_send_data(player, tmp);
                                }
-                               if (tmp->pcm_data)
-                                       g_free(tmp->pcm_data);
-                               g_free(tmp);
+                               MMPLAYER_FREEIF(tmp->pcm_data);
+                               MMPLAYER_FREEIF(tmp);
                        }
                }
                g_list_free(player->audio_stream_buff_list);
@@ -2279,7 +2285,7 @@ __mmplayer_audio_stream_clear_buffer(mm_player_t* player, gboolean send_all)
 }
 
 static void
-__mmplayer_audio_stream_send_data(mm_player_tplayer, mm_player_audio_stream_buff_t *a_buffer)
+__mmplayer_audio_stream_send_data(mm_player_t *player, mm_player_audio_stream_buff_t *a_buffer)
 {
        MMPlayerAudioStreamDataType audio_stream = { 0, };
 
@@ -2301,10 +2307,9 @@ __mmplayer_audio_stream_send_data(mm_player_t* player, mm_player_audio_stream_bu
 }
 
 static void
-__mmplayer_audio_stream_decoded_render_cb(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data)
+__mmplayer_audio_stream_decoded_render_cb(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*) data;
-
+       mm_player_t *player = (mm_player_t *)data;
        gint channel = 0;
        gint rate = 0;
        gint depth = 0;
@@ -2372,7 +2377,7 @@ __mmplayer_audio_stream_decoded_render_cb(GstElement* object, GstBuffer *buffer,
        }
 
        /* create new audio stream data */
-       a_buffer = (mm_player_audio_stream_buff_t*)g_malloc0(sizeof(mm_player_audio_stream_buff_t));
+       a_buffer = (mm_player_audio_stream_buff_t *)g_try_malloc0(sizeof(mm_player_audio_stream_buff_t));
        if (a_buffer == NULL) {
                LOGE("failed to alloc data.");
                goto DONE;
@@ -2380,17 +2385,17 @@ __mmplayer_audio_stream_decoded_render_cb(GstElement* object, GstBuffer *buffer,
        a_buffer->bitrate = rate;
        a_buffer->channel = channel;
        a_buffer->depth = depth;
-       a_buffer->is_little_endian = (endianness == 1234 ? 1 : 0);
+       a_buffer->is_little_endian = (endianness == 1234 ? true : false);
        a_buffer->channel_mask = channel_mask;
        a_buffer->data_size = a_size;
 
        if (!player->audio_stream_sink_sync) {
                /* If sync is FALSE, use buffer list to reduce the IPC. */
                a_buffer->buff_size = (a_size > player->ini.pcm_buffer_size) ? (a_size) : (player->ini.pcm_buffer_size);
-               a_buffer->pcm_data = g_malloc(a_buffer->buff_size);
+               a_buffer->pcm_data = g_try_malloc(a_buffer->buff_size);
                if (a_buffer->pcm_data == NULL) {
                        LOGE("failed to alloc data.");
-                       g_free(a_buffer);
+                       MMPLAYER_FREEIF(a_buffer);
                        goto DONE;
                }
                memcpy(a_buffer->pcm_data, a_data, a_size);
@@ -2400,7 +2405,7 @@ __mmplayer_audio_stream_decoded_render_cb(GstElement* object, GstBuffer *buffer,
                /* If sync is TRUE, send data directly. */
                a_buffer->pcm_data = a_data;
                __mmplayer_audio_stream_send_data(player, a_buffer);
-               g_free(a_buffer);
+               MMPLAYER_FREEIF(a_buffer);
        }
 
 DONE:
@@ -2411,9 +2416,9 @@ DONE:
 static void
 __mmplayer_gst_audio_deinterleave_pad_added(GstElement *elem, GstPad *pad, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
-       MMPlayerGstElementaudiobin = player->pipeline->audiobin;
-       GstPadsinkpad = NULL;
+       mm_player_t *player = (mm_player_t *)data;
+       MMPlayerGstElement *audiobin = player->pipeline->audiobin;
+       GstPad *sinkpad = NULL;
        GstElement *queue = NULL, *sink = NULL;
 
        MMPLAYER_FENTER();
@@ -2421,31 +2426,31 @@ __mmplayer_gst_audio_deinterleave_pad_added(GstElement *elem, GstPad *pad, gpoin
 
        queue = gst_element_factory_make("queue", NULL);
        if (queue == NULL) {
-               LOGD("fail make queue\n");
+               LOGD("fail make queue");
                goto ERROR;
        }
 
        sink = gst_element_factory_make("fakesink", NULL);
        if (sink == NULL) {
-               LOGD("fail make fakesink\n");
+               LOGD("fail make fakesink");
                goto ERROR;
        }
 
        gst_bin_add_many(GST_BIN(audiobin[MMPLAYER_A_BIN].gst), queue, sink, NULL);
 
        if (!gst_element_link_pads_full(queue, "src", sink, "sink", GST_PAD_LINK_CHECK_NOTHING)) {
-               LOGW("failed to link queue & sink\n");
+               LOGW("failed to link queue & sink");
                goto ERROR;
        }
 
        sinkpad = gst_element_get_static_pad(queue, "sink");
 
        if (GST_PAD_LINK_OK != gst_pad_link(pad, sinkpad)) {
-               LOGW("failed to link [%s:%s] to queue\n", GST_DEBUG_PAD_NAME(pad));
+               LOGW("failed to link [%s:%s] to queue", GST_DEBUG_PAD_NAME(pad));
                goto ERROR;
        }
 
-       LOGE("player->audio_stream_sink_sync: %d\n", player->audio_stream_sink_sync);
+       LOGE("player->audio_stream_sink_sync: %d", player->audio_stream_sink_sync);
 
        gst_object_unref(sinkpad);
        g_object_set(sink, "sync", player->audio_stream_sink_sync, NULL);
@@ -2465,7 +2470,7 @@ __mmplayer_gst_audio_deinterleave_pad_added(GstElement *elem, GstPad *pad, gpoin
        return;
 
 ERROR:
-       LOGE("__mmplayer_gst_audio_deinterleave_pad_added ERROR\n");
+       LOGE("__mmplayer_gst_audio_deinterleave_pad_added ERROR");
        if (queue) {
                gst_object_unref(GST_OBJECT(queue));
                queue = NULL;
@@ -2482,7 +2487,8 @@ ERROR:
        return;
 }
 
-void __mmplayer_gst_set_pulsesink_property(mm_player_t* player, MMHandleType attrs)
+void
+__mmplayer_gst_set_pulsesink_property(mm_player_t *player, MMHandleType attrs)
 {
        #define MAX_PROPS_LEN 128
        gint latency_mode = 0;
@@ -2506,10 +2512,10 @@ void __mmplayer_gst_set_pulsesink_property(mm_player_t* player, MMHandleType att
                LOGE("stream_type is null.");
        } else {
                if (player->sound.focus_id)
-                       snprintf(stream_props, sizeof(stream_props)-1, "props,media.role=%s, media.parent_id=%d, media.focus_id=%d",
+                       snprintf(stream_props, sizeof(stream_props) - 1, "props,media.role=%s, media.parent_id=%d, media.focus_id=%d",
                                        stream_type, stream_id, player->sound.focus_id);
                else
-                       snprintf(stream_props, sizeof(stream_props)-1, "props,media.role=%s, media.parent_id=%d",
+                       snprintf(stream_props, sizeof(stream_props) - 1, "props,media.role=%s, media.parent_id=%d",
                                        stream_type, stream_id);
                props = gst_structure_from_string(stream_props, NULL);
                g_object_set(player->pipeline->audiobin[MMPLAYER_A_SINK].gst, "stream-properties", props, NULL);
@@ -2538,12 +2544,13 @@ void __mmplayer_gst_set_pulsesink_property(mm_player_t* player, MMHandleType att
 
        LOGD("audiosink property - latency=%s", latency);
 
-       g_free(latency);
+       MMPLAYER_FREEIF(latency);
 
        MMPLAYER_FLEAVE();
 }
 
-void __mmplayer_gst_set_openalsink_property(mm_player_t* player)
+void
+__mmplayer_gst_set_openalsink_property(mm_player_t *player)
 {
        MMPlayerGstElement *audiobin = NULL;
 
@@ -2561,8 +2568,8 @@ void __mmplayer_gst_set_openalsink_property(mm_player_t* player)
                        player->video360_pitch_radians <= M_PI_2 &&
                        player->video360_pitch_radians >= -M_PI_2) {
                g_object_set(G_OBJECT(audiobin[MMPLAYER_A_SINK].gst),
-                               "source-orientation-y", (int) (player->video360_yaw_radians * 180.0 / M_PI),
-                               "source-orientation-x", (int) (player->video360_pitch_radians * 180.0 / M_PI), NULL);
+                               "source-orientation-y", (int)(player->video360_yaw_radians * 180.0 / M_PI),
+                               "source-orientation-x", (int)(player->video360_pitch_radians * 180.0 / M_PI), NULL);
        } else if (player->video360_metadata.init_view_heading || player->video360_metadata.init_view_pitch) {
                g_object_set(G_OBJECT(audiobin[MMPLAYER_A_SINK].gst),
                                "source-orientation-y", player->video360_metadata.init_view_heading,
@@ -2748,7 +2755,7 @@ ERROR:
 }
 
 static int
-__mmplayer_gst_create_audio_sink_bin(mm_player_tplayer)
+__mmplayer_gst_create_audio_sink_bin(mm_player_t *player)
 {
        MMPlayerGstElement *first_element = NULL;
        MMPlayerGstElement *audiobin = NULL;
@@ -2761,7 +2768,7 @@ __mmplayer_gst_create_audio_sink_bin(mm_player_t* player)
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        /* alloc handles */
-       audiobin = (MMPlayerGstElement*)g_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_A_NUM);
+       audiobin = (MMPlayerGstElement *)g_try_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_A_NUM);
        if (!audiobin) {
                LOGE("failed to allocate memory for audiobin");
                return MM_ERROR_PLAYER_NO_FREE_SPACE;
@@ -2839,7 +2846,7 @@ ERROR:
        for (i = 1; i < MMPLAYER_A_NUM; i++) {
                /* NOTE : skip bin */
                if (audiobin[i].gst) {
-                       GstObjectparent = NULL;
+                       GstObject *parent = NULL;
                        parent = gst_element_get_parent(audiobin[i].gst);
 
                        if (!parent) {
@@ -2861,12 +2868,14 @@ ERROR:
        return MM_ERROR_PLAYER_INTERNAL;
 }
 
-static guint32 _mmplayer_convert_fourcc_string_to_value(const gchar* format_name)
+static guint32
+_mmplayer_convert_fourcc_string_to_value(const gchar *format_name)
 {
        return format_name[0] | (format_name[1] << 8) | (format_name[2] << 16) | (format_name[3] << 24);
 }
 
-int _mmplayer_video_stream_release_bo(mm_player_t* player, void* bo)
+int
+_mmplayer_video_stream_release_bo(mm_player_t *player, void *bo)
 {
        int ret = MM_ERROR_NONE;
        GList *l = NULL;
@@ -2877,7 +2886,7 @@ int _mmplayer_video_stream_release_bo(mm_player_t* player, void* bo)
 
        if (player->video_bo_list) {
                for (l = g_list_first(player->video_bo_list); l; l = g_list_next(l)) {
-                       mm_player_video_bo_info_ttmp = (mm_player_video_bo_info_t *)l->data;
+                       mm_player_video_bo_info_t *tmp = (mm_player_video_bo_info_t *)l->data;
                        if (tmp && tmp->bo == bo) {
                                tmp->used = FALSE;
                                LOGD("release bo %p", bo);
@@ -2898,7 +2907,7 @@ int _mmplayer_video_stream_release_bo(mm_player_t* player, void* bo)
 }
 
 static void
-__mmplayer_video_stream_destroy_bo_list(mm_player_tplayer)
+__mmplayer_video_stream_destroy_bo_list(mm_player_t *player)
 {
        GList *l = NULL;
 
@@ -2909,7 +2918,7 @@ __mmplayer_video_stream_destroy_bo_list(mm_player_t* player)
        if (player->video_bo_list) {
                LOGD("destroy video_bo_list : %d", g_list_length(player->video_bo_list));
                for (l = g_list_first(player->video_bo_list); l; l = g_list_next(l)) {
-                       mm_player_video_bo_info_ttmp = (mm_player_video_bo_info_t *)l->data;
+                       mm_player_video_bo_info_t *tmp = (mm_player_video_bo_info_t *)l->data;
                        if (tmp) {
                                if (tmp->bo)
                                        tbm_bo_unref(tmp->bo);
@@ -2926,8 +2935,8 @@ __mmplayer_video_stream_destroy_bo_list(mm_player_t* player)
        return;
 }
 
-static void*
-__mmplayer_video_stream_get_bo(mm_player_tplayer, int size)
+static void *
+__mmplayer_video_stream_get_bo(mm_player_t *player, int size)
 {
        GList *l = NULL;
        MMPLAYER_RETURN_VAL_IF_FAIL(player, NULL);
@@ -2956,7 +2965,7 @@ __mmplayer_video_stream_get_bo(mm_player_t* player, int size)
                }
 
                for (; idx < player->ini.num_of_video_bo; idx++) {
-                       mm_player_video_bo_info_tbo_info = g_new(mm_player_video_bo_info_t, 1);
+                       mm_player_video_bo_info_t *bo_info = g_new(mm_player_video_bo_info_t, 1);
                        if (!bo_info) {
                                LOGE("Fail to alloc bo_info.");
                                break;
@@ -2964,7 +2973,7 @@ __mmplayer_video_stream_get_bo(mm_player_t* player, int size)
                        bo_info->bo = tbm_bo_alloc(player->bufmgr, size, TBM_BO_DEFAULT);
                        if (!bo_info->bo) {
                                LOGE("Fail to tbm_bo_alloc.");
-                               g_free(bo_info);
+                               MMPLAYER_FREEIF(bo_info);
                                break;
                        }
                        bo_info->used = FALSE;
@@ -2987,7 +2996,7 @@ __mmplayer_video_stream_get_bo(mm_player_t* player, int size)
        while (TRUE) {
                /* get bo from list*/
                for (l = g_list_first(player->video_bo_list); l; l = g_list_next(l)) {
-                       mm_player_video_bo_info_ttmp = (mm_player_video_bo_info_t *)l->data;
+                       mm_player_video_bo_info_t *tmp = (mm_player_video_bo_info_t *)l->data;
                        if (tmp && (tmp->used == FALSE)) {
                                LOGD("found bo %p to use", tmp->bo);
                                tmp->used = TRUE;
@@ -3004,7 +3013,7 @@ __mmplayer_video_stream_get_bo(mm_player_t* player, int size)
                if (player->ini.video_bo_timeout <= 0) {
                        MMPLAYER_VIDEO_BO_WAIT(player);
                } else {
-                       gint64 timeout = g_get_monotonic_time() + player->ini.video_bo_timeout*G_TIME_SPAN_SECOND;
+                       gint64 timeout = g_get_monotonic_time() + player->ini.video_bo_timeout * G_TIME_SPAN_SECOND;
                        ret = MMPLAYER_VIDEO_BO_WAIT_UNTIL(player, timeout);
                }
                continue;
@@ -3012,25 +3021,25 @@ __mmplayer_video_stream_get_bo(mm_player_t* player, int size)
 }
 
 static void
-__mmplayer_video_stream_decoded_preroll_cb(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data)
+__mmplayer_video_stream_decoded_preroll_cb(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(player && player->video_stream_cb);
 
        /* send prerolled pkt */
-       player->video_stream_prerolled = FALSE;
+       player->video_stream_prerolled = false;
 
        __mmplayer_video_stream_decoded_render_cb(object, buffer, pad, data);
 
        /* not to send prerolled pkt again */
-       player->video_stream_prerolled = TRUE;
+       player->video_stream_prerolled = true;
 }
 
 static void
-__mmplayer_video_stream_decoded_render_cb(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data)
+__mmplayer_video_stream_decoded_render_cb(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
        MMPlayerVideoStreamDataType *stream = NULL;
        GstMemory *mem = NULL;
 
@@ -3039,7 +3048,7 @@ __mmplayer_video_stream_decoded_render_cb(GstElement* object, GstBuffer *buffer,
        MMPLAYER_RETURN_IF_FAIL(player->video_stream_cb);
 
        if (player->video_stream_prerolled) {
-               player->video_stream_prerolled = FALSE;
+               player->video_stream_prerolled = false;
                LOGD("skip the prerolled pkt not to send it again");
                return;
        }
@@ -3095,7 +3104,7 @@ ERROR:
                if (stream->bo[0])
                        _mmplayer_video_stream_release_bo(player, stream->bo[0]);
        }
-       g_free(stream);
+       MMPLAYER_FREEIF(stream);
        return;
 }
 
@@ -3147,8 +3156,8 @@ __mmplayer_gst_set_video360_property(mm_player_t *player)
                        player->video360_pitch_radians <= M_PI_2 &&
                        player->video360_pitch_radians >= -M_PI_2) {
                g_object_set(G_OBJECT(videobin[MMPLAYER_V_360].gst),
-                               "pose-yaw", (int) (player->video360_yaw_radians * 180.0 / M_PI),
-                               "pose-pitch", (int) (player->video360_pitch_radians * 180.0 / M_PI), NULL);
+                               "pose-yaw", (int)(player->video360_yaw_radians * 180.0 / M_PI),
+                               "pose-pitch", (int)(player->video360_pitch_radians * 180.0 / M_PI), NULL);
        } else if (player->video360_metadata.init_view_heading || player->video360_metadata.init_view_pitch) {
                g_object_set(G_OBJECT(videobin[MMPLAYER_V_360].gst),
                                "pose-yaw", player->video360_metadata.init_view_heading,
@@ -3205,7 +3214,7 @@ ERROR: /* refer MMPLAYER_CREATE_ELEMENT */
        return MM_ERROR_PLAYER_INTERNAL;
 }
 
-static gchar*
+static gchar *
 __mmplayer_get_videosink_factory_name(mm_player_t *player, MMDisplaySurfaceType surface_type)
 {
        gchar *factory_name = NULL;
@@ -3314,7 +3323,7 @@ __mmplayer_gst_set_videosink_property(mm_player_t *player, MMDisplaySurfaceType
  * - video overlay surface(arm/x86) : tizenwlsink
  */
 static int
-__mmplayer_gst_create_video_sink_bin(mm_player_t* player, GstCaps* caps, MMDisplaySurfaceType surface_type)
+__mmplayer_gst_create_video_sink_bin(mm_player_t *player, GstCaps *caps, MMDisplaySurfaceType surface_type)
 {
        GstPad *pad = NULL;
        GList *element_bucket = NULL;
@@ -3326,7 +3335,7 @@ __mmplayer_gst_create_video_sink_bin(mm_player_t* player, GstCaps* caps, MMDispl
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        /* alloc handles */
-       videobin = (MMPlayerGstElement*)g_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_V_NUM);
+       videobin = (MMPlayerGstElement *)g_try_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_V_NUM);
        if (!videobin)
                return MM_ERROR_PLAYER_NO_FREE_SPACE;
 
@@ -3412,7 +3421,8 @@ ERROR:
        return MM_ERROR_PLAYER_INTERNAL;
 }
 
-static int __mmplayer_gst_create_plain_text_elements(mm_player_t* player)
+static int
+__mmplayer_gst_create_plain_text_elements(mm_player_t *player)
 {
        GList *element_bucket = NULL;
        MMPlayerGstElement *textbin = player->pipeline->textbin;
@@ -3435,14 +3445,14 @@ static int __mmplayer_gst_create_plain_text_elements(mm_player_t* player)
                                                "signal-handoffs", TRUE, "max-lateness", FAKE_SINK_MAX_LATENESS, NULL);
 
        if (!player->play_subtitle) {
-               LOGD("add textbin sink as sink element of whole pipeline.\n");
+               LOGD("add textbin sink as sink element of whole pipeline.");
                __mmplayer_add_sink(player, GST_ELEMENT(textbin[MMPLAYER_T_FAKE_SINK].gst));
        }
 
        /* adding created elements to bin */
-       LOGD("adding created elements to bin\n");
+       LOGD("adding created elements to bin");
        if (!__mmplayer_gst_element_add_bucket_to_bin(GST_BIN(textbin[MMPLAYER_T_BIN].gst), element_bucket)) {
-               LOGE("failed to add elements\n");
+               LOGE("failed to add elements");
                goto ERROR;
        }
 
@@ -3451,9 +3461,9 @@ static int __mmplayer_gst_create_plain_text_elements(mm_player_t* player)
        GST_OBJECT_FLAG_UNSET(textbin[MMPLAYER_T_FAKE_SINK].gst, GST_ELEMENT_FLAG_SINK);
 
        /* linking elements in the bucket by added order. */
-       LOGD("Linking elements in the bucket by added order.\n");
+       LOGD("Linking elements in the bucket by added order.");
        if (__mmplayer_gst_element_link_bucket(element_bucket) == -1) {
-               LOGE("failed to link elements\n");
+               LOGE("failed to link elements");
                goto ERROR;
        }
 
@@ -3474,12 +3484,12 @@ static int __mmplayer_gst_create_plain_text_elements(mm_player_t* player)
                gst_object_unref(pad);
 
                if (!ghostpad) {
-                       LOGE("failed to create ghostpad of textbin\n");
+                       LOGE("failed to create ghostpad of textbin");
                        goto ERROR;
                }
 
                if (!gst_element_add_pad(textbin[MMPLAYER_T_BIN].gst, ghostpad)) {
-                       LOGE("failed to add ghostpad to textbin\n");
+                       LOGE("failed to add ghostpad to textbin");
                        gst_object_unref(ghostpad);
                        goto ERROR;
                }
@@ -3499,7 +3509,8 @@ ERROR:
        return MM_ERROR_PLAYER_INTERNAL;
 }
 
-static int __mmplayer_gst_create_text_sink_bin(mm_player_t* player)
+static int
+__mmplayer_gst_create_text_sink_bin(mm_player_t *player)
 {
        MMPlayerGstElement *textbin = NULL;
        GList *element_bucket = NULL;
@@ -3511,9 +3522,9 @@ static int __mmplayer_gst_create_text_sink_bin(mm_player_t* player)
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        /* alloc handles */
-       textbin = (MMPlayerGstElement*)g_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_T_NUM);
+       textbin = (MMPlayerGstElement *)g_try_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_T_NUM);
        if (!textbin) {
-               LOGE("failed to allocate memory for textbin\n");
+               LOGE("failed to allocate memory for textbin");
                return MM_ERROR_PLAYER_NO_FREE_SPACE;
        }
 
@@ -3521,7 +3532,7 @@ static int __mmplayer_gst_create_text_sink_bin(mm_player_t* player)
        textbin[MMPLAYER_T_BIN].id = MMPLAYER_T_BIN;
        textbin[MMPLAYER_T_BIN].gst = gst_bin_new("textbin");
        if (!textbin[MMPLAYER_T_BIN].gst) {
-               LOGE("failed to create textbin\n");
+               LOGE("failed to create textbin");
                goto ERROR;
        }
 
@@ -3536,7 +3547,7 @@ static int __mmplayer_gst_create_text_sink_bin(mm_player_t* player)
        case MM_DISPLAY_SURFACE_NULL:
        case MM_DISPLAY_SURFACE_REMOTE:
                if (__mmplayer_gst_create_plain_text_elements(player) != MM_ERROR_NONE) {
-                       LOGE("failed to make plain text elements\n");
+                       LOGE("failed to make plain text elements");
                        goto ERROR;
                }
                break;
@@ -3551,7 +3562,7 @@ static int __mmplayer_gst_create_text_sink_bin(mm_player_t* player)
 
 ERROR:
 
-       LOGD("ERROR : releasing textbin\n");
+       LOGD("ERROR : releasing textbin");
 
        g_list_free(element_bucket);
 
@@ -3562,7 +3573,7 @@ ERROR:
        for (i = 1; i < MMPLAYER_T_NUM; i++) {
                /* NOTE : skip bin */
                if (textbin[i].gst) {
-                       GstObjectparent = NULL;
+                       GstObject *parent = NULL;
                        parent = gst_element_get_parent(textbin[i].gst);
 
                        if (!parent) {
@@ -3585,12 +3596,11 @@ ERROR:
        return MM_ERROR_PLAYER_INTERNAL;
 }
 
-
 static int
-__mmplayer_gst_create_text_pipeline(mm_player_tplayer)
+__mmplayer_gst_create_text_pipeline(mm_player_t *player)
 {
-       MMPlayerGstElementmainbin = NULL;
-       MMPlayerGstElementtextbin = NULL;
+       MMPlayerGstElement *mainbin = NULL;
+       MMPlayerGstElement *textbin = NULL;
        MMHandleType attrs = 0;
        GstElement *subsrc = NULL;
        GstElement *subparse = NULL;
@@ -3609,13 +3619,13 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
 
        attrs = MMPLAYER_GET_ATTRS(player);
        if (!attrs) {
-               LOGE("cannot get content attribute\n");
+               LOGE("cannot get content attribute");
                return MM_ERROR_PLAYER_INTERNAL;
        }
 
        mm_attrs_get_string_by_name(attrs, "subtitle_uri", &subtitle_uri);
        if (!subtitle_uri || strlen(subtitle_uri) < 1) {
-               LOGE("subtitle uri is not proper filepath.\n");
+               LOGE("subtitle uri is not proper filepath.");
                return MM_ERROR_PLAYER_INVALID_URI;
        }
 
@@ -3624,7 +3634,7 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
                return MM_ERROR_PLAYER_INVALID_URI;
        }
 
-       SECURE_LOGD("subtitle file path is [%s].\n", subtitle_uri);
+       SECURE_LOGD("subtitle file path is [%s].", subtitle_uri);
 
        MMPLAYER_SUBTITLE_INFO_LOCK(player);
        player->subtitle_language_list = NULL;
@@ -3633,7 +3643,7 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        /* create the subtitle source */
        subsrc = gst_element_factory_make("filesrc", "subtitle_source");
        if (!subsrc) {
-               LOGE("failed to create filesrc element\n");
+               LOGE("failed to create filesrc element");
                goto ERROR;
        }
        g_object_set(G_OBJECT(subsrc), "location", subtitle_uri, NULL);
@@ -3642,7 +3652,7 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        mainbin[MMPLAYER_M_SUBSRC].gst = subsrc;
 
        if (!gst_bin_add(GST_BIN(mainbin[MMPLAYER_M_PIPE].gst), subsrc)) {
-               LOGW("failed to add queue\n");
+               LOGW("failed to add queue");
                gst_object_unref(mainbin[MMPLAYER_M_SUBSRC].gst);
                mainbin[MMPLAYER_M_SUBSRC].gst = NULL;
                mainbin[MMPLAYER_M_SUBSRC].id = MMPLAYER_M_NUM;
@@ -3652,13 +3662,13 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        /* subparse */
        subparse = gst_element_factory_make("subparse", "subtitle_parser");
        if (!subparse) {
-               LOGE("failed to create subparse element\n");
+               LOGE("failed to create subparse element");
                goto ERROR;
        }
 
        charset = util_get_charset(subtitle_uri);
        if (charset) {
-               LOGD("detected charset is %s\n", charset);
+               LOGD("detected charset is %s", charset);
                g_object_set(G_OBJECT(subparse), "subtitle-encoding", charset, NULL);
        }
 
@@ -3666,7 +3676,7 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        mainbin[MMPLAYER_M_SUBPARSE].gst = subparse;
 
        if (!gst_bin_add(GST_BIN(mainbin[MMPLAYER_M_PIPE].gst), subparse)) {
-               LOGW("failed to add subparse\n");
+               LOGW("failed to add subparse");
                gst_object_unref(mainbin[MMPLAYER_M_SUBPARSE].gst);
                mainbin[MMPLAYER_M_SUBPARSE].gst = NULL;
                mainbin[MMPLAYER_M_SUBPARSE].id = MMPLAYER_M_NUM;
@@ -3674,25 +3684,25 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        }
 
        if (!gst_element_link_pads(subsrc, "src", subparse, "sink")) {
-               LOGW("failed to link subsrc and subparse\n");
+               LOGW("failed to link subsrc and subparse");
                goto ERROR;
        }
 
        player->play_subtitle = TRUE;
        player->adjust_subtitle_pos = 0;
 
-       LOGD("play subtitle using subtitle file\n");
+       LOGD("play subtitle using subtitle file");
 
        if (player->pipeline->textbin == NULL) {
                if (MM_ERROR_NONE != __mmplayer_gst_create_text_sink_bin(player)) {
-                       LOGE("failed to create text sink bin. continuing without text\n");
+                       LOGE("failed to create text sink bin. continuing without text");
                        goto ERROR;
                }
 
                textbin = player->pipeline->textbin;
 
                if (!gst_bin_add(GST_BIN(mainbin[MMPLAYER_M_PIPE].gst), GST_ELEMENT(textbin[MMPLAYER_T_BIN].gst))) {
-                       LOGW("failed to add textbin\n");
+                       LOGW("failed to add textbin");
 
                        /* release signal */
                        __mmplayer_release_signal_connection(player, MM_PLAYER_SIGNAL_TYPE_TEXTBIN);
@@ -3716,7 +3726,7 @@ __mmplayer_gst_create_text_pipeline(mm_player_t* player)
        }
 
        if (!gst_element_link_pads(subparse, "src", textbin[MMPLAYER_T_BIN].gst, "text_sink")) {
-               LOGW("failed to link subparse and textbin\n");
+               LOGW("failed to link subparse and textbin");
                goto ERROR;
        }
 
@@ -3763,9 +3773,9 @@ ERROR:
 }
 
 gboolean
-__mmplayer_update_subtitle(GstElementobject, GstBuffer *buffer, GstPad *pad, gpointer data)
+__mmplayer_update_subtitle(GstElement *object, GstBuffer *buffer, GstPad *pad, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*) data;
+       mm_player_t *player = (mm_player_t *)data;
        MMMessageParamType msg = {0, };
        GstClockTime duration = 0;
        gpointer text = NULL;
@@ -3789,19 +3799,19 @@ __mmplayer_update_subtitle(GstElement* object, GstBuffer *buffer, GstPad *pad, g
        duration = GST_BUFFER_DURATION(buffer);
 
        if (player->set_mode.subtitle_off) {
-               LOGD("subtitle is OFF.\n");
+               LOGD("subtitle is OFF.");
                return TRUE;
        }
 
        if (!text || (text_size == 0)) {
-               LOGD("There is no subtitle to be displayed.\n");
+               LOGD("There is no subtitle to be displayed.");
                return TRUE;
        }
 
-       msg.data = (void *) text;
+       msg.data = (void *)text;
        msg.subtitle.duration = GST_TIME_AS_MSECONDS(duration);
 
-       LOGD("update subtitle : [%ld msec] %s\n'", msg.subtitle.duration, (char*)msg.data);
+       LOGD("update subtitle : [%ld msec] %s", msg.subtitle.duration, (char *)msg.data);
 
        MMPLAYER_POST_MSG(player, MM_MESSAGE_UPDATE_SUBTITLE, &msg);
        gst_buffer_unmap(buffer, &mapinfo);
@@ -3814,7 +3824,7 @@ __mmplayer_update_subtitle(GstElement* object, GstBuffer *buffer, GstPad *pad, g
 static GstPadProbeReturn
 __mmplayer_subtitle_adjust_position_probe(GstPad *pad, GstPadProbeInfo *info, gpointer u_data)
 {
-       mm_player_t *player = (mm_player_t *) u_data;
+       mm_player_t *player = (mm_player_t *)u_data;
        GstClockTime cur_timestamp = 0;
        gint64 adjusted_timestamp = 0;
        GstBuffer *buffer = gst_pad_probe_info_get_buffer(info);
@@ -3822,7 +3832,7 @@ __mmplayer_subtitle_adjust_position_probe(GstPad *pad, GstPadProbeInfo *info, gp
        MMPLAYER_RETURN_VAL_IF_FAIL(player, FALSE);
 
        if (player->set_mode.subtitle_off) {
-               LOGD("subtitle is OFF.\n");
+               LOGD("subtitle is OFF.");
                return TRUE;
        }
 
@@ -3832,7 +3842,7 @@ __mmplayer_subtitle_adjust_position_probe(GstPad *pad, GstPadProbeInfo *info, gp
        }
 
        cur_timestamp = GST_BUFFER_TIMESTAMP(buffer);
-       adjusted_timestamp = (gint64) cur_timestamp +((gint64) player->adjust_subtitle_pos * G_GINT64_CONSTANT(1000000));
+       adjusted_timestamp = (gint64)cur_timestamp + ((gint64)player->adjust_subtitle_pos * G_GINT64_CONSTANT(1000000));
 
        if (adjusted_timestamp < 0) {
                LOGD("adjusted_timestamp under zero");
@@ -3847,7 +3857,9 @@ __mmplayer_subtitle_adjust_position_probe(GstPad *pad, GstPadProbeInfo *info, gp
 
        return GST_PAD_PROBE_OK;
 }
-static int __mmplayer_gst_adjust_subtitle_position(mm_player_t* player, int format, int position)
+
+static int
+__mmplayer_gst_adjust_subtitle_position(mm_player_t *player, int format, int position)
 {
        MMPLAYER_FENTER();
 
@@ -3856,7 +3868,7 @@ static int __mmplayer_gst_adjust_subtitle_position(mm_player_t* player, int form
        MMPLAYER_RETURN_VAL_IF_FAIL(player->play_subtitle, MM_ERROR_NOT_SUPPORT_API);
 
        if (position == 0) {
-               LOGD("nothing to do\n");
+               LOGD("nothing to do");
                MMPLAYER_FLEAVE();
                return MM_ERROR_NONE;
        }
@@ -3873,7 +3885,7 @@ static int __mmplayer_gst_adjust_subtitle_position(mm_player_t* player, int form
 
        default:
                {
-                       LOGW("invalid format.\n");
+                       LOGW("invalid format.");
                        MMPLAYER_FLEAVE();
                        return MM_ERROR_INVALID_ARGUMENT;
                }
@@ -3894,7 +3906,7 @@ static int __mmplayer_gst_adjust_subtitle_position(mm_player_t* player, int form
  * @see
  */
 static int
-__mmplayer_gst_create_pipeline(mm_player_tplayer)
+__mmplayer_gst_create_pipeline(mm_player_t *player)
 {
        int ret = MM_ERROR_NONE;
        MMPlayerGstElement *mainbin = NULL;
@@ -3916,12 +3928,12 @@ __mmplayer_gst_create_pipeline(mm_player_t* player)
                goto INIT_ERROR;
        }
 
-       player->pipeline = (MMPlayerGstPipelineInfo*) g_malloc0(sizeof(MMPlayerGstPipelineInfo));
+       player->pipeline = (MMPlayerGstPipelineInfo *)g_malloc0(sizeof(MMPlayerGstPipelineInfo));
        if (player->pipeline == NULL)
                goto INIT_ERROR;
 
        /* create mainbin */
-       mainbin = (MMPlayerGstElement*) g_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_M_NUM);
+       mainbin = (MMPlayerGstElement *)g_try_malloc0(sizeof(MMPlayerGstElement) * MMPLAYER_M_NUM);
        if (mainbin == NULL)
                goto INIT_ERROR;
 
@@ -3948,10 +3960,9 @@ __mmplayer_gst_create_pipeline(mm_player_t* player)
        }
 
        /* Note : check whether subtitle atrribute uri is set. If uri is set, then try to play subtitle file */
-       if (__mmplayer_check_subtitle(player)) {
-               if (__mmplayer_gst_create_text_pipeline(player) != MM_ERROR_NONE)
-                       LOGE("failed to create text pipeline");
-       }
+       if (__mmplayer_check_subtitle(player)
+               && (__mmplayer_gst_create_text_pipeline(player) != MM_ERROR_NONE))
+               LOGE("failed to create text pipeline");
 
        /* add bus watch */
        ret = __mmplayer_gst_add_bus_watch(player);
@@ -3969,7 +3980,7 @@ INIT_ERROR:
 }
 
 static void
-__mmplayer_reset_gapless_state(mm_player_tplayer)
+__mmplayer_reset_gapless_state(mm_player_t *player)
 {
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(player
@@ -3984,7 +3995,7 @@ __mmplayer_reset_gapless_state(mm_player_t* player)
 }
 
 static int
-__mmplayer_gst_destroy_pipeline(mm_player_tplayer)
+__mmplayer_gst_destroy_pipeline(mm_player_t *player)
 {
        gint timeout = 0;
        int ret = MM_ERROR_NONE;
@@ -4021,16 +4032,16 @@ __mmplayer_gst_destroy_pipeline(mm_player_t* player)
 
        /* cleanup gst stuffs */
        if (player->pipeline) {
-               MMPlayerGstElementmainbin = player->pipeline->mainbin;
-               GstTagListtag_list = player->pipeline->tag_list;
+               MMPlayerGstElement *mainbin = player->pipeline->mainbin;
+               GstTagList *tag_list = player->pipeline->tag_list;
 
                /* first we need to disconnect all signal hander */
                __mmplayer_release_signal_connection(player, MM_PLAYER_SIGNAL_TYPE_ALL);
 
                if (mainbin) {
-                       MMPlayerGstElementaudiobin = player->pipeline->audiobin;
-                       MMPlayerGstElementvideobin = player->pipeline->videobin;
-                       MMPlayerGstElementtextbin = player->pipeline->textbin;
+                       MMPlayerGstElement *audiobin = player->pipeline->audiobin;
+                       MMPlayerGstElement *videobin = player->pipeline->videobin;
+                       MMPlayerGstElement *textbin = player->pipeline->textbin;
                        GstBus *bus = gst_pipeline_get_bus(GST_PIPELINE(mainbin[MMPLAYER_M_PIPE].gst));
                        gst_bus_set_sync_handler(bus, NULL, NULL, NULL);
                        gst_object_unref(bus);
@@ -4038,11 +4049,11 @@ __mmplayer_gst_destroy_pipeline(mm_player_t* player)
                        timeout = MMPLAYER_STATE_CHANGE_TIMEOUT(player);
                        ret = __mmplayer_gst_set_state(player, mainbin[MMPLAYER_M_PIPE].gst, GST_STATE_NULL, FALSE, timeout);
                        if (ret != MM_ERROR_NONE) {
-                               LOGE("fail to change state to NULL\n");
+                               LOGE("fail to change state to NULL");
                                return MM_ERROR_PLAYER_INTERNAL;
                        }
 
-                       LOGW("succeeded in changing state to NULL\n");
+                       LOGW("succeeded in changing state to NULL");
 
                        gst_object_unref(GST_OBJECT(mainbin[MMPLAYER_M_PIPE].gst));
 
@@ -4071,6 +4082,7 @@ __mmplayer_gst_destroy_pipeline(mm_player_t* player)
                gst_caps_unref(player->v_stream_caps);
                player->v_stream_caps = NULL;
        }
+
        if (player->a_stream_caps) {
                gst_caps_unref(player->a_stream_caps);
                player->a_stream_caps = NULL;
@@ -4091,14 +4103,15 @@ __mmplayer_gst_destroy_pipeline(mm_player_t* player)
                player->bufmgr = NULL;
        }
 
-       LOGW("finished destroy pipeline\n");
+       LOGW("finished destroy pipeline");
 
        MMPLAYER_FLEAVE();
 
        return ret;
 }
 
-static int __mmplayer_gst_realize(mm_player_t* player)
+static int
+__mmplayer_gst_realize(mm_player_t *player)
 {
        gint timeout = 0;
        int ret = MM_ERROR_NONE;
@@ -4111,7 +4124,7 @@ static int __mmplayer_gst_realize(mm_player_t* player)
 
        ret = __mmplayer_gst_create_pipeline(player);
        if (ret) {
-               LOGE("failed to create pipeline\n");
+               LOGE("failed to create pipeline");
                return ret;
        }
 
@@ -4137,7 +4150,8 @@ static int __mmplayer_gst_realize(mm_player_t* player)
        return ret;
 }
 
-static int __mmplayer_gst_unrealize(mm_player_t* player)
+static int
+__mmplayer_gst_unrealize(mm_player_t *player)
 {
        int ret = MM_ERROR_NONE;
 
@@ -4154,7 +4168,7 @@ static int __mmplayer_gst_unrealize(mm_player_t* player)
        /* destroy pipeline */
        ret = __mmplayer_gst_destroy_pipeline(player);
        if (ret != MM_ERROR_NONE) {
-               LOGE("failed to destory pipeline\n");
+               LOGE("failed to destory pipeline");
                return ret;
        }
 
@@ -4170,26 +4184,27 @@ static int __mmplayer_gst_unrealize(mm_player_t* player)
 }
 
 static int
-__mmplayer_gst_set_message_callback(mm_player_tplayer, MMMessageCallback callback, gpointer user_param)
+__mmplayer_gst_set_message_callback(mm_player_t *player, MMMessageCallback callback, gpointer user_param)
 {
        MMPLAYER_FENTER();
 
        if (!player) {
-               LOGW("set_message_callback is called with invalid player handle\n");
+               LOGW("set_message_callback is called with invalid player handle");
                return MM_ERROR_PLAYER_NOT_INITIALIZED;
        }
 
        player->msg_cb = callback;
        player->msg_cb_param = user_param;
 
-       LOGD("msg_cb : %p     msg_cb_param : %p\n", callback, user_param);
+       LOGD("msg_cb : %p     msg_cb_param : %p", callback, user_param);
 
        MMPLAYER_FLEAVE();
 
        return MM_ERROR_NONE;
 }
 
-int __mmplayer_parse_profile(const char *uri, void *param, MMPlayerParseProfile* data)
+int
+__mmplayer_parse_profile(const char *uri, void *param, MMPlayerParseProfile *data)
 {
        int ret = MM_ERROR_NONE;
        char *path = NULL;
@@ -4228,8 +4243,8 @@ int __mmplayer_parse_profile(const char *uri, void *param, MMPlayerParseProfile*
                ret = MM_ERROR_PLAYER_PERMISSION_DENIED;
 
        /* dump parse result */
-       SECURE_LOGW("incoming uri : %s\n", uri);
-       LOGD("uri_type : %d, mem : %p, mem_size : %d, urgent : %s\n",
+       SECURE_LOGW("incoming uri : %s", uri);
+       LOGD("uri_type : %d, mem : %p, mem_size : %d, urgent : %s",
                data->uri_type, data->input_mem.buf, data->input_mem.len, data->urgent);
 
        MMPLAYER_FLEAVE();
@@ -4295,7 +4310,7 @@ __resource_release_cb(mm_resource_manager_h rm, mm_resource_manager_res_h res,
        MMPLAYER_FENTER();
 
        if (user_data == NULL) {
-               LOGE("- user_data is null\n");
+               LOGE("- user_data is null");
                return FALSE;
        }
        player = (mm_player_t *)user_data;
@@ -4351,7 +4366,7 @@ _mmplayer_create_player(MMHandleType handle)
        int ret = MM_ERROR_PLAYER_INTERNAL;
        bool enabled = false;
 
-       mm_player_tplayer = MM_PLAYER_CAST(handle);
+       mm_player_t *player = MM_PLAYER_CAST(handle);
 
        MMPLAYER_FENTER();
 
@@ -4370,13 +4385,13 @@ _mmplayer_create_player(MMHandleType handle)
        player->attrs = _mmplayer_construct_attribute(handle);
 
        if (!player->attrs) {
-               LOGE("Failed to construct attributes\n");
+               LOGE("Failed to construct attributes");
                return ret;
        }
 
        /* initialize gstreamer with configured parameter */
        if (!__mmplayer_init_gstreamer(player)) {
-               LOGE("Initializing gstreamer failed\n");
+               LOGE("Initializing gstreamer failed");
                _mmplayer_deconstruct_attribute(handle);
                return ret;
        }
@@ -4413,7 +4428,7 @@ _mmplayer_create_player(MMHandleType handle)
 
        ret = _mmplayer_initialize_video_capture(player);
        if (ret != MM_ERROR_NONE) {
-               LOGE("failed to initialize video capture\n");
+               LOGE("failed to initialize video capture");
                goto ERROR;
        }
 
@@ -4493,12 +4508,9 @@ _mmplayer_create_player(MMHandleType handle)
 ERROR:
        /* free lock */
        g_mutex_clear(&player->fsink_lock);
-
        /* free update tag lock */
        g_mutex_clear(&player->update_tag_lock);
-
        g_queue_free(player->bus_msg_q);
-
        /* free gapless play thread */
        if (player->gapless_play_thread) {
                MMPLAYER_GAPLESS_PLAY_THREAD_LOCK(player);
@@ -4522,32 +4534,32 @@ ERROR:
 }
 
 static gboolean
-__mmplayer_init_gstreamer(mm_player_tplayer)
+__mmplayer_init_gstreamer(mm_player_t *player)
 {
        static gboolean initialized = FALSE;
        static const int max_argc = 50;
-       gintargc = NULL;
-       gchar** argv = NULL;
-       gchar** argv2 = NULL;
+       gint *argc = NULL;
+       gchar **argv = NULL;
+       gchar **argv2 = NULL;
        GError *err = NULL;
        int i = 0;
        int arg_count = 0;
 
        if (initialized) {
-               LOGD("gstreamer already initialized.\n");
+               LOGD("gstreamer already initialized.");
                return TRUE;
        }
 
        /* alloc */
        argc = malloc(sizeof(int));
-       argv = malloc(sizeof(gchar*) * max_argc);
-       argv2 = malloc(sizeof(gchar*) * max_argc);
+       argv = malloc(sizeof(gchar *) * max_argc);
+       argv2 = malloc(sizeof(gchar *) * max_argc);
 
        if (!argc || !argv || !argv2)
                goto ERROR;
 
-       memset(argv, 0, sizeof(gchar*) * max_argc);
-       memset(argv2, 0, sizeof(gchar*) * max_argc);
+       memset(argv, 0, sizeof(gchar *) * max_argc);
+       memset(argv2, 0, sizeof(gchar *) * max_argc);
 
        /* add initial */
        *argc = 1;
@@ -4578,18 +4590,18 @@ __mmplayer_init_gstreamer(mm_player_t* player)
                (*argc)++;
        }
 
-       LOGD("initializing gstreamer with following parameter\n");
-       LOGD("argc : %d\n", *argc);
+       LOGD("initializing gstreamer with following parameter");
+       LOGD("argc : %d", *argc);
        arg_count = *argc;
 
        for (i = 0; i < arg_count; i++) {
                argv2[i] = argv[i];
-               LOGD("argv[%d] : %s\n", i, argv2[i]);
+               LOGD("argv[%d] : %s", i, argv2[i]);
        }
 
        /* initializing gstreamer */
        if (!gst_init_check(argc, &argv, &err)) {
-               LOGE("Could not initialize GStreamer: %s\n", err ? err->message : "unknown error occurred");
+               LOGE("Could not initialize GStreamer: %s", err ? err->message : "unknown error occurred");
                if (err)
                        g_error_free(err);
 
@@ -4597,7 +4609,7 @@ __mmplayer_init_gstreamer(mm_player_t* player)
        }
        /* release */
        for (i = 0; i < arg_count; i++) {
-               //LOGD("release - argv[%d] : %s\n", i, argv2[i]);
+               //LOGD("release - argv[%d] : %s", i, argv2[i]);
                MMPLAYER_FREEIF(argv2[i]);
        }
 
@@ -4614,7 +4626,7 @@ ERROR:
 
        /* release */
        for (i = 0; i < arg_count; i++) {
-               LOGD("free[%d] : %s\n", i, argv2[i]);
+               LOGD("free[%d] : %s", i, argv2[i]);
                MMPLAYER_FREEIF(argv2[i]);
        }
 
@@ -4626,12 +4638,12 @@ ERROR:
 }
 
 static void
-__mmplayer_check_async_state_transition(mm_player_tplayer)
+__mmplayer_check_async_state_transition(mm_player_t *player)
 {
        GstState element_state = GST_STATE_VOID_PENDING;
        GstState element_pending_state = GST_STATE_VOID_PENDING;
        GstStateChangeReturn ret = GST_STATE_CHANGE_FAILURE;
-       GstElement * element = NULL;
+       GstElement *element = NULL;
        gboolean async = FALSE;
 
        /* check player handle */
@@ -4652,10 +4664,10 @@ __mmplayer_check_async_state_transition(mm_player_t* player)
 
        /* wait for state transition */
        element = player->pipeline->mainbin[MMPLAYER_M_PIPE].gst;
-       ret = gst_element_get_state(element, &element_state, &element_pending_state, 1*GST_SECOND);
+       ret = gst_element_get_state(element, &element_state, &element_pending_state, 1 * GST_SECOND);
 
        if (ret == GST_STATE_CHANGE_FAILURE) {
-               LOGE(" [%s] state : %s   pending : %s \n",
+               LOGE(" [%s] state : %s   pending : %s",
                        GST_ELEMENT_NAME(element),
                        gst_element_state_get_name(element_state),
                        gst_element_state_get_name(element_pending_state));
@@ -4666,14 +4678,14 @@ __mmplayer_check_async_state_transition(mm_player_t* player)
                return;
        }
 
-       LOGD("[%s] element state has changed\n", GST_ELEMENT_NAME(element));
+       LOGD("[%s] element state has changed", GST_ELEMENT_NAME(element));
        return;
 }
 
 int
 _mmplayer_destroy(MMHandleType handle)
 {
-       mm_player_tplayer = MM_PLAYER_CAST(handle);
+       mm_player_t *player = MM_PLAYER_CAST(handle);
 
        MMPLAYER_FENTER();
 
@@ -4693,11 +4705,11 @@ _mmplayer_destroy(MMHandleType handle)
                MMPLAYER_GAPLESS_PLAY_THREAD_SIGNAL(player);
                MMPLAYER_GAPLESS_PLAY_THREAD_UNLOCK(player);
 
-               LOGD("waitting for gapless play thread exit\n");
+               LOGD("waitting for gapless play thread exit");
                g_thread_join(player->gapless_play_thread);
                g_mutex_clear(&player->gapless_play_thread_mutex);
                g_cond_clear(&player->gapless_play_thread_cond);
-               LOGD("gapless play thread released\n");
+               LOGD("gapless play thread released");
        }
 
        _mmplayer_release_video_capture(player);
@@ -4705,11 +4717,11 @@ _mmplayer_destroy(MMHandleType handle)
        /* de-initialize resource manager */
        if (MM_RESOURCE_MANAGER_ERROR_NONE != mm_resource_manager_destroy(
                        player->resource_manager))
-               LOGE("failed to deinitialize resource manager\n");
+               LOGE("failed to deinitialize resource manager");
 
        /* release pipeline */
        if (MM_ERROR_NONE != __mmplayer_gst_destroy_pipeline(player)) {
-               LOGE("failed to destory pipeline\n");
+               LOGE("failed to destory pipeline");
                return MM_ERROR_PLAYER_INTERNAL;
        }
 
@@ -4752,7 +4764,7 @@ _mmplayer_destroy(MMHandleType handle)
 int
 _mmplayer_realize(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        char *uri = NULL;
        void *param = NULL;
        MMHandleType attrs = 0;
@@ -4768,14 +4780,14 @@ _mmplayer_realize(MMHandleType hplayer)
 
        attrs = MMPLAYER_GET_ATTRS(player);
        if (!attrs) {
-               LOGE("fail to get attributes.\n");
+               LOGE("fail to get attributes.");
                return MM_ERROR_PLAYER_INTERNAL;
        }
        mm_attrs_get_string_by_name(attrs, "profile_uri", &uri);
        mm_attrs_get_data_by_name(attrs, "profile_user_param", &param);
 
        if (player->profile.uri_type == MM_PLAYER_URI_TYPE_NONE) {
-               ret = __mmplayer_parse_profile((const char*)uri, param, &player->profile);
+               ret = __mmplayer_parse_profile((const char *)uri, param, &player->profile);
 
                if (ret != MM_ERROR_NONE) {
                        LOGE("failed to parse profile");
@@ -4791,7 +4803,7 @@ _mmplayer_realize(MMHandleType hplayer)
        }
 
        if (player->profile.uri_type == MM_PLAYER_URI_TYPE_URL_MMS) {
-               LOGW("mms protocol is not supported format.\n");
+               LOGW("mms protocol is not supported format.");
                return MM_ERROR_PLAYER_NOT_SUPPORTED_FORMAT;
        }
 
@@ -4825,7 +4837,7 @@ _mmplayer_realize(MMHandleType hplayer)
        /* realize pipeline */
        ret = __mmplayer_gst_realize(player);
        if (ret != MM_ERROR_NONE)
-               LOGE("fail to realize the player.\n");
+               LOGE("fail to realize the player.");
 
        MMPLAYER_BUS_MSG_THREAD_SIGNAL(player);
 
@@ -4837,7 +4849,7 @@ _mmplayer_realize(MMHandleType hplayer)
 int
 _mmplayer_unrealize(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -4866,7 +4878,7 @@ _mmplayer_unrealize(MMHandleType hplayer)
                                ret = mm_resource_manager_mark_for_release(player->resource_manager,
                                                player->video_decoder_resource);
                                if (ret != MM_RESOURCE_MANAGER_ERROR_NONE)
-                                       LOGE("failed to mark decoder resource for release, ret(0x%x)\n", ret);
+                                       LOGE("failed to mark decoder resource for release, ret(0x%x)", ret);
                                else
                                        player->video_decoder_resource = NULL;
                        }
@@ -4875,14 +4887,14 @@ _mmplayer_unrealize(MMHandleType hplayer)
                                ret = mm_resource_manager_mark_for_release(player->resource_manager,
                                                player->video_overlay_resource);
                                if (ret != MM_RESOURCE_MANAGER_ERROR_NONE)
-                                       LOGE("failed to mark overlay resource for release, ret(0x%x)\n", ret);
+                                       LOGE("failed to mark overlay resource for release, ret(0x%x)", ret);
                                else
                                        player->video_overlay_resource = NULL;
                        }
 
                        ret = mm_resource_manager_commit(player->resource_manager);
                        if (ret != MM_RESOURCE_MANAGER_ERROR_NONE)
-                               LOGE("failed to commit resource releases, ret(0x%x)\n", ret);
+                               LOGE("failed to commit resource releases, ret(0x%x)", ret);
                }
        } else
                LOGE("failed and don't change asm state to stop");
@@ -4895,7 +4907,7 @@ _mmplayer_unrealize(MMHandleType hplayer)
 int
 _mmplayer_set_message_callback(MMHandleType hplayer, MMMessageCallback callback, gpointer user_param)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
@@ -4903,9 +4915,9 @@ _mmplayer_set_message_callback(MMHandleType hplayer, MMMessageCallback callback,
 }
 
 int
-_mmplayer_get_state(MMHandleType hplayer, intstate)
+_mmplayer_get_state(MMHandleType hplayer, int *state)
 {
-       mm_player_t *player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(state, MM_ERROR_INVALID_ARGUMENT);
 
@@ -4918,21 +4930,21 @@ _mmplayer_get_state(MMHandleType hplayer, int* state)
 int
 _mmplayer_set_volume(MMHandleType hplayer, MMPlayerVolumeType volume)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
-       GstElementvol_element = NULL;
+       mm_player_t *player = (mm_player_t *)hplayer;
+       GstElement *vol_element = NULL;
        int i = 0;
 
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       LOGD("volume [L]=%f:[R]=%f\n",
+       LOGD("volume [L]=%f:[R]=%f",
                volume.level[MM_VOLUME_CHANNEL_LEFT], volume.level[MM_VOLUME_CHANNEL_RIGHT]);
 
        /* invalid factor range or not */
        for (i = 0; i < MM_VOLUME_CHANNEL_NUM; i++) {
                if (volume.level[i] < MM_VOLUME_FACTOR_MIN || volume.level[i] > MM_VOLUME_FACTOR_MAX) {
-                       LOGE("Invalid factor!(valid factor:0~1.0)\n");
+                       LOGE("Invalid factor!(valid factor:0~1.0)");
                        return MM_ERROR_INVALID_ARGUMENT;
                }
        }
@@ -4946,8 +4958,8 @@ _mmplayer_set_volume(MMHandleType hplayer, MMPlayerVolumeType volume)
 
        /* check pipeline handle */
        if (!player->pipeline || !player->pipeline->audiobin) {
-               LOGD("audiobin is not created yet\n");
-               LOGD("but, current stored volume will be set when it's created.\n");
+               LOGD("audiobin is not created yet");
+               LOGD("but, current stored volume will be set when it's created.");
 
                /* NOTE : stored volume will be used in create_audiobin
                 * returning MM_ERROR_NONE here makes application to able to
@@ -4960,7 +4972,7 @@ _mmplayer_set_volume(MMHandleType hplayer, MMPlayerVolumeType volume)
        vol_element = player->pipeline->audiobin[MMPLAYER_A_VOL].gst;
 
        if (vol_element) {
-               LOGD("volume is set [%f]\n", player->sound.volume);
+               LOGD("volume is set [%f]", player->sound.volume);
                g_object_set(vol_element, "volume", player->sound.volume, NULL);
        }
 
@@ -4969,11 +4981,10 @@ _mmplayer_set_volume(MMHandleType hplayer, MMPlayerVolumeType volume)
        return MM_ERROR_NONE;
 }
 
-
 int
-_mmplayer_get_volume(MMHandleType hplayer, MMPlayerVolumeTypevolume)
+_mmplayer_get_volume(MMHandleType hplayer, MMPlayerVolumeType *volume)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int i = 0;
 
        MMPLAYER_FENTER();
@@ -4993,8 +5004,8 @@ _mmplayer_get_volume(MMHandleType hplayer, MMPlayerVolumeType* volume)
 int
 _mmplayer_set_mute(MMHandleType hplayer, int mute)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
-       GstElementvol_element = NULL;
+       mm_player_t *player = (mm_player_t *)hplayer;
+       GstElement *vol_element = NULL;
 
        MMPLAYER_FENTER();
 
@@ -5002,7 +5013,7 @@ _mmplayer_set_mute(MMHandleType hplayer, int mute)
 
        /* mute value shoud 0 or 1 */
        if (mute != 0 && mute != 1) {
-               LOGE("bad mute value\n");
+               LOGE("bad mute value");
 
                /* FIXIT : definitly, we need _BAD_PARAM error code */
                return MM_ERROR_INVALID_ARGUMENT;
@@ -5012,7 +5023,7 @@ _mmplayer_set_mute(MMHandleType hplayer, int mute)
 
        /* just hold mute value if pipeline is not ready */
        if (!player->pipeline || !player->pipeline->audiobin) {
-               LOGD("pipeline is not ready. holding mute value\n");
+               LOGD("pipeline is not ready. holding mute value");
                return MM_ERROR_NONE;
        }
 
@@ -5020,10 +5031,10 @@ _mmplayer_set_mute(MMHandleType hplayer, int mute)
 
        /* NOTE : volume will only created when the bt is enabled */
        if (vol_element) {
-               LOGD("mute : %d\n", mute);
+               LOGD("mute : %d", mute);
                g_object_set(vol_element, "mute", mute, NULL);
        } else
-               LOGD("volume elemnet is not created. using volume in audiosink\n");
+               LOGD("volume elemnet is not created. using volume in audiosink");
 
        MMPLAYER_FLEAVE();
 
@@ -5031,9 +5042,9 @@ _mmplayer_set_mute(MMHandleType hplayer, int mute)
 }
 
 int
-_mmplayer_get_mute(MMHandleType hplayer, intpmute)
+_mmplayer_get_mute(MMHandleType hplayer, int *pmute)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -5042,7 +5053,7 @@ _mmplayer_get_mute(MMHandleType hplayer, int* pmute)
 
        /* just hold mute value if pipeline is not ready */
        if (!player->pipeline || !player->pipeline->audiobin) {
-               LOGD("pipeline is not ready. returning stored value\n");
+               LOGD("pipeline is not ready. returning stored value");
                *pmute = player->sound.mute;
                return MM_ERROR_NONE;
        }
@@ -5057,7 +5068,7 @@ _mmplayer_get_mute(MMHandleType hplayer, int* pmute)
 int
 _mmplayer_set_videostream_changed_cb(MMHandleType hplayer, mm_player_stream_changed_callback callback, void *user_param)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -5065,7 +5076,7 @@ _mmplayer_set_videostream_changed_cb(MMHandleType hplayer, mm_player_stream_chan
 
        player->video_stream_changed_cb = callback;
        player->video_stream_changed_cb_user_param = user_param;
-       LOGD("Handle value is %p : %p\n", player, player->video_stream_changed_cb);
+       LOGD("Handle value is %p : %p", player, player->video_stream_changed_cb);
 
        MMPLAYER_FLEAVE();
 
@@ -5075,7 +5086,7 @@ _mmplayer_set_videostream_changed_cb(MMHandleType hplayer, mm_player_stream_chan
 int
 _mmplayer_set_audiostream_changed_cb(MMHandleType hplayer, mm_player_stream_changed_callback callback, void *user_param)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -5083,7 +5094,7 @@ _mmplayer_set_audiostream_changed_cb(MMHandleType hplayer, mm_player_stream_chan
 
        player->audio_stream_changed_cb = callback;
        player->audio_stream_changed_cb_user_param = user_param;
-       LOGD("Handle value is %p : %p\n", player, player->audio_stream_changed_cb);
+       LOGD("Handle value is %p : %p", player, player->audio_stream_changed_cb);
 
        MMPLAYER_FLEAVE();
 
@@ -5093,7 +5104,7 @@ _mmplayer_set_audiostream_changed_cb(MMHandleType hplayer, mm_player_stream_chan
 int
 _mmplayer_set_audiostream_cb(MMHandleType hplayer, bool sync, mm_player_audio_stream_callback callback, void *user_param)
 {
-       mm_player_t *player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -5112,7 +5123,7 @@ _mmplayer_set_audiostream_cb(MMHandleType hplayer, bool sync, mm_player_audio_st
 int
 _mmplayer_set_videostream_cb(MMHandleType hplayer, mm_player_video_stream_callback callback, void *user_param)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -5125,7 +5136,7 @@ _mmplayer_set_videostream_cb(MMHandleType hplayer, mm_player_video_stream_callba
        player->video_stream_cb = callback;
        player->video_stream_cb_user_param = user_param;
 
-       LOGD("Stream cb Handle value is %p : %p, enable:%d\n", player, player->video_stream_cb, player->set_mode.media_packet_video_stream);
+       LOGD("Stream cb Handle value is %p : %p, enable:%d", player, player->video_stream_cb, player->set_mode.media_packet_video_stream);
 
        MMPLAYER_FLEAVE();
 
@@ -5135,7 +5146,7 @@ _mmplayer_set_videostream_cb(MMHandleType hplayer, mm_player_video_stream_callba
 int
 _mmplayer_start(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        gint ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -5148,7 +5159,7 @@ _mmplayer_start(MMHandleType hplayer)
        /* start pipeline */
        ret = __mmplayer_gst_start(player);
        if (ret != MM_ERROR_NONE)
-               LOGE("failed to start player.\n");
+               LOGE("failed to start player.");
 
        if ((player->streamer) && (player->streamer->buffering_state & MM_PLAYER_BUFFERING_IN_PROGRESS)) {
                LOGD("force playing start even during buffering");
@@ -5167,7 +5178,7 @@ _mmplayer_start(MMHandleType hplayer)
  * Because GST_ERROR_MESSAGE is posted by other plugin internally.
  */
 int
-__mmplayer_handle_missed_plugin(mm_player_tplayer)
+__mmplayer_handle_missed_plugin(mm_player_t *player)
 {
        MMMessageParamType msg_param;
        memset(&msg_param, 0, sizeof(MMMessageParamType));
@@ -5177,7 +5188,7 @@ __mmplayer_handle_missed_plugin(mm_player_t* player)
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       LOGD("not_supported_codec = 0x%02x, can_support_codec = 0x%02x\n",
+       LOGD("not_supported_codec = 0x%02x, can_support_codec = 0x%02x",
                        player->not_supported_codec, player->can_support_codec);
 
        if (player->not_found_demuxer) {
@@ -5204,12 +5215,12 @@ __mmplayer_handle_missed_plugin(mm_player_t* player)
                        memset(&msg_param, 0, sizeof(MMMessageParamType));
 
                        if (player->not_supported_codec ==  MISSING_PLUGIN_AUDIO) {
-                               LOGW("not found AUDIO codec, posting error code to application.\n");
+                               LOGW("not found AUDIO codec, posting error code to application.");
 
                                msg_param.code = MM_ERROR_PLAYER_AUDIO_CODEC_NOT_FOUND;
                                msg_param.data = g_strdup_printf("%s", player->unlinked_audio_mime);
                        } else if (player->not_supported_codec ==  MISSING_PLUGIN_VIDEO) {
-                               LOGW("not found VIDEO codec, posting error code to application.\n");
+                               LOGW("not found VIDEO codec, posting error code to application.");
 
                                msg_param.code = MM_ERROR_PLAYER_VIDEO_CODEC_NOT_FOUND;
                                msg_param.data = g_strdup_printf("%s", player->unlinked_video_mime);
@@ -5222,7 +5233,7 @@ __mmplayer_handle_missed_plugin(mm_player_t* player)
                        return MM_ERROR_NONE;
                } else {
                        // no any supported codec case
-                       LOGW("not found any codec, posting error code to application.\n");
+                       LOGW("not found any codec, posting error code to application.");
 
                        if (player->not_supported_codec ==  MISSING_PLUGIN_AUDIO) {
                                msg_param.code = MM_ERROR_PLAYER_AUDIO_CODEC_NOT_FOUND;
@@ -5243,34 +5254,35 @@ __mmplayer_handle_missed_plugin(mm_player_t* player)
        return MM_ERROR_NONE;
 }
 
-static void __mmplayer_check_pipeline(mm_player_t* player)
+static void
+__mmplayer_check_pipeline(mm_player_t *player)
 {
        GstState element_state = GST_STATE_VOID_PENDING;
        GstState element_pending_state = GST_STATE_VOID_PENDING;
        gint timeout = 0;
        int ret = MM_ERROR_NONE;
 
-       if (player->gapless.reconfigure) {
-               LOGW("pipeline is under construction.\n");
+       if (!player->gapless.reconfigure)
+               return;
 
-               MMPLAYER_PLAYBACK_LOCK(player);
-               MMPLAYER_PLAYBACK_UNLOCK(player);
+       LOGW("pipeline is under construction.");
 
-               timeout = MMPLAYER_STATE_CHANGE_TIMEOUT(player);
+       MMPLAYER_PLAYBACK_LOCK(player);
+       MMPLAYER_PLAYBACK_UNLOCK(player);
 
-               /* wait for state transition */
-               ret = gst_element_get_state(player->pipeline->mainbin[MMPLAYER_M_PIPE].gst, &element_state, &element_pending_state, timeout * GST_SECOND);
+       timeout = MMPLAYER_STATE_CHANGE_TIMEOUT(player);
 
-               if (ret == GST_STATE_CHANGE_FAILURE)
-                       LOGE("failed to change pipeline state within %d sec\n", timeout);
-       }
+       /* wait for state transition */
+       ret = gst_element_get_state(player->pipeline->mainbin[MMPLAYER_M_PIPE].gst, &element_state, &element_pending_state, timeout * GST_SECOND);
+       if (ret == GST_STATE_CHANGE_FAILURE)
+               LOGE("failed to change pipeline state within %d sec", timeout);
 }
 
 /* NOTE : it should be able to call 'stop' anytime*/
 int
 _mmplayer_stop(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -5294,7 +5306,7 @@ _mmplayer_stop(MMHandleType hplayer)
        ret = __mmplayer_gst_stop(player);
 
        if (ret != MM_ERROR_NONE)
-               LOGE("failed to stop player.\n");
+               LOGE("failed to stop player.");
 
        MMPLAYER_FLEAVE();
 
@@ -5304,7 +5316,7 @@ _mmplayer_stop(MMHandleType hplayer)
 int
 _mmplayer_pause(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        gint64 pos_nsec = 0;
        gboolean async = FALSE;
        gint ret = MM_ERROR_NONE;
@@ -5343,7 +5355,7 @@ _mmplayer_pause(MMHandleType hplayer)
                        * elements
                        */
                        if (!gst_element_query_position(player->pipeline->mainbin[MMPLAYER_M_PIPE].gst, GST_FORMAT_TIME, &pos_nsec))
-                               LOGW("getting current position failed in paused\n");
+                               LOGW("getting current position failed in paused");
 
                        player->last_position = pos_nsec;
 
@@ -5368,7 +5380,7 @@ _mmplayer_pause(MMHandleType hplayer)
        ret = __mmplayer_gst_pause(player, async);
 
        if (ret != MM_ERROR_NONE)
-               LOGE("failed to pause player. ret : 0x%x\n", ret);
+               LOGE("failed to pause player. ret : 0x%x", ret);
 
        if (MMPLAYER_PREV_STATE(player) == MM_PLAYER_STATE_READY && MMPLAYER_CURRENT_STATE(player) == MM_PLAYER_STATE_PAUSED) {
                if (MM_ERROR_NONE != _mmplayer_update_video_param(player, "display_rotation"))
@@ -5384,7 +5396,7 @@ _mmplayer_pause(MMHandleType hplayer)
 int
 _mmplayer_abort_pause(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -5408,11 +5420,10 @@ _mmplayer_abort_pause(MMHandleType hplayer)
        return ret;
 }
 
-
 int
 _mmplayer_resume(MMHandleType hplayer)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
        gboolean async = FALSE;
 
@@ -5436,7 +5447,7 @@ _mmplayer_resume(MMHandleType hplayer)
 
        ret = __mmplayer_gst_resume(player, async);
        if (ret != MM_ERROR_NONE)
-               LOGE("failed to resume player.\n");
+               LOGE("failed to resume player.");
 
        if ((player->streamer) && (player->streamer->buffering_state & MM_PLAYER_BUFFERING_IN_PROGRESS)) {
                LOGD("force resume even during buffering");
@@ -5451,7 +5462,7 @@ _mmplayer_resume(MMHandleType hplayer)
 int
 _mmplayer_set_playspeed(MMHandleType hplayer, float rate, bool streaming)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        gint64 pos_nsec = 0;
        int ret = MM_ERROR_NONE;
        int mute = FALSE;
@@ -5474,7 +5485,7 @@ _mmplayer_set_playspeed(MMHandleType hplayer, float rate, bool streaming)
 
        /* If the position is reached at start potion during fast backward, EOS is posted.
         * So, This EOS have to be classified with it which is posted at reaching the end of stream.
-        * */
+        */
        player->playback_rate = rate;
 
        current_state = MMPLAYER_CURRENT_STATE(player);
@@ -5504,11 +5515,11 @@ _mmplayer_set_playspeed(MMHandleType hplayer, float rate, bool streaming)
                                (GST_SEEK_FLAG_FLUSH | GST_SEEK_FLAG_ACCURATE),
                                GST_SEEK_TYPE_SET, start,
                                GST_SEEK_TYPE_SET, stop)) {
-               LOGE("failed to set speed playback\n");
+               LOGE("failed to set speed playback");
                return MM_ERROR_PLAYER_SEEK;
        }
 
-       LOGD("succeeded to set speed playback as %0.1f\n", rate);
+       LOGD("succeeded to set speed playback as %0.1f", rate);
 
        MMPLAYER_FLEAVE();
 
@@ -5518,7 +5529,7 @@ _mmplayer_set_playspeed(MMHandleType hplayer, float rate, bool streaming)
 int
 _mmplayer_set_position(MMHandleType hplayer, gint64 position)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -5538,7 +5549,7 @@ _mmplayer_set_position(MMHandleType hplayer, gint64 position)
 int
 _mmplayer_get_position(MMHandleType hplayer, gint64 *position)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -5551,7 +5562,7 @@ _mmplayer_get_position(MMHandleType hplayer, gint64 *position)
 int
 _mmplayer_get_duration(MMHandleType hplayer, gint64 *duration)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -5564,7 +5575,7 @@ _mmplayer_get_duration(MMHandleType hplayer, gint64 *duration)
 int
 _mmplayer_get_buffer_position(MMHandleType hplayer, int *start_pos, int *end_pos)
 {
-       mm_player_t *player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -5577,7 +5588,7 @@ _mmplayer_get_buffer_position(MMHandleType hplayer, int *start_pos, int *end_pos
 int
 _mmplayer_adjust_subtitle_postion(MMHandleType hplayer, int format, int position)
 {
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -5592,7 +5603,7 @@ _mmplayer_adjust_subtitle_postion(MMHandleType hplayer, int format, int position
 }
 
 static gboolean
-__mmplayer_is_midi_type(gcharstr_caps)
+__mmplayer_is_midi_type(gchar *str_caps)
 {
        if ((g_strrstr(str_caps, "audio/midi")) ||
                (g_strrstr(str_caps, "application/x-gst_ff-mmf")) ||
@@ -5601,7 +5612,7 @@ __mmplayer_is_midi_type(gchar* str_caps)
                (g_strrstr(str_caps, "audio/mobile-xmf")) ||
                (g_strrstr(str_caps, "audio/xmf")) ||
                (g_strrstr(str_caps, "audio/mxmf"))) {
-               LOGD("midi\n");
+               LOGD("midi");
                return TRUE;
        }
 
@@ -5618,9 +5629,9 @@ __mmplayer_is_only_mp3_type(gchar *str_caps)
 }
 
 static void
-__mmplayer_set_audio_attrs(mm_player_t* player, GstCaps* caps)
+__mmplayer_set_audio_attrs(mm_player_t *player, GstCaps *caps)
 {
-       GstStructurecaps_structure = NULL;
+       GstStructure *caps_structure = NULL;
        gint samplerate = 0;
        gint channels = 0;
 
@@ -5636,11 +5647,11 @@ __mmplayer_set_audio_attrs(mm_player_t* player, GstCaps* caps)
        gst_structure_get_int(caps_structure, "channels", &channels);
        mm_attrs_set_int_by_name(player->attrs, "content_audio_channels", channels);
 
-       LOGD("audio samplerate : %d     channels : %d\n", samplerate, channels);
+       LOGD("audio samplerate : %d     channels : %d", samplerate, channels);
 }
 
 static void
-__mmplayer_update_content_type_info(mm_player_tplayer)
+__mmplayer_update_content_type_info(mm_player_t *player)
 {
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(player && player->type);
@@ -5671,10 +5682,10 @@ __mmplayer_update_content_type_info(mm_player_t* player)
 
 void
 __mmplayer_typefind_have_type(GstElement *tf, guint probability,
-GstCaps *caps, gpointer data)
+       GstCaps *caps, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
-       GstPadpad = NULL;
+       mm_player_t *player = (mm_player_t *)data;
+       GstPad *pad = NULL;
 
        MMPLAYER_FENTER();
 
@@ -5683,14 +5694,13 @@ GstCaps *caps, gpointer data)
        /* store type string */
        MMPLAYER_FREEIF(player->type);
        player->type = gst_caps_to_string(caps);
-       if (player->type) {
-               LOGD("[handle: %p] media type %s found, probability %d%% / %d\n",
+       if (player->type)
+               LOGD("[handle: %p] media type %s found, probability %d%% / %d",
                                player, player->type, probability, gst_caps_get_size(caps));
-       }
 
        if ((!MMPLAYER_IS_RTSP_STREAMING(player)) &&
                (g_strrstr(player->type, "audio/x-raw-int"))) {
-               LOGE("not support media format\n");
+               LOGE("not support media format");
 
                if (player->msg_posted == FALSE) {
                        MMMessageParamType msg_param;
@@ -5709,13 +5719,13 @@ GstCaps *caps, gpointer data)
 
        pad = gst_element_get_static_pad(tf, "src");
        if (!pad) {
-               LOGE("fail to get typefind src pad.\n");
+               LOGE("fail to get typefind src pad.");
                return;
        }
 
        if (!__mmplayer_gst_create_decoder(player, pad, caps)) {
                gboolean async = FALSE;
-               LOGE("failed to autoplug %s\n", player->type);
+               LOGE("failed to autoplug %s", player->type);
 
                mm_attrs_get_int_by_name(player->attrs, "profile_prepare_async", &async);
 
@@ -5734,7 +5744,7 @@ DONE:
 }
 
 GstElement *
-__mmplayer_gst_make_decodebin(mm_player_tplayer)
+__mmplayer_gst_make_decodebin(mm_player_t *player)
 {
        GstElement *decodebin = NULL;
 
@@ -5744,7 +5754,7 @@ __mmplayer_gst_make_decodebin(mm_player_t* player)
        decodebin = gst_element_factory_make("decodebin", NULL);
 
        if (!decodebin) {
-               LOGE("fail to create decodebin\n");
+               LOGE("fail to create decodebin");
                goto ERROR;
        }
 
@@ -5786,10 +5796,10 @@ ERROR:
        return decodebin;
 }
 
-static GstElement*
+static GstElement *
 __mmplayer_gst_make_queue2(mm_player_t *player)
 {
-       GstElementqueue2 = NULL;
+       GstElement *queue2 = NULL;
        gint64 dur_bytes = 0L;
        guint max_buffer_size_bytes = 0;
        MMPlayerGstElement *mainbin = NULL;
@@ -5825,7 +5835,7 @@ __mmplayer_gst_make_queue2(mm_player_t *player)
        /* NOTE : in case of ts streaming, player cannot get the correct duration info *
         *                skip the pull mode(file or ring buffering) setting. */
        if (!g_strrstr(player->type, "video/mpegts")) {
-               max_buffer_size_bytes = (type == MUXED_BUFFER_TYPE_FILE) ? (player->ini.http_max_size_bytes) : (5*1024*1024);
+               max_buffer_size_bytes = (type == MUXED_BUFFER_TYPE_FILE) ? (player->ini.http_max_size_bytes) : (5 * 1024 * 1024);
                LOGD("max_buffer_size_bytes = %d", max_buffer_size_bytes);
 
                __mm_player_streaming_set_queue2(player->streamer,
@@ -5842,13 +5852,13 @@ __mmplayer_gst_make_queue2(mm_player_t *player)
 }
 
 gboolean
-__mmplayer_gst_create_decoder(mm_player_tplayer, GstPad *srcpad, const GstCaps *caps)
+__mmplayer_gst_create_decoder(mm_player_t *player, GstPad *srcpad, const GstCaps *caps)
 {
-       MMPlayerGstElementmainbin = NULL;
-       GstElementdecodebin = NULL;
-       GstElementqueue2 = NULL;
-       GstPadsinkpad = NULL;
-       GstPadqsrcpad = NULL;
+       MMPlayerGstElement *mainbin = NULL;
+       GstElement *decodebin = NULL;
+       GstElement *queue2 = NULL;
+       GstPad *sinkpad = NULL;
+       GstPad *qsrcpad = NULL;
        gint init_buffering_time = player->streamer->buffering_req.prebuffer_time;
 
        MMPLAYER_FENTER();
@@ -5904,7 +5914,7 @@ __mmplayer_gst_create_decoder(mm_player_t* player, GstPad *srcpad, const GstCaps
        }
 
        if (!gst_bin_add(GST_BIN(mainbin[MMPLAYER_M_PIPE].gst), decodebin)) {
-               LOGE("failed to add decodebin\n");
+               LOGE("failed to add decodebin");
                goto ERROR;
        }
 
@@ -5933,7 +5943,7 @@ __mmplayer_gst_create_decoder(mm_player_t* player, GstPad *srcpad, const GstCaps
        if (MMPLAYER_IS_HTTP_STREAMING(player) || MMPLAYER_IS_HTTP_LIVE_STREAMING(player) || MMPLAYER_IS_DASH_STREAMING(player)) {
            gdouble high_percent = 0.0;
 
-               init_buffering_time = (init_buffering_time != 0) ? (init_buffering_time) : (player->ini.http_buffering_time);
+               init_buffering_time = (init_buffering_time != 0) ? init_buffering_time : player->ini.http_buffering_time;
                high_percent = (gdouble)(init_buffering_time * 100) / GET_MAX_BUFFER_TIME(player->streamer);
 
                LOGD("decodebin setting - bytes: %d, time: %d ms, per: 1~%d",
@@ -5947,7 +5957,7 @@ __mmplayer_gst_create_decoder(mm_player_t* player, GstPad *srcpad, const GstCaps
        }
 
        if (GST_STATE_CHANGE_FAILURE == gst_element_sync_state_with_parent(decodebin)) {
-               LOGE("failed to sync decodebin state with parent\n");
+               LOGE("failed to sync decodebin state with parent");
                goto ERROR;
        }
 
@@ -5995,14 +6005,14 @@ ERROR:
 }
 
 static int
-__mmplayer_check_not_supported_codec(mm_player_t* player, const gchar* factory_class, const gchar* mime)
+__mmplayer_check_not_supported_codec(mm_player_t *player, const gchar *factory_class, const gchar *mime)
 {
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline, MM_ERROR_PLAYER_NOT_INITIALIZED);
        MMPLAYER_RETURN_VAL_IF_FAIL(mime, MM_ERROR_INVALID_ARGUMENT);
 
-       LOGD("class : %s, mime : %s \n", factory_class, mime);
+       LOGD("class : %s, mime : %s", factory_class, mime);
 
        /* add missing plugin */
        /* NOTE : msl should check missing plugin for image mime type.
@@ -6011,7 +6021,7 @@ __mmplayer_check_not_supported_codec(mm_player_t* player, const gchar* factory_c
         */
        if (!(player->pipeline->mainbin[MMPLAYER_M_DEMUX].gst)) {
                if (!(player->can_support_codec | player->videodec_linked | player->audiodec_linked)) {
-                       LOGD("not found demuxer\n");
+                       LOGD("not found demuxer");
                        player->not_found_demuxer = TRUE;
                        player->unlinked_demuxer_mime = g_strdup_printf("%s", mime);
 
@@ -6021,22 +6031,22 @@ __mmplayer_check_not_supported_codec(mm_player_t* player, const gchar* factory_c
 
        if (!g_strrstr(factory_class, "Demuxer")) {
                if ((g_str_has_prefix(mime, "video")) || (g_str_has_prefix(mime, "image"))) {
-                       LOGD("can support codec=0x%X, vdec_linked=%d, adec_linked=%d\n",
+                       LOGD("can support codec=0x%X, vdec_linked=%d, adec_linked=%d",
                                player->can_support_codec, player->videodec_linked, player->audiodec_linked);
 
                        /* check that clip have multi tracks or not */
                        if ((player->can_support_codec & FOUND_PLUGIN_VIDEO) && (player->videodec_linked)) {
-                               LOGD("video plugin is already linked\n");
+                               LOGD("video plugin is already linked");
                        } else {
-                               LOGW("add VIDEO to missing plugin\n");
+                               LOGW("add VIDEO to missing plugin");
                                player->not_supported_codec |= MISSING_PLUGIN_VIDEO;
                                player->unlinked_video_mime = g_strdup_printf("%s", mime);
                        }
                } else if (g_str_has_prefix(mime, "audio")) {
                        if ((player->can_support_codec & FOUND_PLUGIN_AUDIO) && (player->audiodec_linked)) {
-                               LOGD("audio plugin is already linked\n");
+                               LOGD("audio plugin is already linked");
                        } else {
-                               LOGW("add AUDIO to missing plugin\n");
+                               LOGW("add AUDIO to missing plugin");
                                player->not_supported_codec |= MISSING_PLUGIN_AUDIO;
                                player->unlinked_audio_mime = g_strdup_printf("%s", mime);
                        }
@@ -6049,11 +6059,10 @@ DONE:
        return MM_ERROR_NONE;
 }
 
-
 static void
 __mmplayer_pipeline_complete(GstElement *decodebin,  gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
 
        MMPLAYER_FENTER();
 
@@ -6118,12 +6127,12 @@ __mmplayer_get_next_uri(mm_player_t *player)
        LOGD("num of uri list = %d, current uri idx %d", num_of_list, uri_idx);
        for (uri_idx++; uri_idx < num_of_list; uri_idx++) {
                uri = g_list_nth_data(player->uri_info.uri_list, uri_idx);
-               if (uri == NULL) {
+               if (!uri) {
                        LOGW("next uri does not exist");
                        continue;
                }
 
-               if (__mmplayer_parse_profile((const char*)uri, NULL, &profile) != MM_ERROR_NONE) {
+               if (__mmplayer_parse_profile((const char *)uri, NULL, &profile) != MM_ERROR_NONE) {
                        LOGE("failed to parse profile");
                        continue;
                }
@@ -6173,13 +6182,13 @@ __mmplayer_verify_gapless_play_path(mm_player_t *player)
        LOGD("checking for gapless play option");
 
        if (player->pipeline->textbin) {
-               LOGE("subtitle path is enabled. gapless play is not supported.\n");
+               LOGE("subtitle path is enabled. gapless play is not supported.");
                goto ERROR;
        }
 
        attrs = MMPLAYER_GET_ATTRS(player);
        if (!attrs) {
-               LOGE("fail to get attributes.\n");
+               LOGE("fail to get attributes.");
                goto ERROR;
        }
 
@@ -6215,17 +6224,15 @@ __mmplayer_verify_gapless_play_path(mm_player_t *player)
                        /* decrease play count */
                        /* we succeeded to rewind. update play count and then wait for next EOS */
                        count--;
-
                        mm_attrs_set_int_by_name(attrs, "profile_play_count", count);
-
                        /* commit attribute */
                        if (mm_attrs_commit_all(attrs))
                                LOGE("failed to commit attribute");
+
                } else if (count != REPEAT_COUNT_INFINITELY) {
                        LOGD("there is no next uri and no repeat");
                        goto ERROR;
                }
-
                LOGD("looping cnt %d", count);
        } else {
                /* gapless playback path */
@@ -6234,7 +6241,6 @@ __mmplayer_verify_gapless_play_path(mm_player_t *player)
                        goto ERROR;
                }
        }
-
        return TRUE;
 
 ERROR:
@@ -6257,7 +6263,7 @@ __mmplayer_initialize_gapless_play(mm_player_t *player)
        player->is_external_subtitle_added_now = FALSE;
        player->not_supported_codec = MISSING_PLUGIN_NONE;
        player->can_support_codec = FOUND_PLUGIN_NONE;
-       player->pending_seek.is_pending = FALSE;
+       player->pending_seek.is_pending = false;
        player->pending_seek.pos = 0;
        player->msg_posted = FALSE;
        player->has_many_types = FALSE;
@@ -6342,7 +6348,7 @@ __mmplayer_activate_next_source(mm_player_t *player, GstState target)
 
        mm_attrs_get_string_by_name(attrs, "profile_uri", &uri);
 
-       if (__mmplayer_parse_profile((const char*)uri, NULL, &player->profile) != MM_ERROR_NONE) {
+       if (__mmplayer_parse_profile((const char *)uri, NULL, &player->profile) != MM_ERROR_NONE) {
                LOGE("failed to parse profile");
                msg_param.code = MM_ERROR_PLAYER_INVALID_URI;
                goto ERROR;
@@ -6577,46 +6583,49 @@ ERROR:
        return;
 }
 
-int _mmplayer_set_file_buffering_path(MMHandleType hplayer, const char* file_path)
+int
+_mmplayer_set_file_buffering_path(MMHandleType hplayer, const char *file_path)
 {
        int result = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        if (file_path) {
-               player->http_file_buffering_path = (gchar*)file_path;
-               LOGD("temp file path: %s\n", player->http_file_buffering_path);
+               player->http_file_buffering_path = (gchar *)file_path;
+               LOGD("temp file path: %s", player->http_file_buffering_path);
        }
        MMPLAYER_FLEAVE();
        return result;
 }
 
-int _mmplayer_set_uri(MMHandleType hplayer, const char* uri)
+int
+_mmplayer_set_uri(MMHandleType hplayer, const char *uri)
 {
        int result = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        mm_attrs_set_string_by_name(player->attrs, "profile_uri", uri);
        if (mm_attrs_commit_all(player->attrs)) {
-               LOGE("failed to commit the original uri.\n");
+               LOGE("failed to commit the original uri.");
                result = MM_ERROR_PLAYER_INTERNAL;
        } else {
                if (_mmplayer_set_next_uri(hplayer, uri, TRUE) != MM_ERROR_NONE)
-                       LOGE("failed to add the original uri in the uri list.\n");
+                       LOGE("failed to add the original uri in the uri list.");
        }
 
        MMPLAYER_FLEAVE();
        return result;
 }
 
-int _mmplayer_set_next_uri(MMHandleType hplayer, const char* uri, bool is_first_path)
+int
+_mmplayer_set_next_uri(MMHandleType hplayer, const char *uri, bool is_first_path)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        guint num_of_list = 0;
 
        MMPLAYER_FENTER();
@@ -6625,13 +6634,13 @@ int _mmplayer_set_next_uri(MMHandleType hplayer, const char* uri, bool is_first_
        MMPLAYER_RETURN_VAL_IF_FAIL(uri, MM_ERROR_INVALID_ARGUMENT);
 
        if (player->pipeline && player->pipeline->textbin) {
-               LOGE("subtitle path is enabled.\n");
+               LOGE("subtitle path is enabled.");
                return MM_ERROR_PLAYER_INVALID_STATE;
        }
 
        num_of_list = g_list_length(player->uri_info.uri_list);
 
-       if (is_first_path == TRUE) {
+       if (is_first_path) {
                if (num_of_list == 0) {
                        player->uri_info.uri_list = g_list_append(player->uri_info.uri_list, g_strdup(uri));
                        LOGD("add original path : %s", uri);
@@ -6671,9 +6680,10 @@ int _mmplayer_set_next_uri(MMHandleType hplayer, const char* uri, bool is_first_
        return MM_ERROR_NONE;
 }
 
-int _mmplayer_get_next_uri(MMHandleType hplayer, char** uri)
+int
+_mmplayer_get_next_uri(MMHandleType hplayer, char **uri)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        char *next_uri = NULL;
        guint num_of_list = 0;
 
@@ -6691,7 +6701,7 @@ int _mmplayer_get_next_uri(MMHandleType hplayer, char** uri)
                        uri_idx = 0;
 
                next_uri = g_list_nth_data(player->uri_info.uri_list, uri_idx);
-               LOGE("next uri idx : %d, uri = %s\n", uri_idx, next_uri);
+               LOGE("next uri idx : %d, uri = %s", uri_idx, next_uri);
 
                *uri = g_strdup(next_uri);
        }
@@ -6701,13 +6711,13 @@ int _mmplayer_get_next_uri(MMHandleType hplayer, char** uri)
 }
 
 static void
-__mmplayer_gst_decode_unknown_type(GstElement *elem,  GstPadpad,
-GstCaps *caps, gpointer data)
+__mmplayer_gst_decode_unknown_type(GstElement *elem,  GstPad *pad,
+       GstCaps *caps, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
-       const gcharklass = NULL;
-       const gcharmime = NULL;
-       gcharcaps_str = NULL;
+       mm_player_t *player = (mm_player_t *)data;
+       const gchar *klass = NULL;
+       const gchar *mime = NULL;
+       gchar *caps_str = NULL;
 
        klass = gst_element_factory_get_metadata(gst_element_get_factory(elem), GST_ELEMENT_METADATA_KLASS);
        mime = gst_structure_get_name(gst_caps_get_structure(caps, 0));
@@ -6723,18 +6733,18 @@ GstCaps *caps, gpointer data)
 }
 
 static gboolean
-__mmplayer_gst_decode_autoplug_continue(GstElement *bin,  GstPadpad,
-GstCaps * caps,  gpointer data)
+__mmplayer_gst_decode_autoplug_continue(GstElement *bin,  GstPad *pad,
+       GstCaps *caps,  gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
-       const charmime = NULL;
+       mm_player_t *player = (mm_player_t *)data;
+       const char *mime = NULL;
        gboolean ret = TRUE;
 
        MMPLAYER_LOG_GST_CAPS_TYPE(caps);
        mime = gst_structure_get_name(gst_caps_get_structure(caps, 0));
 
        if (g_str_has_prefix(mime, "audio")) {
-               GstStructurecaps_structure = NULL;
+               GstStructure *caps_structure = NULL;
                gint samplerate = 0;
                gint channels = 0;
                gchar *caps_str = NULL;
@@ -6771,7 +6781,7 @@ GstCaps * caps,  gpointer data)
 }
 
 static int
-__mmplayer_check_codec_info(mm_player_t* player, const char* klass, GstCaps* caps, char* factory_name)
+__mmplayer_check_codec_info(mm_player_t *player, const char *klass, GstCaps *caps, char *factory_name)
 {
        int ret = MM_ERROR_NONE;
        int idx = 0;
@@ -6856,7 +6866,7 @@ __mmplayer_check_codec_info(mm_player_t* player, const char* klass, GstCaps* cap
                        /* acquire resources for video playing */
                        if (mm_resource_manager_commit(player->resource_manager)
                                        != MM_RESOURCE_MANAGER_ERROR_NONE) {
-                               LOGE("could not acquire resources for video decoding\n");
+                               LOGE("could not acquire resources for video decoding");
                                ret = MM_ERROR_PLAYER_INTERNAL;
                                goto DONE;
                        }
@@ -6873,8 +6883,8 @@ DONE:
 }
 
 gint
-__mmplayer_gst_decode_autoplug_select(GstElement *bin,  GstPadpad,
-GstCaps* caps, GstElementFactory* factory, gpointer data)
+__mmplayer_gst_decode_autoplug_select(GstElement *bin,  GstPad *pad,
+       GstCaps *caps, GstElementFactory *factory, gpointer data)
 {
        /* NOTE : GstAutoplugSelectResult is defined in gstplay-enum.h but not exposed
         We are defining our own and will be removed when it actually exposed */
@@ -6885,11 +6895,11 @@ GstCaps* caps, GstElementFactory* factory, gpointer data)
        } GstAutoplugSelectResult;
 
        GstAutoplugSelectResult result = GST_AUTOPLUG_SELECT_TRY;
-       mm_player_t* player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
 
-       gcharfactory_name = NULL;
-       gcharcaps_str = NULL;
-       const gcharklass = NULL;
+       gchar *factory_name = NULL;
+       gchar *caps_str = NULL;
+       const gchar *klass = NULL;
        gint idx = 0;
 
        factory_name = GST_OBJECT_NAME(factory);
@@ -6942,7 +6952,7 @@ GstCaps* caps, GstElementFactory* factory, gpointer data)
         * So, those plugins should be skipped for error handling.
         */
        if (g_strrstr(klass, "Codec/Decoder/Image")) {
-               LOGD("skipping [%s] by not required\n", factory_name);
+               LOGD("skipping [%s] by not required", factory_name);
                result = GST_AUTOPLUG_SELECT_SKIP;
                goto DONE;
        }
@@ -6950,13 +6960,13 @@ GstCaps* caps, GstElementFactory* factory, gpointer data)
        if ((MMPLAYER_IS_MS_BUFF_SRC(player)) &&
                (g_strrstr(klass, "Codec/Demuxer") || (g_strrstr(klass, "Codec/Parser")))) {
                // TO CHECK : subtitle if needed, add subparse exception.
-               LOGD("skipping parser/demuxer [%s] in es player by not required\n", factory_name);
+               LOGD("skipping parser/demuxer [%s] in es player by not required", factory_name);
                result = GST_AUTOPLUG_SELECT_SKIP;
                goto DONE;
        }
 
        if (g_strrstr(factory_name, "mpegpsdemux")) {
-               LOGD("skipping PS container - not support\n");
+               LOGD("skipping PS container - not support");
                result = GST_AUTOPLUG_SELECT_SKIP;
                goto DONE;
        }
@@ -7011,30 +7021,33 @@ DONE:
 }
 
 static void
-__mmplayer_gst_decode_pad_removed(GstElement *elem,  GstPadnew_pad,
-gpointer data)
+__mmplayer_gst_decode_pad_removed(GstElement *elem,  GstPad *new_pad,
+       gpointer data)
 {
-       //mm_player_t* player = (mm_player_t*)data;
-       GstCapscaps = NULL;
+       //mm_player_t *player = (mm_player_t *)data;
+       GstCaps *caps = NULL;
 
-       LOGD("[Decodebin2] pad-removed signal\n");
+       LOGD("[Decodebin2] pad-removed signal");
 
        caps = gst_pad_query_caps(new_pad, NULL);
-       if (caps) {
-               gchar* caps_str = NULL;
-               caps_str = gst_caps_to_string(caps);
+       if (!caps) {
+               LOGW("query caps is NULL");
+               return;
+       }
 
-               LOGD("pad removed caps : %s from %s", caps_str, GST_ELEMENT_NAME(elem));
+       gchar *caps_str = NULL;
+       caps_str = gst_caps_to_string(caps);
 
-               MMPLAYER_FREEIF(caps_str);
-               gst_caps_unref(caps);
-       }
+       LOGD("pad removed caps : %s from %s", caps_str, GST_ELEMENT_NAME(elem));
+
+       MMPLAYER_FREEIF(caps_str);
+       gst_caps_unref(caps);
 }
 
 static void
 __mmplayer_gst_decode_drained(GstElement *bin, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
+       mm_player_t *player = (mm_player_t *)data;
        GstIterator *iter = NULL;
        GValue item = { 0, };
        GstPad *pad = NULL;
@@ -7108,28 +7121,28 @@ __mmplayer_gst_decode_drained(GstElement *bin, gpointer data)
 void
 __mmplayer_gst_element_added(GstElement *bin, GstElement *element, gpointer data)
 {
-       mm_player_t* player = (mm_player_t*)data;
-       const gcharklass = NULL;
-       gcharfactory_name = NULL;
+       mm_player_t *player = (mm_player_t *)data;
+       const gchar *klass = NULL;
+       gchar *factory_name = NULL;
 
        klass = gst_element_factory_get_metadata(gst_element_get_factory(element), GST_ELEMENT_METADATA_KLASS);
        factory_name = GST_OBJECT_NAME(gst_element_get_factory(element));
 
-       LOGD("new elem klass: %s, factory_name: %s, new elem name : %s\n", klass, factory_name, GST_ELEMENT_NAME(element));
+       LOGD("new elem klass: %s, factory_name: %s, new elem name : %s", klass, factory_name, GST_ELEMENT_NAME(element));
 
        if (__mmplayer_add_dump_buffer_probe(player, element))
                LOGD("add buffer probe");
 
        //<-
        if (g_strrstr(klass, "Codec/Decoder/Audio")) {
-               gcharselected = NULL;
+               gchar *selected = NULL;
                selected = g_strdup(GST_ELEMENT_NAME(element));
                player->audio_decoders = g_list_append(player->audio_decoders, selected);
        }
        //-> temp code
 
        if (g_strrstr(klass, "Parser")) {
-               gcharselected = NULL;
+               gchar *selected = NULL;
 
                selected = g_strdup(factory_name);
                player->parsers = g_list_append(player->parsers, selected);
@@ -7161,10 +7174,10 @@ __mmplayer_gst_element_added(GstElement *bin, GstElement *element, gpointer data
 
        // to support trust-zone only
        if (g_strrstr(factory_name, "asfdemux")) {
-               LOGD("set file-location %s\n", player->profile.uri);
+               LOGD("set file-location %s", player->profile.uri);
                g_object_set(G_OBJECT(element), "file-location", player->profile.uri, NULL);
        } else if (g_strrstr(factory_name, "legacyh264parse")) {
-               LOGD("[%s] output-format to legacyh264parse\n", "mssdemux");
+               LOGD("[%s] output-format to legacyh264parse", "mssdemux");
                g_object_set(G_OBJECT(element), "output-format", 1, NULL); /* NALU/Byte Stream format */
        } else if (g_strrstr(factory_name, "mpegaudioparse")) {
                if ((MMPLAYER_IS_HTTP_STREAMING(player)) &&
@@ -7197,7 +7210,7 @@ __mmplayer_gst_element_added(GstElement *bin, GstElement *element, gpointer data
 }
 
 static void
-__mmplayer_release_misc(mm_player_tplayer)
+__mmplayer_release_misc(mm_player_t *player)
 {
        int i;
        bool cur_mode = player->set_mode.rich_audio;
@@ -7207,7 +7220,7 @@ __mmplayer_release_misc(mm_player_t* player)
 
        player->video_stream_cb = NULL;
        player->video_stream_cb_user_param = NULL;
-       player->video_stream_prerolled = FALSE;
+       player->video_stream_prerolled = false;
 
        player->audio_stream_render_cb = NULL;
        player->audio_stream_cb_user_param = NULL;
@@ -7234,7 +7247,7 @@ __mmplayer_release_misc(mm_player_t* player)
        player->http_content_size = 0;
        player->not_supported_codec = MISSING_PLUGIN_NONE;
        player->can_support_codec = FOUND_PLUGIN_NONE;
-       player->pending_seek.is_pending = FALSE;
+       player->pending_seek.is_pending = false;
        player->pending_seek.pos = 0;
        player->msg_posted = FALSE;
        player->has_many_types = FALSE;
@@ -7300,7 +7313,7 @@ __mmplayer_release_misc(mm_player_t* player)
 }
 
 static void
-__mmplayer_release_misc_post(mm_player_tplayer)
+__mmplayer_release_misc_post(mm_player_t *player)
 {
        char *original_uri = NULL;
        MMPLAYER_FENTER();
@@ -7339,10 +7352,10 @@ __mmplayer_release_misc_post(mm_player_t* player)
 
                if (player->attrs) {
                        mm_attrs_set_string_by_name(player->attrs, "profile_uri", original_uri);
-                       LOGD("restore original uri = %s\n", original_uri);
+                       LOGD("restore original uri = %s", original_uri);
 
                        if (mm_attrs_commit_all(player->attrs))
-                               LOGE("failed to commit the original uri.\n");
+                               LOGE("failed to commit the original uri.");
                }
 
                GList *uri_list = player->uri_info.uri_list;
@@ -7373,7 +7386,7 @@ __mmplayer_release_misc_post(mm_player_t* player)
 }
 
 gboolean
-__mmplayer_check_subtitle(mm_player_tplayer)
+__mmplayer_check_subtitle(mm_player_t *player)
 {
        MMHandleType attrs = 0;
        char *subtitle_uri = NULL;
@@ -7400,7 +7413,7 @@ __mmplayer_check_subtitle(mm_player_t* player)
 }
 
 void
-__mmplayer_cancel_eos_timer(mm_player_tplayer)
+__mmplayer_cancel_eos_timer(mm_player_t *player)
 {
        MMPLAYER_RETURN_IF_FAIL(player);
 
@@ -7414,38 +7427,36 @@ __mmplayer_cancel_eos_timer(mm_player_t* player)
 }
 
 static void
-__mmplayer_add_sink(mm_player_t* player, GstElement* sink)
+__mmplayer_add_sink(mm_player_t *player, GstElement *sink)
 {
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_IF_FAIL(player);
        MMPLAYER_RETURN_IF_FAIL(sink);
 
-       player->sink_elements =
-               g_list_append(player->sink_elements, sink);
+       player->sink_elements = g_list_append(player->sink_elements, sink);
 
        MMPLAYER_FLEAVE();
 }
 
 static void
-__mmplayer_del_sink(mm_player_t* player, GstElement* sink)
+__mmplayer_del_sink(mm_player_t *player, GstElement *sink)
 {
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_IF_FAIL(player);
        MMPLAYER_RETURN_IF_FAIL(sink);
 
-       player->sink_elements =
-                       g_list_remove(player->sink_elements, sink);
+       player->sink_elements = g_list_remove(player->sink_elements, sink);
 
        MMPLAYER_FLEAVE();
 }
 
 void
-__mmplayer_add_signal_connection(mm_player_t* player, GObject* object,
-                       MMPlayerSignalType type, const gchar* signal, GCallback cb_funct, gpointer u_data)
+__mmplayer_add_signal_connection(mm_player_t *player, GObject *object,
+       MMPlayerSignalType type, const gchar *signal, GCallback cb_funct, gpointer u_data)
 {
-       MMPlayerSignalItemitem = NULL;
+       MMPlayerSignalItem *item = NULL;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_IF_FAIL(player);
@@ -7455,7 +7466,7 @@ __mmplayer_add_signal_connection(mm_player_t* player, GObject* object,
                return;
        }
 
-       item = (MMPlayerSignalItem*)g_malloc(sizeof(MMPlayerSignalItem));
+       item = (MMPlayerSignalItem *)g_try_malloc(sizeof(MMPlayerSignalItem));
        if (!item) {
                LOGE("cannot connect signal [%s]", signal);
                return;
@@ -7487,10 +7498,10 @@ __mmplayer_add_signal_connection(mm_player_t* player, GObject* object,
  *    g_signal_handler_disconnect(instance, id);
  */
 static void
-__mmplayer_release_signal_connection(mm_player_tplayer, MMPlayerSignalType type)
+__mmplayer_release_signal_connection(mm_player_t *player, MMPlayerSignalType type)
 {
-       GListsig_list = NULL;
-       MMPlayerSignalItemitem = NULL;
+       GList *sig_list = NULL;
+       MMPlayerSignalItem *item = NULL;
 
        MMPLAYER_FENTER();
 
@@ -7528,9 +7539,10 @@ __mmplayer_release_signal_connection(mm_player_t* player, MMPlayerSignalType typ
        return;
 }
 
-int _mmplayer_change_videosink(MMHandleType handle, MMDisplaySurfaceType surface_type, void *display_overlay)
+int
+_mmplayer_change_videosink(MMHandleType handle, MMDisplaySurfaceType surface_type, void *display_overlay)
 {
-       mm_player_tplayer = 0;
+       mm_player_t *player = 0;
        int prev_display_surface_type = 0;
        void *prev_display_overlay = NULL;
 
@@ -7586,9 +7598,10 @@ int _mmplayer_change_videosink(MMHandleType handle, MMDisplaySurfaceType surface
 }
 
 /* Note : if silent is true, then subtitle would not be displayed. :*/
-int _mmplayer_set_subtitle_silent(MMHandleType hplayer, int silent)
+int
+_mmplayer_set_subtitle_silent(MMHandleType hplayer, int silent)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -7597,17 +7610,18 @@ int _mmplayer_set_subtitle_silent(MMHandleType hplayer, int silent)
 
        player->set_mode.subtitle_off = silent;
 
-       LOGD("subtitle is %s.\n", player->set_mode.subtitle_off ? "ON" : "OFF");
+       LOGD("subtitle is %s.", player->set_mode.subtitle_off ? "ON" : "OFF");
 
        MMPLAYER_FLEAVE();
 
        return MM_ERROR_NONE;
 }
 
-int _mmplayer_sync_subtitle_pipeline(mm_player_t* player)
+int
+_mmplayer_sync_subtitle_pipeline(mm_player_t *player)
 {
-       MMPlayerGstElementmainbin = NULL;
-       MMPlayerGstElementtextbin = NULL;
+       MMPlayerGstElement *mainbin = NULL;
+       MMPlayerGstElement *textbin = NULL;
        GstStateChangeReturn ret = GST_STATE_CHANGE_FAILURE;
        GstState current_state = GST_STATE_VOID_PENDING;
        GstState element_state = GST_STATE_VOID_PENDING;
@@ -7651,7 +7665,7 @@ int _mmplayer_sync_subtitle_pipeline(mm_player_t* player)
 
                ret = gst_element_get_state(mainbin[MMPLAYER_M_SUBSRC].gst, &element_state, &element_pending_state, 5 * GST_SECOND);
                if (GST_STATE_CHANGE_FAILURE == ret) {
-                       LOGE("fail to state change.\n");
+                       LOGE("fail to state change.");
                        result = MM_ERROR_PLAYER_INTERNAL;
                        goto ERROR;
                }
@@ -7667,7 +7681,7 @@ int _mmplayer_sync_subtitle_pipeline(mm_player_t* player)
        // seek to current position
        if (!gst_element_query_position(mainbin[MMPLAYER_M_PIPE].gst, GST_FORMAT_TIME, &time)) {
                result = MM_ERROR_PLAYER_INVALID_STATE;
-               LOGE("gst_element_query_position failed, invalid state\n");
+               LOGE("gst_element_query_position failed, invalid state");
                goto ERROR;
        }
 
@@ -7708,16 +7722,16 @@ ERROR:
 }
 
 static int
-__mmplayer_change_external_subtitle_language(mm_player_t* player, const char* filepath)
+__mmplayer_change_external_subtitle_language(mm_player_t *player, const char *filepath)
 {
        GstStateChangeReturn ret = GST_STATE_CHANGE_FAILURE;
        GstState current_state = GST_STATE_VOID_PENDING;
 
        MMHandleType attrs = 0;
-       MMPlayerGstElementmainbin = NULL;
-       MMPlayerGstElementtextbin = NULL;
+       MMPlayerGstElement *mainbin = NULL;
+       MMPlayerGstElement *textbin = NULL;
 
-       gcharsubtitle_uri = NULL;
+       gchar *subtitle_uri = NULL;
        int result = MM_ERROR_NONE;
        const gchar *charset = NULL;
 
@@ -7736,20 +7750,20 @@ __mmplayer_change_external_subtitle_language(mm_player_t* player, const char* fi
        current_state = GST_STATE(mainbin[MMPLAYER_M_PIPE].gst);
        if (current_state < GST_STATE_READY) {
                result = MM_ERROR_PLAYER_INVALID_STATE;
-               LOGE("Pipeline is not in proper state\n");
+               LOGE("Pipeline is not in proper state");
                goto EXIT;
        }
 
        attrs = MMPLAYER_GET_ATTRS(player);
        if (!attrs) {
-               LOGE("cannot get content attribute\n");
+               LOGE("cannot get content attribute");
                result = MM_ERROR_PLAYER_INTERNAL;
                goto EXIT;
        }
 
        mm_attrs_get_string_by_name(attrs, "subtitle_uri", &subtitle_uri);
        if (!subtitle_uri || strlen(subtitle_uri) < 1) {
-               LOGE("subtitle uri is not proper filepath\n");
+               LOGE("subtitle uri is not proper filepath");
                result = MM_ERROR_PLAYER_INVALID_URI;
                goto EXIT;
        }
@@ -7760,16 +7774,16 @@ __mmplayer_change_external_subtitle_language(mm_player_t* player, const char* fi
                goto EXIT;
        }
 
-       LOGD("old subtitle file path is [%s]\n", subtitle_uri);
-       LOGD("new subtitle file path is [%s]\n", filepath);
+       LOGD("old subtitle file path is [%s]", subtitle_uri);
+       LOGD("new subtitle file path is [%s]", filepath);
 
        if (!strcmp(filepath, subtitle_uri)) {
-               LOGD("No need to swtich subtitle, as input filepath is same as current filepath\n");
+               LOGD("No need to swtich subtitle, as input filepath is same as current filepath");
                goto EXIT;
        } else {
                mm_attrs_set_string_by_name(player->attrs, "subtitle_uri", filepath);
                if (mm_attrs_commit_all(player->attrs)) {
-                       LOGE("failed to commit.\n");
+                       LOGE("failed to commit.");
                        goto EXIT;
                }
        }
@@ -7806,7 +7820,7 @@ __mmplayer_change_external_subtitle_language(mm_player_t* player, const char* fi
 
        charset = util_get_charset(filepath);
        if (charset) {
-               LOGD("detected charset is %s\n", charset);
+               LOGD("detected charset is %s", charset);
                g_object_set(G_OBJECT(mainbin[MMPLAYER_M_SUBPARSE].gst), "subtitle-encoding", charset, NULL);
        }
 
@@ -7818,10 +7832,11 @@ EXIT:
 }
 
 /* API to switch between external subtitles */
-int _mmplayer_set_external_subtitle_path(MMHandleType hplayer, const char* filepath)
+int
+_mmplayer_set_external_subtitle_path(MMHandleType hplayer, const char *filepath)
 {
        int result = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        char *path = NULL;
 
        MMPLAYER_FENTER();
@@ -7889,14 +7904,14 @@ int _mmplayer_set_external_subtitle_path(MMHandleType hplayer, const char* filep
 }
 
 static int
-__mmplayer_change_selector_pad(mm_player_tplayer, MMPlayerTrackType type, int index)
+__mmplayer_change_selector_pad(mm_player_t *player, MMPlayerTrackType type, int index)
 {
        int result = MM_ERROR_NONE;
-       gcharchange_pad_name = NULL;
-       GstPadsinkpad = NULL;
-       MMPlayerGstElementmainbin = NULL;
+       gchar *change_pad_name = NULL;
+       GstPad *sinkpad = NULL;
+       MMPlayerGstElement *mainbin = NULL;
        enum MainElementID elem_idx = MMPLAYER_M_NUM;
-       GstCapscaps = NULL;
+       GstCaps *caps = NULL;
        gint total_track_num = 0;
 
        MMPLAYER_FENTER();
@@ -7904,7 +7919,7 @@ __mmplayer_change_selector_pad(mm_player_t* player, MMPlayerTrackType type, int
        MMPLAYER_RETURN_VAL_IF_FAIL(player && player->pipeline && player->pipeline->mainbin,
                                                                                                        MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       LOGD("Change Track(%d) to %d\n", type, index);
+       LOGD("Change Track(%d) to %d", type, index);
 
        mainbin = player->pipeline->mainbin;
 
@@ -7914,26 +7929,26 @@ __mmplayer_change_selector_pad(mm_player_t* player, MMPlayerTrackType type, int
                elem_idx = MMPLAYER_M_T_INPUT_SELECTOR;
        } else {
                /* Changing Video Track is not supported. */
-               LOGE("Track Type Error\n");
+               LOGE("Track Type Error");
                goto EXIT;
        }
 
        if (mainbin[elem_idx].gst == NULL) {
                result = MM_ERROR_PLAYER_NO_OP;
-               LOGD("Req track doesn't exist\n");
+               LOGD("Req track doesn't exist");
                goto EXIT;
        }
 
        total_track_num = player->selector[type].total_track_num;
        if (total_track_num <= 0) {
                result = MM_ERROR_PLAYER_NO_OP;
-               LOGD("Language list is not available \n");
+               LOGD("Language list is not available");
                goto EXIT;
        }
 
        if ((index < 0) || (index >= total_track_num)) {
                result = MM_ERROR_INVALID_ARGUMENT;
-               LOGD("Not a proper index : %d \n", index);
+               LOGD("Not a proper index : %d", index);
                goto EXIT;
        }
 
@@ -7941,11 +7956,11 @@ __mmplayer_change_selector_pad(mm_player_t* player, MMPlayerTrackType type, int
        change_pad_name = g_strdup_printf("sink_%u", index);
        if (change_pad_name == NULL) {
                result = MM_ERROR_PLAYER_INTERNAL;
-               LOGD("Pad does not exists\n");
+               LOGD("Pad does not exists");
                goto EXIT;
        }
 
-       LOGD("new active pad name: %s\n", change_pad_name);
+       LOGD("new active pad name: %s", change_pad_name);
 
        sinkpad = gst_element_get_static_pad(mainbin[elem_idx].gst, change_pad_name);
        if (sinkpad == NULL) {
@@ -7954,7 +7969,7 @@ __mmplayer_change_selector_pad(mm_player_t* player, MMPlayerTrackType type, int
                goto EXIT;
        }
 
-       LOGD("Set Active Pad - %s:%s\n", GST_DEBUG_PAD_NAME(sinkpad));
+       LOGD("Set Active Pad - %s:%s", GST_DEBUG_PAD_NAME(sinkpad));
        g_object_set(mainbin[elem_idx].gst, "active-pad", sinkpad, NULL);
 
        caps = gst_pad_get_current_caps(sinkpad);
@@ -7967,30 +7982,30 @@ __mmplayer_change_selector_pad(mm_player_t* player, MMPlayerTrackType type, int
                __mmplayer_set_audio_attrs(player, caps);
 
 EXIT:
-
        MMPLAYER_FREEIF(change_pad_name);
        return result;
 }
 
-int _mmplayer_change_track_language(MMHandleType hplayer, MMPlayerTrackType type, int index)
+int
+ _mmplayer_change_track_language(MMHandleType hplayer, MMPlayerTrackType type, int index)
 {
        int result = MM_ERROR_NONE;
-       mm_player_tplayer = NULL;
-       MMPlayerGstElementmainbin = NULL;
+       mm_player_t *player = NULL;
+       MMPlayerGstElement *mainbin = NULL;
 
        gint current_active_index = 0;
 
        GstState current_state = GST_STATE_VOID_PENDING;
-       GstEventevent = NULL;
+       GstEvent *event = NULL;
        gint64 time = 0;
 
        MMPLAYER_FENTER();
 
-       player = (mm_player_t*)hplayer;
+       player = (mm_player_t *)hplayer;
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
        if (!player->pipeline) {
-               LOGE("Track %d pre setting -> %d\n", type, index);
+               LOGE("Track %d pre setting -> %d", type, index);
 
                player->selector[type].active_pad_index = index;
                goto EXIT;
@@ -8012,20 +8027,22 @@ int _mmplayer_change_track_language(MMHandleType hplayer, MMPlayerTrackType type
        current_state = GST_STATE(mainbin[MMPLAYER_M_PIPE].gst);
        if (current_state < GST_STATE_PAUSED) {
                result = MM_ERROR_PLAYER_INVALID_STATE;
-               LOGW("Pipeline not in porper state\n");
+               LOGW("Pipeline not in porper state");
                goto EXIT;
        }
 
        result = __mmplayer_change_selector_pad(player, type, index);
        if (result != MM_ERROR_NONE) {
-               LOGE("change selector pad error\n");
+               LOGE("change selector pad error");
                goto EXIT;
        }
 
        player->selector[type].active_pad_index = index;
 
        if (current_state == GST_STATE_PLAYING) {
-               event = gst_event_new_seek(player->playback_rate, GST_FORMAT_TIME, (GstSeekFlags)(GST_SEEK_FLAG_SEGMENT | GST_SEEK_FLAG_FLUSH | GST_SEEK_FLAG_SKIP), GST_SEEK_TYPE_SET, time, GST_SEEK_TYPE_NONE, -1);
+               event = gst_event_new_seek(player->playback_rate, GST_FORMAT_TIME,
+                       (GstSeekFlags)(GST_SEEK_FLAG_SEGMENT | GST_SEEK_FLAG_FLUSH | GST_SEEK_FLAG_SKIP),
+                       GST_SEEK_TYPE_SET, time, GST_SEEK_TYPE_NONE, -1);
                if (event) {
                        __mmplayer_gst_send_event_to_sink(player, event);
                } else {
@@ -8038,9 +8055,10 @@ EXIT:
        return result;
 }
 
-int _mmplayer_get_subtitle_silent(MMHandleType hplayer, int* silent)
+int
+_mmplayer_get_subtitle_silent(MMHandleType hplayer, int *silent)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -8049,7 +8067,7 @@ int _mmplayer_get_subtitle_silent(MMHandleType hplayer, int* silent)
 
        *silent = player->set_mode.subtitle_off;
 
-       LOGD("subtitle is %s.\n", silent ? "ON" : "OFF");
+       LOGD("subtitle is %s.", silent ? "ON" : "OFF");
 
        MMPLAYER_FLEAVE();
 
@@ -8071,8 +8089,7 @@ __mmplayer_add_dump_buffer_probe(mm_player_t *player, GstElement *element)
                if (g_strrstr(factory_name, player->ini.dump_element_keyword[idx])) {
                        LOGD("dump [%s] sink pad", player->ini.dump_element_keyword[idx]);
                        mm_player_dump_t *dump_s;
-                       dump_s = g_malloc(sizeof(mm_player_dump_t));
-
+                       dump_s = g_try_malloc(sizeof(mm_player_dump_t));
                        if (dump_s == NULL) {
                                LOGE("malloc fail");
                                return FALSE;
@@ -8083,8 +8100,8 @@ __mmplayer_add_dump_buffer_probe(mm_player_t *player, GstElement *element)
                        dump_s->dump_pad = gst_element_get_static_pad(element, "sink");
 
                        if (dump_s->dump_pad) {
-                               memset(dump_file_name, 0x00, PLAYER_INI_MAX_STRLEN*2);
-                               snprintf(dump_file_name, PLAYER_INI_MAX_STRLEN*2, "%s/%s_sink_pad.dump", player->ini.dump_element_path, player->ini.dump_element_keyword[idx]);
+                               memset(dump_file_name, 0x00, PLAYER_INI_MAX_STRLEN * 2);
+                               snprintf(dump_file_name, PLAYER_INI_MAX_STRLEN * 2, "%s/%s_sink_pad.dump", player->ini.dump_element_path, player->ini.dump_element_keyword[idx]);
                                dump_s->dump_element_file = fopen(dump_file_name, "w+");
                                dump_s->probe_handle_id = gst_pad_add_probe(dump_s->dump_pad, GST_PAD_PROBE_TYPE_BUFFER, __mmplayer_dump_buffer_probe_cb, dump_s->dump_element_file, NULL);
                                /* add list for removed buffer probe and close FILE */
@@ -8092,8 +8109,7 @@ __mmplayer_add_dump_buffer_probe(mm_player_t *player, GstElement *element)
                                LOGD("%s sink pad added buffer probe for dump", factory_name);
                                return TRUE;
                        } else {
-                               g_free(dump_s);
-                               dump_s = NULL;
+                               MMPLAYER_FREEIF(dump_s);
                                LOGE("failed to get %s sink pad added", factory_name);
                        }
                }
@@ -8104,12 +8120,12 @@ __mmplayer_add_dump_buffer_probe(mm_player_t *player, GstElement *element)
 static GstPadProbeReturn
 __mmplayer_dump_buffer_probe_cb(GstPad *pad,  GstPadProbeInfo *info, gpointer u_data)
 {
-       FILE *dump_data = (FILE *) u_data;
+       FILE *dump_data = (FILE *)u_data;
 //     int written = 0;
        GstBuffer *buffer = gst_pad_probe_info_get_buffer(info);
        GstMapInfo probe_info = GST_MAP_INFO_INIT;
 
-       MMPLAYER_RETURN_VAL_IF_FAIL(dump_data, FALSE);
+       MMPLAYER_RETURN_VAL_IF_FAIL(dump_data, GST_PAD_PROBE_PASS);
 
        gst_buffer_map(buffer, &probe_info, GST_MAP_READ);
 
@@ -8123,43 +8139,46 @@ __mmplayer_dump_buffer_probe_cb(GstPad *pad,  GstPadProbeInfo *info, gpointer u_
 static void
 __mmplayer_release_dump_list(GList *dump_list)
 {
-       if (dump_list) {
-               GList *d_list = dump_list;
-               for (; d_list; d_list = g_list_next(d_list)) {
-                       mm_player_dump_t *dump_s = d_list->data;
-                       if (dump_s->dump_pad) {
-                               if (dump_s->probe_handle_id)
-                                       gst_pad_remove_probe(dump_s->dump_pad, dump_s->probe_handle_id);
-                       }
-                       if (dump_s->dump_element_file) {
-                               fclose(dump_s->dump_element_file);
-                               dump_s->dump_element_file = NULL;
-                       }
-                       MMPLAYER_FREEIF(dump_s);
+       GList *d_list = dump_list;
+
+       if (!d_list)
+               return;
+
+       for (; d_list; d_list = g_list_next(d_list)) {
+               mm_player_dump_t *dump_s = d_list->data;
+               if (dump_s->dump_pad) {
+                       if (dump_s->probe_handle_id)
+                               gst_pad_remove_probe(dump_s->dump_pad, dump_s->probe_handle_id);
+               }
+               if (dump_s->dump_element_file) {
+                       fclose(dump_s->dump_element_file);
+                       dump_s->dump_element_file = NULL;
                }
-               g_list_free(dump_list);
-               dump_list = NULL;
+               MMPLAYER_FREEIF(dump_s);
        }
+       g_list_free(dump_list);
+       dump_list = NULL;
 }
 
 int
-_mmplayer_has_closed_caption(MMHandleType hplayer, boolexist)
+_mmplayer_has_closed_caption(MMHandleType hplayer, bool *exist)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
        MMPLAYER_RETURN_VAL_IF_FAIL(exist, MM_ERROR_INVALID_ARGUMENT);
 
-       *exist = player->has_closed_caption;
+       *exist = (bool)player->has_closed_caption;
 
        MMPLAYER_FLEAVE();
 
        return MM_ERROR_NONE;
 }
 
-void _mm_player_video_stream_internal_buffer_unref(void *buffer)
+void
+_mm_player_video_stream_internal_buffer_unref(void *buffer)
 {
        MMPLAYER_FENTER();
        if (buffer) {
@@ -8170,9 +8189,10 @@ void _mm_player_video_stream_internal_buffer_unref(void *buffer)
        MMPLAYER_FLEAVE();
 }
 
-int _mmplayer_get_timeout(MMHandleType hplayer, int *timeout)
+int
+_mmplayer_get_timeout(MMHandleType hplayer, int *timeout)
 {
-       mm_player_t *player = (mm_player_t *) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -8180,9 +8200,9 @@ int _mmplayer_get_timeout(MMHandleType hplayer, int *timeout)
        MMPLAYER_RETURN_VAL_IF_FAIL(timeout, MM_ERROR_COMMON_INVALID_ARGUMENT);
 
        if (MMPLAYER_IS_STREAMING(player))
-               *timeout = player->ini.live_state_change_timeout;
+               *timeout = (int)player->ini.live_state_change_timeout;
        else
-               *timeout = player->ini.localplayback_state_change_timeout;
+               *timeout = (int)player->ini.localplayback_state_change_timeout;
 
        LOGD("timeout = %d", *timeout);
 
@@ -8190,9 +8210,10 @@ int _mmplayer_get_timeout(MMHandleType hplayer, int *timeout)
        return MM_ERROR_NONE;
 }
 
-int _mmplayer_get_num_of_video_out_buffers(MMHandleType hplayer, int *num, int *extra_num)
+int
+_mmplayer_get_num_of_video_out_buffers(MMHandleType hplayer, int *num, int *extra_num)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
 
@@ -8202,14 +8223,14 @@ int _mmplayer_get_num_of_video_out_buffers(MMHandleType hplayer, int *num, int *
        *num = player->video_num_buffers;
        *extra_num = player->video_extra_num_buffers;
 
-       LOGD("state %d, num %d(%d)\n", MMPLAYER_CURRENT_STATE(player), *num, *extra_num);
+       LOGD("state %d, num %d(%d)", MMPLAYER_CURRENT_STATE(player), *num, *extra_num);
 
        MMPLAYER_FLEAVE();
        return MM_ERROR_NONE;
 }
 
 static void
-__mmplayer_initialize_storage_info(mm_player_tplayer, MMPlayerPathType path_type)
+__mmplayer_initialize_storage_info(mm_player_t *player, MMPlayerPathType path_type)
 {
        int i = 0;
        MMPLAYER_FENTER();
@@ -8231,10 +8252,11 @@ __mmplayer_initialize_storage_info(mm_player_t* player, MMPlayerPathType path_ty
        MMPLAYER_FLEAVE();
 }
 
-int _mmplayer_manage_external_storage_state(MMHandleType hplayer, int id, int state)
+int
+_mmplayer_manage_external_storage_state(MMHandleType hplayer, int id, int state)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*)hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        MMMessageParamType msg_param = {0, };
 
        MMPLAYER_FENTER();
@@ -8246,8 +8268,10 @@ int _mmplayer_manage_external_storage_state(MMHandleType hplayer, int id, int st
                return MM_ERROR_NONE;
 
        /* FIXME: text path should be handled seperately. */
-       if (((player->storage_info[MMPLAYER_PATH_VOD].type == STORAGE_TYPE_EXTERNAL) && (player->storage_info[MMPLAYER_PATH_VOD].id == id)) ||
-               ((player->storage_info[MMPLAYER_PATH_TEXT].type == STORAGE_TYPE_EXTERNAL) && (player->storage_info[MMPLAYER_PATH_TEXT].id == id))) {
+       if (((player->storage_info[MMPLAYER_PATH_VOD].type == STORAGE_TYPE_EXTERNAL)
+               && (player->storage_info[MMPLAYER_PATH_VOD].id == id)) ||
+               ((player->storage_info[MMPLAYER_PATH_TEXT].type == STORAGE_TYPE_EXTERNAL)
+               && (player->storage_info[MMPLAYER_PATH_TEXT].id == id))) {
                LOGW("external storage is removed");
 
                if (player->msg_posted == FALSE) {
@@ -8267,10 +8291,11 @@ int _mmplayer_manage_external_storage_state(MMHandleType hplayer, int id, int st
        return ret;
 }
 
-int _mmplayer_get_adaptive_variant_info(MMHandleType hplayer, int *num, char **var_info)
+int
+_mmplayer_get_adaptive_variant_info(MMHandleType hplayer, int *num, char **var_info)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int idx = 0, total = 0;
        gchar *result = NULL, *tmp = NULL;
 
@@ -8309,10 +8334,11 @@ int _mmplayer_get_adaptive_variant_info(MMHandleType hplayer, int *num, char **v
        return ret;
 }
 
-int _mmplayer_set_max_adaptive_variant_limit(MMHandleType hplayer, int bandwidth, int width, int height)
+int
+_mmplayer_set_max_adaptive_variant_limit(MMHandleType hplayer, int bandwidth, int width, int height)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -8336,10 +8362,11 @@ int _mmplayer_set_max_adaptive_variant_limit(MMHandleType hplayer, int bandwidth
 
 }
 
-int _mmplayer_get_max_adaptive_variant_limit(MMHandleType hplayer, int *bandwidth, int *width, int *height)
+int
+_mmplayer_get_max_adaptive_variant_limit(MMHandleType hplayer, int *bandwidth, int *width, int *height)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -8355,10 +8382,11 @@ int _mmplayer_get_max_adaptive_variant_limit(MMHandleType hplayer, int *bandwidt
        return ret;
 }
 
-int _mmplayer_set_streaming_buffering_time(MMHandleType hplayer, int buffer_ms, int rebuffer_ms)
+int
+_mmplayer_set_streaming_buffering_time(MMHandleType hplayer, int buffer_ms, int rebuffer_ms)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -8384,10 +8412,11 @@ int _mmplayer_set_streaming_buffering_time(MMHandleType hplayer, int buffer_ms,
 
 }
 
-int _mmplayer_get_streaming_buffering_time(MMHandleType hplayer, int *buffer_ms, int *rebuffer_ms)
+int
+_mmplayer_get_streaming_buffering_time(MMHandleType hplayer, int *buffer_ms, int *rebuffer_ms)
 {
        int ret = MM_ERROR_NONE;
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
 
        MMPLAYER_FENTER();
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
@@ -8407,11 +8436,12 @@ int _mmplayer_get_streaming_buffering_time(MMHandleType hplayer, int *buffer_ms,
        return ret;
 }
 
-int _mmplayer_set_codec_type(MMHandleType hplayer, MMPlayerStreamType stream_type, MMPlayerVideoCodecType codec_type)
+int
+_mmplayer_set_codec_type(MMHandleType hplayer, MMPlayerStreamType stream_type, MMPlayerVideoCodecType codec_type)
 {
 #define IDX_FIRST_SW_CODEC 0
-       mm_player_t* player = (mm_player_t*) hplayer;
-       const charattr_name = (stream_type == MM_PLAYER_STREAM_TYPE_AUDIO) ? (MM_PLAYER_AUDIO_CODEC_TYPE) : (MM_PLAYER_VIDEO_CODEC_TYPE);
+       mm_player_t *player = (mm_player_t *)hplayer;
+       const char *attr_name = (stream_type == MM_PLAYER_STREAM_TYPE_AUDIO) ? (MM_PLAYER_AUDIO_CODEC_TYPE) : (MM_PLAYER_VIDEO_CODEC_TYPE);
        MMHandleType attrs = 0;
 
        MMPLAYER_FENTER();
@@ -8469,8 +8499,8 @@ int _mmplayer_set_codec_type(MMHandleType hplayer, MMPlayerStreamType stream_typ
 int
 _mmplayer_set_replaygain_enabled(MMHandleType hplayer, bool enabled)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
-       GstElementrg_vol_element = NULL;
+       mm_player_t *player = (mm_player_t *)hplayer;
+       GstElement *rg_vol_element = NULL;
 
        MMPLAYER_FENTER();
 
@@ -8480,7 +8510,7 @@ _mmplayer_set_replaygain_enabled(MMHandleType hplayer, bool enabled)
 
        /* just hold rgvolume enable value if pipeline is not ready */
        if (!player->pipeline || !player->pipeline->audiobin) {
-               LOGD("pipeline is not ready. holding rgvolume enable value\n");
+               LOGD("pipeline is not ready. holding rgvolume enable value");
                return MM_ERROR_NONE;
        }
 
@@ -8504,8 +8534,8 @@ _mmplayer_set_replaygain_enabled(MMHandleType hplayer, bool enabled)
 int
 _mmplayer_is_replaygain_enabled(MMHandleType hplayer, bool *enabled)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
-       GstElementrg_vol_element = NULL;
+       mm_player_t *player = (mm_player_t *)hplayer;
+       GstElement *rg_vol_element = NULL;
        gboolean enable = FALSE;
 
        MMPLAYER_FENTER();
@@ -8515,7 +8545,7 @@ _mmplayer_is_replaygain_enabled(MMHandleType hplayer, bool *enabled)
 
        /* just hold enable_rg value if pipeline is not ready */
        if (!player->pipeline || !player->pipeline->audiobin) {
-               LOGD("pipeline is not ready. holding rgvolume value (%d)\n", player->sound.rg_enable);
+               LOGD("pipeline is not ready. holding rgvolume value (%d)", player->sound.rg_enable);
                *enabled = player->sound.rg_enable;
                return MM_ERROR_NONE;
        }
@@ -8528,7 +8558,7 @@ _mmplayer_is_replaygain_enabled(MMHandleType hplayer, bool *enabled)
        }
 
        g_object_get(rg_vol_element, "enable-rgvolume", &enable, NULL);
-       *enabled = enable;
+       *enabled = (bool)enable;
 
        MMPLAYER_FLEAVE();
 
@@ -8538,7 +8568,7 @@ _mmplayer_is_replaygain_enabled(MMHandleType hplayer, bool *enabled)
 int
 _mmplayer_set_video_roi_area(MMHandleType hplayer, double scale_x, double scale_y, double scale_width, double scale_height)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        MMHandleType attrs = 0;
        void *handle = NULL;
        int ret = MM_ERROR_NONE;
@@ -8581,7 +8611,7 @@ _mmplayer_set_video_roi_area(MMHandleType hplayer, double scale_x, double scale_
 int
 _mmplayer_get_video_roi_area(MMHandleType hplayer, double *scale_x, double *scale_y, double *scale_width, double *scale_height)
 {
-       mm_player_t* player = (mm_player_t*) hplayer;
+       mm_player_t *player = (mm_player_t *)hplayer;
        int ret = MM_ERROR_NONE;
 
        MMPLAYER_FENTER();
@@ -8622,10 +8652,9 @@ __mmplayer_update_duration_attrs(mm_player_t *player, MMHandleType attrs)
        player->streaming_type =  __mmplayer_get_stream_service_type(player);
 
        /* check duration is OK */
-       if (dur_nsec == 0 && !MMPLAYER_IS_LIVE_STREAMING(player)) {
+       if (dur_nsec == 0 && !MMPLAYER_IS_LIVE_STREAMING(player))
                /* FIXIT : find another way to get duration here. */
                LOGW("finally it's failed to get duration from pipeline. progressbar will not work correctely!");
-       }
 
        return ret;
 }
@@ -8638,9 +8667,9 @@ __mmplayer_update_audio_attrs(mm_player_t *player, MMHandleType attrs)
        decoder. Below code only valid when we are not using 'resampler' just before
        'audioconverter'. */
        GstCaps *caps_a = NULL;
-       GstPadpad = NULL;
+       GstPad *pad = NULL;
        gint samplerate = 0, channels = 0;
-       GstStructurep = NULL;
+       GstStructure *p = NULL;
 
        LOGD("try to update audio attrs");
 
@@ -8656,7 +8685,6 @@ __mmplayer_update_audio_attrs(mm_player_t *player, MMHandleType attrs)
        }
 
        caps_a = gst_pad_get_current_caps(pad);
-
        if (!caps_a) {
                LOGW("not ready to get audio caps");
                gst_object_unref(pad);
@@ -8687,10 +8715,10 @@ __mmplayer_update_video_attrs(mm_player_t *player, MMHandleType attrs)
        LOGD("try to update video attrs");
 
        GstCaps *caps_v = NULL;
-       GstPadpad = NULL;
+       GstPad *pad = NULL;
        gint tmpNu, tmpDe;
        gint width, height;
-       GstStructurep = NULL;
+       GstStructure *p = NULL;
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player->pipeline->videobin, FALSE);
        MMPLAYER_RETURN_VAL_IF_FAIL(player->pipeline->videobin[MMPLAYER_V_SINK].gst, FALSE);
@@ -8784,14 +8812,16 @@ __mmplayer_update_bitrate_attrs(mm_player_t *player, MMHandleType attrs)
 
        return ret;
 }
+
 static void
-__mmplayer_copy_uri_and_set_type(MMPlayerParseProfiledata, const char *uri, int uri_type)
+__mmplayer_copy_uri_and_set_type(MMPlayerParseProfile *data, const char *uri, int uri_type)
 {
        strncpy(data->uri, uri, MM_MAX_URL_LEN - 1);
        data->uri_type = uri_type;
 }
+
 static int
-__mmplayer_set_mem_uri(MMPlayerParseProfiledata, char *path, void *param)
+__mmplayer_set_mem_uri(MMPlayerParseProfile *data, char *path, void *param)
 {
        int ret = MM_ERROR_PLAYER_INVALID_URI;
        int mem_size = 0;
@@ -8831,7 +8861,7 @@ __mmplayer_set_mem_uri(MMPlayerParseProfile* data, char *path, void *param)
                }
        }
 
-       LOGD("ext: %s, mem_size: %d, mmap(param): %p\n", ext, mem_size, param);
+       LOGD("ext: %s, mem_size: %d, mmap(param): %p", ext, mem_size, param);
 
        if (mem_size && param) {
                if (data->input_mem.buf)
@@ -8855,7 +8885,7 @@ __mmplayer_set_mem_uri(MMPlayerParseProfile* data, char *path, void *param)
 }
 
 static int
-__mmplayer_set_file_uri(MMPlayerParseProfiledata, const char *uri)
+__mmplayer_set_file_uri(MMPlayerParseProfile *data, const char *uri)
 {
        gchar *location = NULL;
        GError *err = NULL;
@@ -8869,8 +8899,8 @@ __mmplayer_set_file_uri(MMPlayerParseProfile* data, const char *uri)
                                (err != NULL) ? err->message : "unknown error");
                        if (err)
                                g_error_free(err);
-                       if (location)
-                               g_free(location);
+
+                       MMPLAYER_FREEIF(location);
 
                        data->uri_type = MM_PLAYER_URI_TYPE_NONE;
                        return MM_ERROR_PLAYER_INVALID_URI;
@@ -8887,7 +8917,7 @@ __mmplayer_set_file_uri(MMPlayerParseProfile* data, const char *uri)
        if (ret == MM_ERROR_NONE) {
                g_snprintf(data->uri,  MM_MAX_URL_LEN, "file://%s", path);
                if (util_is_sdp_file(path)) {
-                       LOGD("uri is actually a file but it's sdp file. giving it to rtspsrc\n");
+                       LOGD("uri is actually a file but it's sdp file. giving it to rtspsrc");
                        data->uri_type = MM_PLAYER_URI_TYPE_URL_RTSP;
                } else {
                        data->uri_type = MM_PLAYER_URI_TYPE_FILE;
@@ -8895,17 +8925,16 @@ __mmplayer_set_file_uri(MMPlayerParseProfile* data, const char *uri)
        } else if (ret == MM_ERROR_PLAYER_PERMISSION_DENIED) {
                data->uri_type = MM_PLAYER_URI_TYPE_NO_PERMISSION;
        } else {
-               LOGE("invalid uri, could not play..\n");
+               LOGE("invalid uri, could not play..");
                data->uri_type = MM_PLAYER_URI_TYPE_NONE;
        }
 
-       if (location)
-               g_free(location);
+       MMPLAYER_FREEIF(location);
 
        return ret;
 }
 
-static MMPlayerVideoStreamDataType*
+static MMPlayerVideoStreamDataType *
 __mmplayer_create_stream_from_pad(GstPad *pad)
 {
        GstCaps *caps = NULL;
@@ -8938,7 +8967,7 @@ __mmplayer_create_stream_from_pad(GstPad *pad)
                return NULL;
        }
 
-       stream = (MMPlayerVideoStreamDataType *)g_malloc0(sizeof(MMPlayerVideoStreamDataType));
+       stream = (MMPlayerVideoStreamDataType *)g_try_malloc0(sizeof(MMPlayerVideoStreamDataType));
        if (!stream) {
                LOGE("failed to alloc mem for video data");
                return NULL;
@@ -9149,9 +9178,9 @@ __mmplayer_set_playing_state(mm_player_t *player)
                mm_attrs_set_string_by_name(player->attrs, "content_audio_codec", audio_codec);
 
                if (mm_attrs_commit_all(player->attrs))
-                       LOGE("failed to update attributes\n");
+                       LOGE("failed to update attributes");
 
-               LOGD("set audio codec type with caps\n");
+               LOGD("set audio codec type with caps");
        }
 
        return;