From: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Date: Sun, 27 Jan 2019 02:02:27 +0000 (-0500)
Subject: rtph265depay; Fix handling of marker on aggregated packet
X-Git-Tag: 1.16.2~174
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6d3859bf70a8bf29e97aeb107954967720f76e11;p=platform%2Fupstream%2Fgst-plugins-good.git

rtph265depay; Fix handling of marker on aggregated packet

When multiple nals are aggrgated, the marker bit should be associated only
with the last NAL of the packet. Otherwise we may break rendering in with
AU alignment.
---

diff --git a/gst/rtp/gstrtph265depay.c b/gst/rtp/gstrtph265depay.c
index f344a06..551e08a 100644
--- a/gst/rtp/gstrtph265depay.c
+++ b/gst/rtp/gstrtph265depay.c
@@ -1363,6 +1363,7 @@ gst_rtp_h265_depay_process (GstRTPBaseDepayload * depayload, GstRTPBuffer * rtp)
 #endif
 
         while (payload_len > 2) {
+          gboolean last = FALSE;
 
           nalu_size = (payload[0] << 8) | payload[1];
 
@@ -1389,8 +1390,11 @@ gst_rtp_h265_depay_process (GstRTPBaseDepayload * depayload, GstRTPBuffer * rtp)
 
           gst_rtp_copy_video_meta (rtph265depay, outbuf, rtp->buffer);
 
+          if (payload_len - nalu_size <= 2)
+            last = TRUE;
+
           gst_rtp_h265_depay_handle_nal (rtph265depay, outbuf, timestamp,
-              marker);
+              marker && last);
 
           payload += nalu_size;
           payload_len -= nalu_size;