fixup! Add mutex to stream_info structure for virtual stream APIs 18/235918/3 accepted/tizen/unified/20200612.022353 accepted/tizen/unified/20200618.130038 submit/tizen/20200611.091235 submit/tizen/20200617.090424 submit/tizen/20200619.022637
authorJaechul Lee <jcsing.lee@samsung.com>
Thu, 11 Jun 2020 03:40:00 +0000 (12:40 +0900)
committerJaechul Lee <jcsing.lee@samsung.com>
Thu, 11 Jun 2020 05:37:27 +0000 (14:37 +0900)
sound_manager_destroy_stream_information is not allowed to return INVALID_STATE

[Version] 0.6.8
[Issue Type] None

Change-Id: Ie7c68818bc0cee7521f79de046f7573d6e5cf85e
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
packaging/capi-media-sound-manager.spec
src/sound_manager.c
src/sound_manager_internal.c

index cac5194..944764e 100644 (file)
@@ -1,6 +1,6 @@
 Name:       capi-media-sound-manager
 Summary:    Sound Manager library
-Version:    0.6.7
+Version:    0.6.8
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
index e36578a..e90c738 100644 (file)
@@ -193,7 +193,7 @@ int sound_manager_destroy_stream_information(sound_stream_info_h stream_info)
        if (stream_h->vstream) {
                LOGE("virtual stream is still alive");
                SM_LEAVE_CRITICAL_SECTION(&stream_h->vstream_mutex);
-               return _convert_sound_manager_error_code(__func__, MM_ERROR_SOUND_INVALID_STATE);
+               return _convert_sound_manager_error_code(__func__, MM_ERROR_SOUND_INVALID_OPERATION);
        }
        SM_LEAVE_CRITICAL_SECTION(&stream_h->vstream_mutex);
 
index 6a9dde5..77be664 100644 (file)
@@ -489,7 +489,7 @@ int sound_manager_create_virtual_stream(sound_stream_info_h stream_info, virtual
        if (stream_h->vstream) {
                LOGE("virtual stream was already created. stream_h(%p), vstream(%p)", stream_h, stream_h->vstream);
                SM_LEAVE_CRITICAL_SECTION(&stream_h->vstream_mutex);
-               return _convert_sound_manager_error_code(__func__, MM_ERROR_SOUND_INVALID_STATE);
+               return _convert_sound_manager_error_code(__func__, MM_ERROR_SOUND_INVALID_OPERATION);
        }
 
        ret = _create_virtual_stream(stream_h, &vstream_h);