SignalPacketReceived should pass packet as a pointer instead of a non-const reference.

webrtc/pc/rtptransport.cc

 /*
  *  Copyright 2017 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 172 matching lines @@
   // transport. We check the RTP payload type to determine if it is RTCP.
   bool rtcp = transport == rtcp_packet_transport() ||
               IsRtcp(data, static_cast<int>(len));
   rtc::CopyOnWriteBuffer packet(data, len);
 
   if (!WantsPacket(rtcp, &packet)) {
     return;
   }
 
   // This mutates |packet| if it is protected.
-  SignalPacketReceived(rtcp, packet, packet_time);
+  SignalPacketReceived(rtcp, &packet, packet_time);
 }
 
 bool RtpTransport::WantsPacket(bool rtcp,
                                const rtc::CopyOnWriteBuffer* packet) {
   // Protect ourselves against crazy data.
   if (!packet || !cricket::IsValidRtpRtcpPacketSize(rtcp, packet->size())) {
     LOG(LS_ERROR) << "Dropping incoming " << cricket::RtpRtcpStringLiteral(rtcp)
                   << " packet: wrong size=" << packet->size();
     return false;
   }
   if (rtcp) {
     // Permit all (seemingly valid) RTCP packets.
     return true;
   }
   // Check whether we handle this payload.
   return HandlesPacket(packet->data(), packet->size());
 }
 
 }  // namespace webrtc