rtpsession: Fix collection of statistics
authorSantiago Carot-Nemesio <sancane@gmail.com>
Thu, 21 May 2015 07:35:58 +0000 (09:35 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Thu, 21 May 2015 09:56:12 +0000 (12:56 +0300)
Stats should be collected on the media rtp source not in the
sender one.

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

gst/rtpmanager/rtpsession.c

index 0acac15..d14c03a 100644 (file)
@@ -2390,8 +2390,6 @@ rtp_session_process_pli (RTPSession * sess, guint32 sender_ssrc,
     return;
 
   rtp_session_request_local_key_unit (sess, src, FALSE, current_time);
-
-  src->stats.recv_pli_count++;
 }
 
 static void
@@ -2448,7 +2446,6 @@ rtp_session_process_fir (RTPSession * sess, guint32 sender_ssrc,
     return;
 
   rtp_session_request_local_key_unit (sess, src, TRUE, current_time);
-  src->stats.recv_fir_count++;
 }
 
 static void
@@ -2531,10 +2528,12 @@ rtp_session_process_feedback (RTPSession * sess, GstRTCPPacket * packet,
       case GST_RTCP_TYPE_PSFB:
         switch (fbtype) {
           case GST_RTCP_PSFB_TYPE_PLI:
+            src->stats.recv_pli_count++;
             rtp_session_process_pli (sess, sender_ssrc, media_ssrc,
                 current_time);
             break;
           case GST_RTCP_PSFB_TYPE_FIR:
+            src->stats.recv_fir_count++;
             rtp_session_process_fir (sess, sender_ssrc, fci_data, fci_length,
                 current_time);
             break;