From: Sebastian Dröge <sebastian@centricular.com>
Date: Tue, 16 Mar 2021 17:25:36 +0000 (+0200)
Subject: rtpjitterbuffer: Fix parsing of the mediaclk:direct= field
X-Git-Tag: 1.19.3~509^2~257
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=00e73e1657ca63de7621994d10fb1612444c109c;p=platform%2Fupstream%2Fgstreamer.git

rtpjitterbuffer: Fix parsing of the mediaclk:direct= field

Due to an off-by-one when parsing the string, the most significant digit
or the clock offset was skipped when parsing the offset.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/merge_requests/907>
---

diff --git a/gst/rtpmanager/gstrtpjitterbuffer.c b/gst/rtpmanager/gstrtpjitterbuffer.c
index 8046988..ab38672 100644
--- a/gst/rtpmanager/gstrtpjitterbuffer.c
+++ b/gst/rtpmanager/gstrtpjitterbuffer.c
@@ -1516,7 +1516,7 @@ gst_jitter_buffer_sink_parse_caps (GstRtpJitterBuffer * jitterbuffer,
       GST_DEBUG_OBJECT (jitterbuffer, "Got media clock %s", mediaclk);
 
       if (!g_str_has_prefix (mediaclk, "direct=") ||
-          !g_ascii_string_to_unsigned (&mediaclk[8], 10, 0, G_MAXUINT64,
+          !g_ascii_string_to_unsigned (&mediaclk[7], 10, 0, G_MAXUINT64,
               &clock_offset, NULL))
         GST_FIXME_OBJECT (jitterbuffer, "Unsupported media clock");
       if (strstr (mediaclk, "rate=") != NULL) {