Merge "Fix to deallocate memory of sound and speak data" into tizen

[platform/core/uifw/tts.git] / server / ttsd_player.c

diff --git a/server/ttsd_player.c b/server/ttsd_player.c
index be47d32..a2d2e26 100644 (file)
--- a/server/ttsd_player.c
+++ b/server/ttsd_player.c
@@ -351,7 +351,9 @@ int ttsd_player_check_current_playback_focus(bool *is_current_interrupt)
 
                ret = sound_manager_get_current_playback_focus(&reason, &sound_behavior, &extra_info);
 
-               if (SOUND_MANAGER_ERROR_NONE == ret && NULL != extra_info) {
+               SLOG(LOG_DEBUG, tts_tag(), "[Player] current playback focus: extra_info(%s), reason(%d), sound_behavior(%d)", extra_info, reason, sound_behavior, extra_info);
+
+               if (SOUND_MANAGER_ERROR_NONE == ret && NULL != extra_info && 0 < strlen(extra_info)) {
                        if (SOUND_STREAM_FOCUS_CHANGED_BY_VOICE_INFORMATION == reason && 0 == strncmp(extra_info, "TTSD_MODE_INTERRUPT", strlen(extra_info))) {
                                SLOG(LOG_DEBUG, tts_tag(), "[Player] The current focus in Interrupt. Cannot play the requested sound data");
                                *is_current_interrupt = true;
@@ -476,12 +478,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                        /* unset volume policy, volume will be 100% */
                                        __unset_policy_for_playing();
 
-                                       if (NULL != sound_data->data) {
-                                               free(sound_data->data);
-                                               sound_data->data = NULL;
-                                       }
-                                       free(sound_data);
-                                       sound_data = NULL;
+                                       ttsd_data_clear_sound_data(&sound_data);
                                        return;
                                }
 
@@ -506,12 +503,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                                SLOG(LOG_WARN, tts_tag(), "[Send WARNIING] Current player is not valid");
                                                /* unset volume policy, volume will be 100% */
                                                __unset_policy_for_playing();
-                                               if (NULL != sound_data->data) {
-                                                       free(sound_data->data);
-                                                       sound_data->data = NULL;
-                                               }
-                                               free(sound_data);
-                                               sound_data = NULL;
+                                               ttsd_data_clear_sound_data(&sound_data);
                                                return;
                                        }
                                        if (0 != ttsdc_send_utt_finish_message(pid, player->uid, sound_data->utt_id)) {
@@ -520,12 +512,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                        }
                                }
                                SLOG(LOG_INFO, tts_tag(), "[Player] Finish utterance : uid(%d), uttid(%d)", player->uid, sound_data->utt_id);
-                               if (NULL != sound_data->data) {
-                                       free(sound_data->data);
-                                       sound_data->data = NULL;
-                               }
-                               free(sound_data);
-                               sound_data = NULL;
+                               ttsd_data_clear_sound_data(&sound_data);
                                continue;
                        }
                }
@@ -541,12 +528,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                /* unset volume policy, volume will be 100% */
                                __unset_policy_for_playing();
 
-                               if (NULL != sound_data->data) {
-                                       free(sound_data->data);
-                                       sound_data->data = NULL;
-                               }
-                               free(sound_data);
-                               sound_data = NULL;
+                               ttsd_data_clear_sound_data(&sound_data);
 
                                return;
                        }
@@ -573,13 +555,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                        /* unset volume policy, volume will be 100% */
                                        __unset_policy_for_playing();
 
-                                       if (NULL != sound_data->data) {
-                                               free(sound_data->data);
-                                               sound_data->data = NULL;
-                                       }
-
-                                       free(sound_data);
-                                       sound_data = NULL;
+                                       ttsd_data_clear_sound_data(&sound_data);
 
                                        return;
                                }
@@ -605,15 +581,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                /* unset volume policy, volume will be 100% */
                                __unset_policy_for_playing();
 
-                               if (NULL != sound_data) {
-                                       if (NULL != sound_data->data) {
-                                               free(sound_data->data);
-                                               sound_data->data = NULL;
-                                       }
-
-                                       free(sound_data);
-                                       sound_data = NULL;
-                               }
+                               ttsd_data_clear_sound_data(&sound_data);
 
                                return;
                        }
@@ -654,15 +622,7 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                SLOG(LOG_DEBUG, tts_tag(), "[Player SUCCESS] Unprepare audio");
                        }
 
-                       if (NULL != sound_data) {
-                               if (NULL != sound_data->data) {
-                                       free(sound_data->data);
-                                       sound_data->data = NULL;
-                               }
-
-                               free(sound_data);
-                               sound_data = NULL;
-                       }
+                       ttsd_data_clear_sound_data(&sound_data);
                        /* unset volume policy, volume will be 100% */
                        __unset_policy_for_playing();
                        return;
@@ -677,13 +637,8 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                SLOG(LOG_WARN, tts_tag(), "[Send WARNIING] Current player is not valid");
                                /* unset volume policy, volume will be 100% */
                                __unset_policy_for_playing();
-                               if (NULL != sound_data->data) {
-                                       free(sound_data->data);
-                                       sound_data->data = NULL;
-                               }
 
-                               free(sound_data);
-                               sound_data = NULL;
+                               ttsd_data_clear_sound_data(&sound_data);
                                return;
                        }
 
@@ -692,28 +647,15 @@ static void __play_thread(void *data, Ecore_Thread *thread)
                                        pid, player->uid, sound_data->utt_id);
                                /* unset volume policy, volume will be 100% */
                                __unset_policy_for_playing();
-                               if (NULL != sound_data->data) {
-                                       free(sound_data->data);
-                                       sound_data->data = NULL;
-                               }
 
-                               free(sound_data);
-                               sound_data = NULL;
+                               ttsd_data_clear_sound_data(&sound_data);
                                return;
                        }
 
                        SLOG(LOG_INFO, tts_tag(), "[Player] Finish utterance : uid(%d), uttid(%d)", player->uid, sound_data->utt_id);
                }
 
-               if (NULL != sound_data) {
-                       if (NULL != sound_data->data) {
-                               free(sound_data->data);
-                               sound_data->data = NULL;
-                       }
-
-                       free(sound_data);
-                       sound_data = NULL;
-               }
+               ttsd_data_clear_sound_data(&sound_data);
 
                if (NULL == g_playing_info) {
                        SLOG(LOG_ERROR, tts_tag(), "[Player ERROR] Current player is NULL");