From: Sangchul Lee <sc11.lee@samsung.com>
Date: Mon, 20 Jun 2022 02:09:38 +0000 (+0900)
Subject: Remove unnecessary null check of a parameter in webrtc_set_stun_server()
X-Git-Tag: submit/tizen/20220623.232237^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=02d8360b79277e339736658d0d47c04e0b6f8003;p=platform%2Fcore%2Fapi%2Fwebrtc.git

Remove unnecessary null check of a parameter in webrtc_set_stun_server()

Default value of the parameter can be null. webrtc_get_stun_server() also
can return the value of null. So, it is fixed as a bug.

[Version] 0.3.126
[Issue Type] Bug fix

Change-Id: I6615690c37b5dc07fed444b909a2ffcd31f31806
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
---

diff --git a/packaging/capi-media-webrtc.spec b/packaging/capi-media-webrtc.spec
index 0b6a079d..d1b00ba1 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:    0.3.125
+Version:    0.3.126
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/webrtc.c b/src/webrtc.c
index 39fe2860..e4161cb1 100644
--- a/src/webrtc.c
+++ b/src/webrtc.c
@@ -857,7 +857,6 @@ int webrtc_set_stun_server(webrtc_h webrtc, const char *stun_server)
 	webrtc_s *_webrtc = (webrtc_s *)webrtc;
 
 	RET_VAL_IF(_webrtc == NULL, WEBRTC_ERROR_INVALID_PARAMETER, "webrtc is NULL");
-	RET_VAL_IF(stun_server == NULL, WEBRTC_ERROR_INVALID_PARAMETER, "stun_server is NULL");
 
 	locker = g_mutex_locker_new(&_webrtc->mutex);