[FFmpeg-cvslog] rtpdec_hevc: Drop extra sanity check for size of input packet

Vittorio Giovara git at videolan.org
Mon Mar 9 20:16:39 CET 2015


ffmpeg | branch: master | Vittorio Giovara <vittorio.giovara at gmail.com> | Sun Mar  8 23:42:59 2015 +0000| [bfeb83a8b7d3fcf09a54d8dbc9c521e10bb17530] | committer: Vittorio Giovara

rtpdec_hevc: Drop extra sanity check for size of input packet

In this case len is always at least 3, since it is checked against
RTP_HEVC_PAYLOAD_HEADER_SIZE + 1 before entering the switch block.

Bug-Id: CID 1238784

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=bfeb83a8b7d3fcf09a54d8dbc9c521e10bb17530
---

 libavformat/rtpdec_hevc.c |    8 --------
 1 file changed, 8 deletions(-)

diff --git a/libavformat/rtpdec_hevc.c b/libavformat/rtpdec_hevc.c
index 4ec9767..ef47388 100644
--- a/libavformat/rtpdec_hevc.c
+++ b/libavformat/rtpdec_hevc.c
@@ -245,14 +245,6 @@ static int hevc_handle_packet(AVFormatContext *ctx, PayloadContext *rtp_hevc_ctx
     case 39:
     /* single NAL unit packet */
     default:
-        /* sanity check for size of input packet: 1 byte payload at least */
-        if (len < 1) {
-            av_log(ctx, AV_LOG_ERROR,
-                   "Too short RTP/HEVC packet, got %d bytes of NAL unit type %d\n",
-                   len, nal_type);
-            return AVERROR_INVALIDDATA;
-        }
-
         /* create A/V packet */
         if ((res = av_new_packet(pkt, sizeof(start_sequence) + len)) < 0)
             return res;



More information about the ffmpeg-cvslog mailing list