rtph26*pay: Update param set timestamp even if parameters unchanged

rtph264pay and rtph265pay skip updating the parameter set timestamp if
the units they see contain no new configuration. This can result in
them injecting duplicate parameters.

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

diff --git a/gst/rtp/gstrtph264pay.c b/gst/rtp/gstrtph264pay.c
index 6e18894..ee46bab 100644 (file)
--- a/gst/rtp/gstrtph264pay.c
+++ b/gst/rtp/gstrtph264pay.c
@@ -716,7 +716,7 @@ gst_rtp_h264_pay_decode_nal (GstRtpH264Pay * payloader,
         payloader->sps, payloader->pps, nal);
 
     /* remember when we last saw SPS */
-    if (updated && pts != -1)
+    if (pts != -1)
       payloader->last_spspps =
           gst_segment_to_running_time (&GST_RTP_BASE_PAYLOAD_CAST
           (payloader)->segment, GST_FORMAT_TIME, pts);

diff --git a/gst/rtp/gstrtph265pay.c b/gst/rtp/gstrtph265pay.c
index 1f366b7..733612e 100644 (file)
--- a/gst/rtp/gstrtph265pay.c
+++ b/gst/rtp/gstrtph265pay.c
@@ -801,7 +801,7 @@ gst_rtp_h265_pay_decode_nal (GstRtpH265Pay * payloader,
         payloader->vps, payloader->sps, payloader->pps, nal);
 
     /* remember when we last saw VPS */
-    if (updated && pts != -1)
+    if (pts != -1)
       payloader->last_vps_sps_pps =
           gst_segment_to_running_time (&GST_RTP_BASE_PAYLOAD_CAST
           (payloader)->segment, GST_FORMAT_TIME, pts);