From 11d583f41484913fd1e7b3e283966eb7b7e11ed2 Mon Sep 17 00:00:00 2001 From: "henrik.lundin" Date: Fri, 18 Sep 2015 01:28:05 -0700 Subject: [PATCH] Fix a bug in RtpFileSource related to RTCP packets in rtpdump files According to http://www.cs.columbia.edu/irt/software/rtptools/#rtpdump, RTCP packets are marked with plen==0. In this class, plen is mapped to original_length, not length. Review URL: https://codereview.webrtc.org/1356543002 Cr-Commit-Position: refs/heads/master@{#9981} --- webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc b/webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc index be3a62bd13..9681ad17ea 100644 --- a/webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc +++ b/webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc @@ -47,7 +47,7 @@ Packet* RtpFileSource::NextPacket() { if (!rtp_reader_->NextPacket(&temp_packet)) { return NULL; } - if (temp_packet.length == 0) { + if (temp_packet.original_length == 0) { // May be an RTCP packet. // Read the next one. continue;