webrtcstats: Properly report IceCandidate type
authorPhilippe Normand <philn@igalia.com>
Fri, 30 Jun 2023 17:30:46 +0000 (18:30 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Mon, 3 Jul 2023 08:17:01 +0000 (09:17 +0100)
strcmp returns a positive value if s1 is greater than s2, while we actually
needed to check equality here.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/4957>

subprojects/gst-plugins-bad/ext/webrtc/gstwebrtcstats.c

index db922f0..153ade6 100644 (file)
@@ -581,9 +581,9 @@ _get_stats_from_ice_candidates (GstWebRTCBin * webrtc,
       can->stream_id, can->ipaddr, can->port);
   stats = gst_structure_new_empty (id);
 
-  if (strcmp (candidate_tag, "local")) {
+  if (g_str_equal (candidate_tag, "local")) {
     type = GST_WEBRTC_STATS_LOCAL_CANDIDATE;
-  } else if (strcmp (candidate_tag, "remote")) {
+  } else if (g_str_equal (candidate_tag, "remote")) {
     type = GST_WEBRTC_STATS_REMOTE_CANDIDATE;
   } else {
     GST_WARNING_OBJECT (webrtc, "Invalid ice candidate tag: %s", candidate_tag);