rtpdec_hevc: Drop extra sanity check for size of input packet
authorVittorio Giovara <vittorio.giovara@gmail.com>
Sun, 8 Mar 2015 23:42:59 +0000 (23:42 +0000)
committerVittorio Giovara <vittorio.giovara@gmail.com>
Mon, 9 Mar 2015 12:57:01 +0000 (12:57 +0000)
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

libavformat/rtpdec_hevc.c

index 4ec9767..ef47388 100644 (file)
@@ -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;