Remove unnecessary null check of a parameter in webrtc_set_stun_server() 19/276519/2 submit/tizen/20220623.232237
authorSangchul Lee <sc11.lee@samsung.com>
Mon, 20 Jun 2022 02:09:38 +0000 (11:09 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Tue, 21 Jun 2022 05:41:20 +0000 (14:41 +0900)
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>
packaging/capi-media-webrtc.spec
src/webrtc.c

index 0b6a079d7ceb849fa47e622fcc5426783b1820e3..d1b00ba180103aa07925a1ee47648cb332c7e14e 100644 (file)
@@ -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
index 39fe28609cb397ecaf0be372b37c32706dedd8c8..e4161cb14739e3be637754f967c616f740f32e6b 100644 (file)
@@ -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);