From fbbedf32f9f105168929d1af41f00515c2c6b95a Mon Sep 17 00:00:00 2001 From: claincly Date: Thu, 2 Dec 2021 14:14:51 +0000 Subject: [PATCH] Remove the check for dynamic payload type. Issue: google/ExoPlayer#9744 We do not rely on the payload type to determine the sample MIME type, we depend on the SDP message, so it's worthless checking the payload type. After removing the line, a server can use payload type 35 (an unassigned payload type) for H264; while normally H264 requires payload type >= 96). PiperOrigin-RevId: 413658076 --- .../java/androidx/media3/exoplayer/rtsp/RtspMediaTrack.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/libraries/exoplayer_rtsp/src/main/java/androidx/media3/exoplayer/rtsp/RtspMediaTrack.java b/libraries/exoplayer_rtsp/src/main/java/androidx/media3/exoplayer/rtsp/RtspMediaTrack.java index 2604d977c4..5b6b9a4607 100644 --- a/libraries/exoplayer_rtsp/src/main/java/androidx/media3/exoplayer/rtsp/RtspMediaTrack.java +++ b/libraries/exoplayer_rtsp/src/main/java/androidx/media3/exoplayer/rtsp/RtspMediaTrack.java @@ -127,8 +127,6 @@ import com.google.common.collect.ImmutableMap; } checkArgument(clockRate > 0); - // Checks if payload type is "dynamic" as defined in RFC3551 Section 3. - checkArgument(rtpPayloadType >= 96); return new RtpPayloadFormat(formatBuilder.build(), rtpPayloadType, clockRate, fmtpParameters); }