aboutsummaryrefslogtreecommitdiff
path: root/plugins
diff options
context:
space:
mode:
authorMarvin W <git@larma.de>2021-05-15 19:52:39 +0200
committerMarvin W <git@larma.de>2021-05-15 19:55:44 +0200
commit686035ca1ee65d8180585d0e8c4b30faa0caacc4 (patch)
tree7dc4d5198905cdfb0185e2b45d3e66c803122537 /plugins
parent6399892bf2c53fc3701438c752718638baff8e27 (diff)
downloaddino-686035ca1ee65d8180585d0e8c4b30faa0caacc4.tar.gz
dino-686035ca1ee65d8180585d0e8c4b30faa0caacc4.zip
RTP: Handle missing rtp pay/depay elements
Diffstat (limited to 'plugins')
-rw-r--r--plugins/rtp/src/codec_util.vala24
-rw-r--r--plugins/rtp/src/module.vala6
2 files changed, 16 insertions, 14 deletions
diff --git a/plugins/rtp/src/codec_util.vala b/plugins/rtp/src/codec_util.vala
index d018501d..417dc4be 100644
--- a/plugins/rtp/src/codec_util.vala
+++ b/plugins/rtp/src/codec_util.vala
@@ -212,7 +212,8 @@ public class Dino.Plugins.Rtp.CodecUtil {
return null;
}
- public bool is_element_supported(string element_name) {
+ public bool is_element_supported(string? element_name) {
+ if (element_name == null) return false;
if (unsupported_elements.contains(element_name)) return false;
if (supported_elements.contains(element_name)) return true;
var test_element = Gst.ElementFactory.make(element_name, @"test-$element_name");
@@ -220,14 +221,14 @@ public class Dino.Plugins.Rtp.CodecUtil {
supported_elements.add(element_name);
return true;
} else {
- debug("%s is not supported on this platform", element_name);
+ warning("%s is not supported on this platform", element_name);
unsupported_elements.add(element_name);
return false;
}
}
public string? get_encode_element_name(string media, string? codec) {
- if (!is_element_supported(get_pay_element_name(media, codec))) return null;
+ if (get_pay_element_name(media, codec) == null) return null;
foreach (string candidate in get_encode_candidates(media, codec)) {
if (is_element_supported(candidate)) return candidate;
}
@@ -235,12 +236,13 @@ public class Dino.Plugins.Rtp.CodecUtil {
}
public string? get_pay_element_name(string media, string? codec) {
- string candidate = get_pay_candidate(media, codec);
- if (is_element_supported(candidate)) return candidate;
+ string? candidate = get_pay_candidate(media, codec);
+ if (candidate != null && is_element_supported(candidate)) return candidate;
return null;
}
public string? get_decode_element_name(string media, string? codec) {
+ if (get_depay_element_name(media, codec) == null) return null;
foreach (string candidate in get_decode_candidates(media, codec)) {
if (is_element_supported(candidate)) return candidate;
}
@@ -248,8 +250,8 @@ public class Dino.Plugins.Rtp.CodecUtil {
}
public string? get_depay_element_name(string media, string? codec) {
- string candidate = get_depay_candidate(media, codec);
- if (is_element_supported(candidate)) return candidate;
+ string? candidate = get_depay_candidate(media, codec);
+ if (candidate != null && is_element_supported(candidate)) return candidate;
return null;
}
@@ -260,8 +262,8 @@ public class Dino.Plugins.Rtp.CodecUtil {
public string? get_decode_bin_description(string media, string? codec, JingleRtp.PayloadType? payload_type, string? element_name = null, string? name = null) {
if (codec == null) return null;
string base_name = name ?? @"encode-$codec-$(Random.next_int())";
- string depay = get_depay_element_name(media, codec);
- string decode = element_name ?? get_decode_element_name(media, codec);
+ string? depay = get_depay_element_name(media, codec);
+ string? decode = element_name ?? get_decode_element_name(media, codec);
if (depay == null || decode == null) return null;
string decode_prefix = get_decode_prefix(media, codec, decode, payload_type) ?? "";
string decode_args = get_decode_args(media, codec, decode, payload_type) ?? "";
@@ -285,8 +287,8 @@ public class Dino.Plugins.Rtp.CodecUtil {
public string? get_encode_bin_description(string media, string? codec, JingleRtp.PayloadType? payload_type, string? element_name = null, string? name = null) {
if (codec == null) return null;
string base_name = name ?? @"encode_$(codec)_$(Random.next_int())";
- string pay = get_pay_element_name(media, codec);
- string encode = element_name ?? get_encode_element_name(media, codec);
+ string? pay = get_pay_element_name(media, codec);
+ string? encode = element_name ?? get_encode_element_name(media, codec);
if (pay == null || encode == null) return null;
string encode_prefix = get_encode_prefix(media, codec, encode, payload_type) ?? "";
string encode_args = get_encode_args(media, codec, encode, payload_type) ?? "";
diff --git a/plugins/rtp/src/module.vala b/plugins/rtp/src/module.vala
index 19a7501d..0edbb3e1 100644
--- a/plugins/rtp/src/module.vala
+++ b/plugins/rtp/src/module.vala
@@ -72,7 +72,7 @@ public class Dino.Plugins.Rtp.Module : JingleRtp.Module {
string? encode_element = codec_util.get_encode_element_name(media, codec);
string? decode_element = codec_util.get_decode_element_name(media, codec);
if (encode_element == null || decode_element == null) {
- debug("No suitable encoder or decoder found for %s", codec);
+ warning("No suitable encoder or decoder found for %s", codec);
unsupported_codecs.add(codec);
return false;
}
@@ -83,7 +83,7 @@ public class Dino.Plugins.Rtp.Module : JingleRtp.Module {
codec_util.mark_element_unsupported(encode_element);
encode_element = codec_util.get_encode_element_name(media, codec);
if (encode_element == null) {
- debug("No suitable encoder found for %s", codec);
+ warning("No suitable encoder found for %s", codec);
unsupported_codecs.add(codec);
return false;
}
@@ -97,7 +97,7 @@ public class Dino.Plugins.Rtp.Module : JingleRtp.Module {
codec_util.mark_element_unsupported(decode_element);
decode_element = codec_util.get_decode_element_name(media, codec);
if (decode_element == null) {
- debug("No suitable decoder found for %s", codec);
+ warning("No suitable decoder found for %s", codec);
unsupported_codecs.add(codec);
return false;
}