Add a libfuzzer for RtpHeaderParser.

webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h

 /*
  *  Copyright (c) 2012 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 42 matching lines @@
 enum ProtectionType {
   kUnprotectedPacket,
   kProtectedPacket
 };
 
 enum StorageType {
   kDontRetransmit,
   kAllowRetransmission
 };
 
+const int kNumberOfRtpHeaderExtensions = 7;

[phoglund, 2016/06/15 08:20:25]

Maybe just add this as the last entry in the enum? It will get the value 7 and
automatically update when new entries are added. Have a blank line between it
and the others to signfify it's a special entry.

You'll have to bring in an OWNER for rtp_rtcp anyway, and they probably know
better than me.

[katrielc, 2016/06/15 09:05:30]

The problem with adding it to the enum is that it's then a valid enum value,
which you don't want.

(For example, you'd have to change all the switches in the rest of the code to
handle it.)

 enum RTPExtensionType {
   kRtpExtensionNone,
   kRtpExtensionTransmissionTimeOffset,
   kRtpExtensionAudioLevel,
   kRtpExtensionAbsoluteSendTime,
   kRtpExtensionVideoRotation,
   kRtpExtensionTransportSequenceNumber,
   kRtpExtensionPlayoutDelay,
 };
 
@@ skipping 327 matching lines @@
 class TransportSequenceNumberAllocator {
  public:
   TransportSequenceNumberAllocator() {}
   virtual ~TransportSequenceNumberAllocator() {}
 
   virtual uint16_t AllocateSequenceNumber() = 0;
 };
 
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_DEFINES_H_