From 93cd4be8d595eb7f540297a8442d36006f05d373 Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Mon, 22 Feb 2016 14:03:02 +0000 Subject: [PATCH] rtpvp9pay: add missing break VP9_PAY_PICTURE_ID_7BITS and VP9_PAY_PICTURE_ID_15BITS are mutually exclusive options of the picture-id-mode. We can break after the first case. 1 or 2 bytes need to be added to the header length depending on the PictureID size. https://tools.ietf.org/html/draft-uberti-payload-vp9-00#section-4.2 CID 1353479 --- gst/rtp/gstrtpvp9pay.c | 1 + 1 file changed, 1 insertion(+) diff --git a/gst/rtp/gstrtpvp9pay.c b/gst/rtp/gstrtpvp9pay.c index 281ceaa..9f80fe5 100644 --- a/gst/rtp/gstrtpvp9pay.c +++ b/gst/rtp/gstrtpvp9pay.c @@ -308,6 +308,7 @@ gst_rtp_vp9_calc_header_len (GstRtpVP9Pay * self, gboolean start) switch (self->picture_id_mode) { case VP9_PAY_PICTURE_ID_7BITS: len += 1; + break; case VP9_PAY_PICTURE_ID_15BITS: len += 2; default: -- 2.7.4