Merge commit 'f412b2c9f3a7add0ab8021262ec4bad249347e30'
authorMichael Niedermayer <michaelni@gmx.at>
Fri, 20 Dec 2013 23:11:59 +0000 (00:11 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 20 Dec 2013 23:14:11 +0000 (00:14 +0100)
* commit 'f412b2c9f3a7add0ab8021262ec4bad249347e30':
  flv: Use the correct type to hold the file offset

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavformat/flvenc.c

diff --combined libavformat/flvenc.c
@@@ -1,43 -1,40 +1,43 @@@
  /*
   * FLV muxer
 - * Copyright (c) 2003 The Libav Project
 + * Copyright (c) 2003 The FFmpeg Project
   *
 - * 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
   */
  
 +#include "libavutil/intreadwrite.h"
  #include "libavutil/dict.h"
  #include "libavutil/intfloat.h"
 +#include "libavutil/avassert.h"
  #include "avc.h"
  #include "avformat.h"
  #include "flv.h"
  #include "internal.h"
  #include "metadata.h"
  
 -#undef NDEBUG
 -#include <assert.h>
  
  static const AVCodecTag flv_video_codec_ids[] = {
      { AV_CODEC_ID_FLV1,     FLV_CODECID_H263 },
 +    { AV_CODEC_ID_H263,     FLV_CODECID_REALH263 },
 +    { AV_CODEC_ID_MPEG4,    FLV_CODECID_MPEG4 },
      { AV_CODEC_ID_FLASHSV,  FLV_CODECID_SCREEN },
      { AV_CODEC_ID_FLASHSV2, FLV_CODECID_SCREEN2 },
      { AV_CODEC_ID_VP6F,     FLV_CODECID_VP6 },
 +    { AV_CODEC_ID_VP6,      FLV_CODECID_VP6 },
      { AV_CODEC_ID_VP6A,     FLV_CODECID_VP6A },
      { AV_CODEC_ID_H264,     FLV_CODECID_H264 },
      { AV_CODEC_ID_NONE,     0 }
@@@ -80,12 -77,12 +80,12 @@@ static int get_audio_flags(AVFormatCont
      else if (enc->codec_id == AV_CODEC_ID_SPEEX) {
          if (enc->sample_rate != 16000) {
              av_log(s, AV_LOG_ERROR,
 -                   "flv only supports wideband (16kHz) Speex audio\n");
 -            return -1;
 +                   "FLV only supports wideband (16kHz) Speex audio\n");
 +            return AVERROR(EINVAL);
          }
          if (enc->channels != 1) {
 -            av_log(s, AV_LOG_ERROR, "flv only supports mono Speex audio\n");
 -            return -1;
 +            av_log(s, AV_LOG_ERROR, "FLV only supports mono Speex audio\n");
 +            return AVERROR(EINVAL);
          }
          return FLV_CODECID_SPEEX | FLV_SAMPLERATE_11025HZ | FLV_SAMPLESSIZE_16BIT;
      } else {
              }
          default:
              av_log(s, AV_LOG_ERROR,
 -                   "flv does not support that sample rate, "
 -                   "choose from (44100, 22050, 11025).\n");
 -            return -1;
 +                   "FLV does not support sample rate %d, "
 +                   "choose from (44100, 22050, 11025)\n", enc->sample_rate);
 +            return AVERROR(EINVAL);
          }
      }
  
          flags |= enc->codec_tag << 4;
          break;
      default:
 -        av_log(s, AV_LOG_ERROR, "codec not compatible with flv\n");
 -        return -1;
 +        av_log(s, AV_LOG_ERROR, "Audio codec '%s' not compatible with FLV\n",
 +               avcodec_get_name(enc->codec_id));
 +        return AVERROR(EINVAL);
      }
  
      return flags;
