Fix build warnings

[Version] 0.1.13
[Issue Type] Build

Change-Id: I3fda37e4c7e05c9e681488517c2a6e22fc28fe57
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>

diff --git a/hal-backend-audio.c b/hal-backend-audio.c
index 881d082..aa44540 100644 (file)
--- a/hal-backend-audio.c
+++ b/hal-backend-audio.c
@@ -40,12 +40,14 @@ static int audio_emul_init(void **data)
     return 0;
 }
 
-static void audio_emul_exit(void *data)
+static int audio_emul_exit(void *data)
 {
     if (!data)
-        return;
+        return -EINVAL;
 
     free(data);
+
+    return 0;
 }
 
 hal_backend hal_backend_audio_data = {

diff --git a/packaging/audio-hal-emul.spec b/packaging/audio-hal-emul.spec
index 5251cbb..ef8b047 100644 (file)
--- a/packaging/audio-hal-emul.spec
+++ b/packaging/audio-hal-emul.spec
@@ -1,6 +1,6 @@
 Name:       audio-hal-emul
 Summary:    TIZEN Audio HAL for Emulator
-Version:    0.1.12
+Version:    0.1.13
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0

diff --git a/tizen-audio-volume.c b/tizen-audio-volume.c
index d31e29f..820a497 100644 (file)
--- a/tizen-audio-volume.c
+++ b/tizen-audio-volume.c
@@ -181,8 +181,15 @@ static audio_return_e __load_volume_value_table_from_ini(audio_hal_t *ah)
     uint32_t vol_type_idx, vol_level_idx, gain_type_idx;
     audio_volume_value_table_t *volume_value_table = ah->volume.volume_value_table;
     int size = 0;
+    const char delimiter[] = ", ";
+    const char *table_str = "volumes";
+    const char *tmp_str = NULL;
+    const char *gain_str = NULL;
+    char *list_str = NULL, *ptr = NULL;
+    char *key, *token;
 
-    dict = iniparser_load(VOLUME_INI_TEMP_PATH);
+    if (access(VOLUME_INI_TEMP_PATH, F_OK) == 0)
+        dict = iniparser_load(VOLUME_INI_TEMP_PATH);
     if (!dict) {
         AUDIO_LOG_DEBUG("Use default volume&gain ini file");
         dict = iniparser_load(VOLUME_INI_DEFAULT_PATH);
@@ -192,10 +199,6 @@ static audio_return_e __load_volume_value_table_from_ini(audio_hal_t *ah)
         }
     }
 
-    const char delimiter[] = ", ";
-    char *key, *list_str, *token, *ptr = NULL;
-    const char *table_str = "volumes";
-
     /* Load volume table */
     for (vol_type_idx = 0; vol_type_idx < AUDIO_VOLUME_TYPE_MAX; vol_type_idx++) {
         const char *vol_type_str = __get_volume_type_string_by_idx(vol_type_idx);
@@ -205,7 +208,9 @@ static audio_return_e __load_volume_value_table_from_ini(audio_hal_t *ah)
         key = malloc(size);
         if (key) {
             snprintf(key, size, "%s:%s", table_str, vol_type_str);
-            list_str = iniparser_getstring(dict, key, NULL);
+            if ((tmp_str = iniparser_getstring(dict, key, NULL)))
+                list_str = strdup(tmp_str);
+
             if (list_str) {
                 token = strtok_r(list_str, delimiter, &ptr);
                 while (token) {
@@ -216,6 +221,8 @@ static audio_return_e __load_volume_value_table_from_ini(audio_hal_t *ah)
                     volume_value_table->volume[vol_type_idx][ah->volume.volume_level_max[vol_type_idx]++] = vol_value;
                     token = strtok_r(NULL, delimiter, &ptr);
                 }
+                free(list_str);
+                list_str = NULL;
             } else {
                 ah->volume.volume_level_max[vol_type_idx] = 1;
                 for (vol_level_idx = 0; vol_level_idx < AUDIO_VOLUME_LEVEL_MAX; vol_level_idx++) {
@@ -235,9 +242,9 @@ static audio_return_e __load_volume_value_table_from_ini(audio_hal_t *ah)
         key = malloc(size);
         if (key) {
             snprintf(key, size, "%s:gain_%s", table_str, gain_type_str);
-            token = iniparser_getstring(dict, key, NULL);
-            if (token) {
-                volume_value_table->gain[gain_type_idx] = atof(token);
+            gain_str = iniparser_getstring(dict, key, NULL);
+            if (gain_str) {
+                volume_value_table->gain[gain_type_idx] = atof(gain_str);
             } else {
                 volume_value_table->gain[gain_type_idx] = GAIN_VALUE_MAX;
             }