Merge "add logs for sttd" into tizen
authorSooyeon Kim <sooyeon.kim@samsung.com>
Mon, 15 Apr 2019 08:57:36 +0000 (08:57 +0000)
committerGerrit Code Review <gerrit@review.ap-northeast-2.compute.internal>
Mon, 15 Apr 2019 08:57:36 +0000 (08:57 +0000)
common/stt_config_mgr.c
common/stt_config_parser.c
server/sttd_config.c
server/sttd_engine_agent.c

index 2314cbb..6f3c474 100644 (file)
@@ -1041,6 +1041,7 @@ int stt_config_mgr_get_engine(char** engine)
                *engine = strdup(g_config_info->engine_id);
        } else {
                SLOG(LOG_ERROR, stt_tag(), " Engine id is NULL"); //LCOV_EXCL_LINE
+               *engine = NULL;
                return STT_CONFIG_ERROR_ENGINE_NOT_FOUND;
        }
 
@@ -1514,7 +1515,7 @@ bool stt_config_check_default_engine_is_valid(const char* engine)
        while (NULL != iter) {
                engine_info = iter->data;
 
-               if (NULL != engine_info) {
+               if (NULL != engine_info && NULL != engine_info->uuid) {
                        if (0 == strcmp(engine, engine_info->uuid)) {
                                return true;
                        }
index 8b45fc3..b179ffc 100644 (file)
@@ -352,6 +352,9 @@ int stt_parser_load_config(stt_config_s** config_info)
 
                        if (STT_RETRY_COUNT == retry_count) {
                                SLOG(LOG_ERROR, stt_tag(), "[ERROR] Fail to parse file error : %s", STT_CONFIG);
+                               SLOG(LOG_ERROR, stt_tag(), "[ERROR] Remove the file");
+                               remove(STT_CONFIG);
+
                                doc = xmlParseFile(STT_DEFAULT_CONFIG);
                                if (NULL == doc) {
                                        SLOG(LOG_ERROR, stt_tag(), "[ERROR] Fail to parse file error : %s", STT_DEFAULT_CONFIG);
index 7d836dd..982c897 100644 (file)
@@ -119,6 +119,7 @@ int sttd_config_get_default_engine(char** engine_id)
 
        if (0 != stt_config_mgr_get_engine(engine_id)) {
                SLOG(LOG_ERROR, TAG_STTD, "[Config ERROR] Fail to get engine id");
+               return -1;
        }
 
        return 0;
index 62866b2..9b68876 100644 (file)
@@ -302,7 +302,7 @@ int sttd_engine_agent_load_current_engine(stte_request_callback_s *callback)
        /* get current engine from config */
        if (0 == sttd_config_get_default_engine(&cur_engine_uuid)) {
                SECURE_SLOG(LOG_DEBUG, TAG_STTD, "[Engine Agent] current engine from config : %s", cur_engine_uuid);
-               if (NULL != g_engine_info->engine_uuid) {
+               if (NULL != g_engine_info->engine_uuid && NULL != cur_engine_uuid) {
                        if (!strcmp(g_engine_info->engine_uuid, cur_engine_uuid)) {
                                is_default_engine = true;
                        }