From: Seungbae Shin <seungbae.shin@samsung.com>
Date: Wed, 8 Aug 2018 03:51:29 +0000 (+0900)
Subject: Fix coverity issues
X-Git-Tag: submit/tizen/20180808.054211^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F92%2F186192%2F3;p=platform%2Fcore%2Fapi%2Fsound-manager.git

Fix coverity issues

[Version] 0.5.9
[Issue Type] Coverity

Change-Id: I1b3656a8f50b1635c5f99427b0ca598fa95c4a81
---

diff --git a/packaging/capi-media-sound-manager.spec b/packaging/capi-media-sound-manager.spec
index 79e0dc5..6fbd2f7 100644
--- a/packaging/capi-media-sound-manager.spec
+++ b/packaging/capi-media-sound-manager.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-sound-manager
 Summary:    Sound Manager library
-Version:    0.5.8
+Version:    0.5.9
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/sound_manager.c b/src/sound_manager.c
index e178d5b..6c74e77 100644
--- a/src/sound_manager.c
+++ b/src/sound_manager.c
@@ -623,11 +623,10 @@ int sound_manager_get_current_playback_focus(sound_stream_focus_change_reason_e
 			LOGI("                                : reason[%d], flags[0x%x], extra_info[%s]", *acquired_by, option, extra_info_str);
 			*flags = option;
 			if (extra_info)
-				*extra_info = extra_info_str;
-			else
-				SM_SAFE_FREE(extra_info_str);
+				*extra_info = strdup(extra_info_str);
 		}
 		SM_SAFE_FREE(stream_type_str);
+		SM_SAFE_FREE(extra_info_str);
 	}
 
 	return _convert_sound_manager_error_code(__func__, ret);
@@ -662,11 +661,10 @@ int sound_manager_get_current_recording_focus(sound_stream_focus_change_reason_e
 			LOGI("                                 : reason[%d], flags[0x%x], extra_info[%s]", *acquired_by, option, extra_info_str);
 			*flags = option;
 			if (extra_info)
-				*extra_info = extra_info_str;
-			else
-				SM_SAFE_FREE(extra_info_str);
+				*extra_info = strdup(extra_info_str);
 		}
 		SM_SAFE_FREE(stream_type_str);
+		SM_SAFE_FREE(extra_info_str);
 	}
 
 	return _convert_sound_manager_error_code(__func__, ret);