Check pend state in webrtc_start_media_source() 26/317026/1 accepted/tizen_unified_dev accepted/tizen/unified/20240904.172707 accepted/tizen/unified/dev/20240910.111429 accepted/tizen/unified/toolchain/20241004.101915 accepted/tizen/unified/x/20240905.023710 accepted/tizen/unified/x/asan/20241014.000257
authorSangchul Lee <sc11.lee@samsung.com>
Tue, 3 Sep 2024 04:53:28 +0000 (13:53 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Tue, 3 Sep 2024 04:54:10 +0000 (13:54 +0900)
The source must be constructed even if it is called right after
calling webrtc_start() which only changes the pend state inside
of it.

[Version] 1.1.30
[Issue Type] Bug fix

Change-Id: I612b07594ac6d4eb94dfddbe484e94204f31156e
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
packaging/capi-media-webrtc.spec
src/webrtc.c

index 2c72b3559650510f72903119a3c960ab666efbc3..81d74e7b8aff67003d86ea0cbbca7a835ec9925c 100644 (file)
@@ -1,6 +1,6 @@
 Name:       capi-media-webrtc
 Summary:    A WebRTC library in Tizen Native API
-Version:    1.1.29
+Version:    1.1.30
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
index 53a4fa7cbdc0ebe3a818ea407a1f80a2b1560036..6c341a0648ee19b1c4eb44f7d4548063c99a8c73 100644 (file)
@@ -854,7 +854,7 @@ int webrtc_start_media_source(webrtc_h webrtc, unsigned int source_id)
 
        locker = g_mutex_locker_new(&_webrtc->mutex);
 
-       if (_webrtc->state == WEBRTC_STATE_IDLE) {
+       if (_webrtc->state == WEBRTC_STATE_IDLE && _webrtc->pend_state == WEBRTC_STATE_IDLE) {
                LOG_DEBUG("webrtc_start() will take charge of starting media sources");
                return WEBRTC_ERROR_NONE;
        }