[Bug 780446] New: gst_rtp_buffer_ext_timestamp() math is incorrect for big jumps

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=780446

            Bug ID: 780446
           Summary: gst_rtp_buffer_ext_timestamp() math is incorrect for
                    big jumps
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: gkiagia at tolabaki.gr
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Created attachment 348572
  --> https://bugzilla.gnome.org/attachment.cgi?id=348572&action=edit
gstrtpbuffer: fix math in gst_rtp_buffer_ext_timestamp()

The math in gst_rtp_buffer_ext_timestamp() is not entirely correct. When doing
big jumps it will produce strange results:

guint64 ext = 0;
gst_rtp_buffer_ext_timestamp (&ext, G_MAXINT32 + 1);
-> ext should now be G_MAXINT32 + 1
-> before this change it would be G_MAXUINT32 + G_MAXINT32 + 2

guint64 ext = 1087500;
gst_rtp_buffer_ext_timestamp (&ext, 24);
> ext should now be G_MAXUINT32 + 1 + 24
-> before this change it would be just 24

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.