From 385caba166d76537c76abda8642e86bfccb891a4 Mon Sep 17 00:00:00 2001 From: Sangchul Lee Date: Thu, 25 Jul 2024 12:10:07 +0900 Subject: [PATCH] webrtc_source: Add condition to return payload type when destroying source It is to avoid printing error log in some cases. [Version] 1.1.18 [Issue Type] Improvement Change-Id: Idcfcbe670afb1530bdb3d8cabca3f2cfce6d8e9d Signed-off-by: Sangchul Lee --- packaging/capi-media-webrtc.spec | 2 +- src/webrtc_source.c | 11 +++++++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/packaging/capi-media-webrtc.spec b/packaging/capi-media-webrtc.spec index 29030ca9..127303db 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.17 +Version: 1.1.18 Release: 0 Group: Multimedia/API License: Apache-2.0 diff --git a/src/webrtc_source.c b/src/webrtc_source.c index 5716abe6..059bc226 100644 --- a/src/webrtc_source.c +++ b/src/webrtc_source.c @@ -664,6 +664,7 @@ void _source_slot_destroy_cb(gpointer data) webrtc_gst_slot_s *source = (webrtc_gst_slot_s *)data; int i; int j; + unsigned int pt; RET_IF(source == NULL, "source is NULL"); @@ -682,8 +683,9 @@ void _source_slot_destroy_cb(gpointer data) _remove_probe_from_pad_for_pause(source, i); _remove_probe_from_pad_for_render(source, i); - if (source->av[i].pt > 0) - _return_payload_type(source->webrtc, source->av[i].pt); + pt = source->av[i].pt; + if (pt >= MIN_DYNAMIC_PAYLOAD_TYPE && !source->av[i].pt_set_by_api) + _return_payload_type(source->webrtc, pt); _destroy_looopback_render_pipeline(source, i); @@ -698,8 +700,9 @@ void _source_slot_destroy_cb(gpointer data) g_free(source->av[i].payloader_factory_name); for (j = 0; j < source->av[i].num_of_codecs; j++) { - if (source->av[i].multiple_codecs[j].pt > 0) - _return_payload_type(source->webrtc, source->av[i].multiple_codecs[j].pt); + pt = source->av[i].multiple_codecs[j].pt; + if (pt >= MIN_DYNAMIC_PAYLOAD_TYPE && !source->av[i].multiple_codecs[j].pt_set_by_api) + _return_payload_type(source->webrtc, pt); } } -- 2.34.1