@@@ -219,9 -215,8 +219,9 @@@ static int flv_write_header(AVFormatCon
              }
              video_enc = enc;
              if (enc->codec_tag == 0) {
 -                av_log(s, AV_LOG_ERROR, "video codec not compatible with flv\n");
 -                return -1;
 +                av_log(s, AV_LOG_ERROR, "Video codec '%s' for stream %d is not compatible with FLV\n",
 +                       avcodec_get_name(enc->codec_id), i);
 +                return AVERROR(EINVAL);
              }
              break;
          case AVMEDIA_TYPE_AUDIO:
              break;
          case AVMEDIA_TYPE_DATA:
              if (enc->codec_id != AV_CODEC_ID_TEXT) {
 -                av_log(s, AV_LOG_ERROR, "codec not compatible with flv\n");
 +                av_log(s, AV_LOG_ERROR, "Data codec '%s' for stream %d is not compatible with FLV\n",
 +                       avcodec_get_name(enc->codec_id), i);
                  return AVERROR_INVALIDDATA;
              }
              data_enc = enc;
              break;
          default:
 -            av_log(s, AV_LOG_ERROR, "codec not compatible with flv\n");
 -            return -1;
 +            av_log(s, AV_LOG_ERROR, "Codec type '%s' for stream %d is not compatible with FLV\n",
 +                   av_get_media_type_string(enc->codec_type), i);
 +            return AVERROR(EINVAL);
          }
          avpriv_set_pts_info(s->streams[i], 32, 1, 1000); /* 32 bit pts in ms */
  
      }
  
      while ((tag = av_dict_get(s->metadata, "", tag, AV_DICT_IGNORE_SUFFIX))) {
 +        if(   !strcmp(tag->key, "width")
 +            ||!strcmp(tag->key, "height")
 +            ||!strcmp(tag->key, "videodatarate")
 +            ||!strcmp(tag->key, "framerate")
 +            ||!strcmp(tag->key, "videocodecid")
 +            ||!strcmp(tag->key, "audiodatarate")
 +            ||!strcmp(tag->key, "audiosamplerate")
 +            ||!strcmp(tag->key, "audiosamplesize")
 +            ||!strcmp(tag->key, "stereo")
 +            ||!strcmp(tag->key, "audiocodecid")
 +            ||!strcmp(tag->key, "duration")
 +            ||!strcmp(tag->key, "onMetaData")
 +        ){
 +            av_log(s, AV_LOG_DEBUG, "Ignoring metadata for %s\n", tag->key);
 +            continue;
 +        }
          put_amf_string(pb, tag->key);
          avio_w8(pb, AMF_DATA_TYPE_STRING);
          put_amf_string(pb, tag->value);
  
      for (i = 0; i < s->nb_streams; i++) {
          AVCodecContext *enc = s->streams[i]->codec;
 -        if (enc->codec_id == AV_CODEC_ID_AAC || enc->codec_id == AV_CODEC_ID_H264) {
 +        if (enc->codec_id == AV_CODEC_ID_AAC || enc->codec_id == AV_CODEC_ID_H264 || enc->codec_id == AV_CODEC_ID_MPEG4) {
              int64_t pos;
              avio_w8(pb, enc->codec_type == AVMEDIA_TYPE_VIDEO ?
                      FLV_TAG_TYPE_VIDEO : FLV_TAG_TYPE_AUDIO);
@@@ -429,7 -406,7 +429,7 @@@ static int flv_write_trailer(AVFormatCo
          AVCodecContext *enc = s->streams[i]->codec;
          FLVStreamContext *sc = s->streams[i]->priv_data;
          if (enc->codec_type == AVMEDIA_TYPE_VIDEO &&
 -            enc->codec_id == AV_CODEC_ID_H264)
 +                (enc->codec_id == AV_CODEC_ID_H264 || enc->codec_id == AV_CODEC_ID_MPEG4))
              put_avc_eos_tag(pb, sc->last_ts);
      }
  
@@@ -458,12 -435,12 +458,12 @@@ static int flv_write_packet(AVFormatCon
      unsigned ts;
      int size = pkt->size;
      uint8_t *data = NULL;
 -    int flags = 0, flags_size;
 +    int flags = -1, flags_size, ret;
  
      if (enc->codec_id == AV_CODEC_ID_VP6F || enc->codec_id == AV_CODEC_ID_VP6A ||
 -        enc->codec_id == AV_CODEC_ID_AAC)
 +        enc->codec_id == AV_CODEC_ID_VP6  || enc->codec_id == AV_CODEC_ID_AAC)
          flags_size = 2;
 -    else if (enc->codec_id == AV_CODEC_ID_H264)
 +    else if (enc->codec_id == AV_CODEC_ID_H264 || enc->codec_id == AV_CODEC_ID_MPEG4)
          flags_size = 5;
      else
          flags_size = 1;
          flags = enc->codec_tag;
          if (flags == 0) {
              av_log(s, AV_LOG_ERROR,
 -                   "video codec %X not compatible with flv\n",
 -                   enc->codec_id);
 -            return -1;
 +                   "Video codec '%s' is not compatible with FLV\n",
 +                   avcodec_get_name(enc->codec_id));
 +            return AVERROR(EINVAL);
          }
  
          flags |= pkt->flags & AV_PKT_FLAG_KEY ? FLV_FRAME_KEY : FLV_FRAME_INTER;
      case AVMEDIA_TYPE_AUDIO:
          flags = get_audio_flags(s, enc);
  
 -        assert(size);
 +        av_assert0(size);
  
          avio_w8(pb, FLV_TAG_TYPE_AUDIO);
          break;
          return AVERROR(EINVAL);
      }
  
 -    if (enc->codec_id == AV_CODEC_ID_H264)
 -        /* check if extradata looks like MP4 */
 +    if (enc->codec_id == AV_CODEC_ID_H264 || enc->codec_id == AV_CODEC_ID_MPEG4) {
 +        /* check if extradata looks like mp4 formated */
          if (enc->extradata_size > 0 && *(uint8_t*)enc->extradata != 1)
 -            if (ff_avc_parse_nal_units_buf(pkt->data, &data, &size) < 0)
 -                return -1;
 +            if ((ret = ff_avc_parse_nal_units_buf(pkt->data, &data, &size)) < 0)
 +                return ret;
 +    } else if (enc->codec_id == AV_CODEC_ID_AAC && pkt->size > 2 &&
 +               (AV_RB16(pkt->data) & 0xfff0) == 0xfff0) {
 +        if (!s->streams[pkt->stream_index]->nb_frames) {
 +        av_log(s, AV_LOG_ERROR, "Malformed AAC bitstream detected: "
 +               "use audio bitstream filter 'aac_adtstoasc' to fix it "
 +               "('-bsf:a aac_adtstoasc' option with ffmpeg)\n");
 +        return AVERROR_INVALIDDATA;
 +        }
 +        av_log(s, AV_LOG_WARNING, "aac bitstream error\n");
 +    }
  
      if (flv->delay == AV_NOPTS_VALUE)
          flv->delay = -pkt->dts;
  
      if (enc->codec_type == AVMEDIA_TYPE_DATA) {
          int data_size;
-         int metadata_size_pos = avio_tell(pb);
+         int64_t metadata_size_pos = avio_tell(pb);
          avio_w8(pb, AMF_DATA_TYPE_STRING);
          put_amf_string(pb, "onTextData");
          avio_w8(pb, AMF_DATA_TYPE_MIXEDARRAY);
          avio_seek(pb, data_size + 10 - 3, SEEK_CUR);
          avio_wb32(pb, data_size + 11);
      } else {
 +        av_assert1(flags>=0);
          avio_w8(pb,flags);
 +        if (enc->codec_id == AV_CODEC_ID_VP6)
 +            avio_w8(pb,0);
          if (enc->codec_id == AV_CODEC_ID_VP6F || enc->codec_id == AV_CODEC_ID_VP6A) {
              if (enc->extradata_size)
                  avio_w8(pb, enc->extradata[0]);
                               (FFALIGN(enc->height, 16) - enc->height));
          } else if (enc->codec_id == AV_CODEC_ID_AAC)
              avio_w8(pb, 1); // AAC raw
 -        else if (enc->codec_id == AV_CODEC_ID_H264) {
 +        else if (enc->codec_id == AV_CODEC_ID_H264 || enc->codec_id == AV_CODEC_ID_MPEG4) {
              avio_w8(pb, 1); // AVC NALU
              avio_wb24(pb, pkt->pts - pkt->dts);
          }