Merge commit 'a4fec9a7eab842ea5eea1b1ee98624356cb31422'
authorClément Bœsch <u@pkh.me>
Wed, 26 Apr 2017 14:34:51 +0000 (16:34 +0200)
committerClément Bœsch <u@pkh.me>
Wed, 26 Apr 2017 14:36:13 +0000 (16:36 +0200)
* commit 'a4fec9a7eab842ea5eea1b1ee98624356cb31422':
  rtmppkt: Check for packet size mismatches

See 7d57ca4d9a75562fa32e40766211de150f8b3ee7

Merged-by: Clément Bœsch <u@pkh.me>
1  2 
libavformat/rtmppkt.c

diff --combined libavformat/rtmppkt.c
@@@ -2,20 -2,20 +2,20 @@@
   * RTMP input format
   * Copyright (c) 2009 Konstantin Shishkov
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -236,11 -236,11 +236,11 @@@ static int rtmp_packet_read_one_chunk(U
          timestamp += prev_pkt[channel_id].timestamp;
  
      if (prev_pkt[channel_id].read && size != prev_pkt[channel_id].size) {
-         av_log(NULL, AV_LOG_ERROR, "RTMP packet size mismatch %d != %d\n",
-                 size,
-                 prev_pkt[channel_id].size);
+         av_log(h, AV_LOG_ERROR, "RTMP packet size mismatch %d != %d\n",
+                                 size, prev_pkt[channel_id].size);
          ff_rtmp_packet_destroy(&prev_pkt[channel_id]);
          prev_pkt[channel_id].read = 0;
+         return AVERROR_INVALIDDATA;
      }
  
      if (!prev_pkt[channel_id].read) {
         prev->data = p->data;
         prev->read = p->read;
         prev->offset = p->offset;
 +       p->data      = NULL;
         return AVERROR(EAGAIN);
      }
  
@@@ -448,7 -447,6 +448,7 @@@ int ff_amf_tag_size(const uint8_t *data
      case AMF_DATA_TYPE_STRING:      return 3 + AV_RB16(data);
      case AMF_DATA_TYPE_LONG_STRING: return 5 + AV_RB32(data);
      case AMF_DATA_TYPE_NULL:        return 1;
 +    case AMF_DATA_TYPE_DATE:        return 11;
      case AMF_DATA_TYPE_ARRAY:
          parse_key = 0;
      case AMF_DATA_TYPE_MIXEDARRAY: