From: Sebastian Dröge Date: Thu, 22 Feb 2024 11:29:19 +0000 (+0200) Subject: rtp: Fix constant for maximum two-byte RTP header extension length X-Git-Tag: 1.24.0~45 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=eb641af4504a66c47e16c0f559120de7f7fc9dc4;p=platform%2Fupstream%2Fgstreamer.git rtp: Fix constant for maximum two-byte RTP header extension length The value is stored as an 8 bit integer, with 0 meaning that there is not data for this extension. That means that the maximum length is 255 bytes and not 256 bytes. On the other hand, the one-byte RTP header extensions are storing the length as a 4 bit integer with an offset of 1 (i.e. 0 means 1 byte extension length), so here 16 is the correct maximum length. Part-of: --- diff --git a/subprojects/gst-plugins-base/gst-libs/gst/rtp/gstrtpbasepayload.c b/subprojects/gst-plugins-base/gst-libs/gst/rtp/gstrtpbasepayload.c index 648b34e153..4939001db7 100644 --- a/subprojects/gst-plugins-base/gst-libs/gst/rtp/gstrtpbasepayload.c +++ b/subprojects/gst-plugins-base/gst-libs/gst/rtp/gstrtpbasepayload.c @@ -114,7 +114,7 @@ static guint gst_rtp_base_payload_signals[LAST_SIGNAL] = { 0 }; #define DEFAULT_AUTO_HEADER_EXTENSION TRUE #define RTP_HEADER_EXT_ONE_BYTE_MAX_SIZE 16 -#define RTP_HEADER_EXT_TWO_BYTE_MAX_SIZE 256 +#define RTP_HEADER_EXT_TWO_BYTE_MAX_SIZE 255 #define RTP_HEADER_EXT_ONE_BYTE_MAX_ID 14 #define RTP_HEADER_EXT_TWO_BYTE_MAX_ID 255