rtp: fix nal unit type check
authorLuis de Bethencourt <luis.bg@samsung.com>
Thu, 8 Jan 2015 15:27:44 +0000 (15:27 +0000)
committerTim-Philipp Müller <tim@centricular.com>
Tue, 16 Feb 2016 00:24:40 +0000 (00:24 +0000)
After further investigation the previous commit is wrong. The code intended to
check if the type is 39 or the ranges 41-44 and 48-55. Just like gsth265parse.c
does. Type 40 would not be complete.

gst/rtp/gstrtph265depay.c

index edfdb9b..cb9339d 100644 (file)
@@ -914,7 +914,8 @@ gst_rtp_h265_depay_handle_nal (GstRtpH265Depay * rtph265depay, GstBuffer * nal,
         }
         complete = TRUE;
       } else if ((nal_type >= 32 && nal_type <= 35)
-          || nal_type >= 39) {
+          || nal_type == 39 || (nal_type >= 41 && nal_type <= 44)
+          || (nal_type >= 48 && nal_type <= 55)) {
         /* VPS, SPS, PPS, SEI, ... terminate an access unit */
         complete = TRUE;
       }