From: Sangchul Lee <sc11.lee@samsung.com>
Date: Tue, 3 Sep 2024 04:53:28 +0000 (+0900)
Subject: Check pend state in webrtc_start_media_source()
X-Git-Tag: accepted/tizen/unified/20240904.172707^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fheads%2Faccepted%2Ftizen_unified_dev;p=platform%2Fcore%2Fapi%2Fwebrtc.git

Check pend state in webrtc_start_media_source()

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>
---

diff --git a/packaging/capi-media-webrtc.spec b/packaging/capi-media-webrtc.spec
index 2c72b355..81d74e7b 100644
--- a/packaging/capi-media-webrtc.spec
+++ b/packaging/capi-media-webrtc.spec
@@ -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
diff --git a/src/webrtc.c b/src/webrtc.c
index 53a4fa7c..6c341a06 100644
--- a/src/webrtc.c
+++ b/src/webrtc.c
@@ -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;
 	}