Merge commit '4dfbc7a7559ccab666a8fd39de4224eb4b02c768'
authorHendrik Leppkes <h.leppkes@gmail.com>
Tue, 10 Nov 2015 16:54:06 +0000 (17:54 +0100)
committerHendrik Leppkes <h.leppkes@gmail.com>
Tue, 10 Nov 2015 16:54:06 +0000 (17:54 +0100)
* commit '4dfbc7a7559ccab666a8fd39de4224eb4b02c768':
  msnwc_tcp: Correctly report failure

Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
1  2 
libavformat/msnwc_tcp.c

diff --combined libavformat/msnwc_tcp.c
@@@ -1,20 -1,20 +1,20 @@@
  /*
   * Copyright (C) 2008  Ramiro Polla
   *
 - * 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
   */
  
@@@ -67,7 -67,7 +67,7 @@@ static int msnwc_tcp_probe(AVProbeData 
          }
      }
  
 -    return -1;
 +    return 0;
  }
  
  static int msnwc_tcp_read_header(AVFormatContext *ctx)
  
      /* Some files start with "connected\r\n\r\n".
       * So skip until we find the first byte of struct size */
 -    while(avio_r8(pb) != HEADER_SIZE && !pb->eof_reached);
 +    while(avio_r8(pb) != HEADER_SIZE && !avio_feof(pb));
  
 -    if(pb->eof_reached) {
 -        av_log(ctx, AV_LOG_ERROR, "Could not find valid start.");
 +    if(avio_feof(pb)) {
 +        av_log(ctx, AV_LOG_ERROR, "Could not find valid start.\n");
          return -1;
      }
  
@@@ -104,6 -104,7 +104,7 @@@ static int msnwc_tcp_read_packet(AVForm
      AVIOContext *pb = ctx->pb;
      uint16_t keyframe;
      uint32_t size, timestamp;
+     int ret;
  
      avio_skip(pb, 1); /* one byte has been read ahead */
      avio_skip(pb, 2);
      avio_skip(pb, 4);
      timestamp = avio_rl32(pb);
  
-     if(!size || av_get_packet(pb, pkt, size) != size)
-         return -1;
+     if (!size)
+         return AVERROR_INVALIDDATA;
+     if ((ret = av_get_packet(pb, pkt, size)) < 0)
+         return ret;
  
      avio_skip(pb, 1); /* Read ahead one byte of struct size like read_header */