Merge remote-tracking branch 'qatar/master'
authorMichael Niedermayer <michaelni@gmx.at>
Sun, 28 Oct 2012 13:04:43 +0000 (14:04 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sun, 28 Oct 2012 13:24:27 +0000 (14:24 +0100)
* qatar/master:
  yuv4mpeg: reject unsupported codecs
  nutenc: K&R formatting cosmetics
  assdec: fix qsort() callback signature
  configure: detect sparc64 automatically
  vp8: fix memset() crossing array boundary
  h264: fix invalid pointer arithmetic
  amrwbdec: fix invalid pointer arithmetic

Conflicts:
libavformat/nutenc.c
libavformat/yuv4mpeg.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
configure
libavcodec/amrwbdec.c
libavcodec/vp8.c
libavformat/assdec.c
libavformat/nutenc.c
libavformat/yuv4mpeg.c

diff --cc configure
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -37,36 -38,43 +39,40 @@@ static int find_expected_header(AVCodec
  
      AV_WB24(out, 1);
  
-     if(c->codec_id == AV_CODEC_ID_MPEG4){
-         if(key_frame){
+     if (c->codec_id == AV_CODEC_ID_MPEG4) {
+         if (key_frame) {
              return 3;
-         }else{
-             out[3]= 0xB6;
+         } else {
+             out[3] = 0xB6;
              return 4;
          }
-     }else if(c->codec_id == AV_CODEC_ID_MPEG1VIDEO || c->codec_id == AV_CODEC_ID_MPEG2VIDEO){
+     } else if (c->codec_id == AV_CODEC_ID_MPEG1VIDEO ||
+                c->codec_id == AV_CODEC_ID_MPEG2VIDEO) {
          return 3;
-     }else if(c->codec_id == AV_CODEC_ID_H264){
+     } else if (c->codec_id == AV_CODEC_ID_H264) {
          return 3;
-     }else if(c->codec_id == AV_CODEC_ID_MP3 || c->codec_id == AV_CODEC_ID_MP2){
+     } else if (c->codec_id == AV_CODEC_ID_MP3 ||
+                c->codec_id == AV_CODEC_ID_MP2) {
          int lsf, mpeg25, sample_rate_index, bitrate_index, frame_size;
-         int layer= c->codec_id == AV_CODEC_ID_MP3 ? 3 : 2;
-         unsigned int header= 0xFFF00000;
+         int layer           = c->codec_id == AV_CODEC_ID_MP3 ? 3 : 2;
+         unsigned int header = 0xFFF00000;
  
-         lsf     = sample_rate < (24000+32000)/2;
-         mpeg25  = sample_rate < (12000+16000)/2;
+         lsf           = sample_rate < (24000 + 32000) / 2;
+         mpeg25        = sample_rate < (12000 + 16000) / 2;
          sample_rate <<= lsf + mpeg25;
-         if     (sample_rate < (32000 + 44100)/2) sample_rate_index=2;
-         else if(sample_rate < (44100 + 48000)/2) sample_rate_index=0;
-         else                                     sample_rate_index=1;
 -        if (sample_rate < (32000 + 44100) / 2)
 -            sample_rate_index = 2;
 -        else if (sample_rate < (44100 + 48000) / 2)
 -            sample_rate_index = 0;
 -        else
 -            sample_rate_index = 1;
++        if      (sample_rate < (32000 + 44100) / 2) sample_rate_index = 2;
++        else if (sample_rate < (44100 + 48000) / 2) sample_rate_index = 0;
++        else                                        sample_rate_index = 1;
  
-         sample_rate= avpriv_mpa_freq_tab[sample_rate_index] >> (lsf + mpeg25);
+         sample_rate = avpriv_mpa_freq_tab[sample_rate_index] >> (lsf + mpeg25);
  
-         for(bitrate_index=2; bitrate_index<30; bitrate_index++){
-             frame_size = avpriv_mpa_bitrate_tab[lsf][layer-1][bitrate_index>>1];
-             frame_size = (frame_size * 144000) / (sample_rate << lsf) + (bitrate_index&1);
+         for (bitrate_index = 2; bitrate_index < 30; bitrate_index++) {
+             frame_size =
+                 avpriv_mpa_bitrate_tab[lsf][layer - 1][bitrate_index >> 1];
+             frame_size = (frame_size * 144000) / (sample_rate << lsf) +
+                 (bitrate_index & 1);
  
-             if(frame_size == size)
+             if (frame_size == size)
                  break;
          }
  
@@@ -90,7 -98,9 +96,8 @@@
      return 0;
  }
  
- static int find_header_idx(AVFormatContext *s, AVCodecContext *c, int size, int frame_type){
 -static int find_header_idx(AVFormatContext *s, AVCodecContext *c, int size,
 -                           int frame_type)
++static int find_header_idx(AVFormatContext *s, AVCodecContext *c, int size, int frame_type)
+ {
      NUTContext *nut = s->priv_data;
      uint8_t out[64];
      int i;
@@@ -149,114 -161,97 +158,113 @@@ static void build_frame_code(AVFormatCo
          start++;
      }
  
-     for(stream_id= 0; stream_id<s->nb_streams; stream_id++){
-         int start2= start + (end-start)*stream_id / s->nb_streams;
-         int end2  = start + (end-start)*(stream_id+1) / s->nb_streams;
+     for (stream_id = 0; stream_id < s->nb_streams; stream_id++) {
+         int start2 = start + (end - start) * stream_id       / s->nb_streams;
+         int end2   = start + (end - start) * (stream_id + 1) / s->nb_streams;
          AVCodecContext *codec = s->streams[stream_id]->codec;
-         int is_audio= codec->codec_type == AVMEDIA_TYPE_AUDIO;
-         int intra_only= /*codec->intra_only || */is_audio;
+         int is_audio          = codec->codec_type == AVMEDIA_TYPE_AUDIO;
+         int intra_only        = /*codec->intra_only || */ is_audio;
          int pred_count;
-             if(f.den == 1 && f.num>0)
 +        int frame_size = 0;
 +
 +        if (codec->codec_type == AVMEDIA_TYPE_AUDIO) {
 +            frame_size = av_get_audio_frame_duration(codec, 0);
 +            if (codec->codec_id == AV_CODEC_ID_VORBIS && !frame_size)
 +                frame_size = 64;
 +        } else {
 +            AVRational f = av_div_q(codec->time_base, *nut->stream[stream_id].time_base);
-         if(!frame_size)
++            if (f.den == 1 && f.num>0)
 +                frame_size = f.num;
 +        }
++        if (!frame_size)
 +            frame_size = 1;
  
-         for(key_frame=0; key_frame<2; key_frame++){
-             if(intra_only && keyframe_0_esc && key_frame==0)
-                 continue;
-             {
-                 FrameCode *ft= &nut->frame_code[start2];
-                 ft->flags= FLAG_KEY*key_frame;
-                 ft->flags|= FLAG_SIZE_MSB | FLAG_CODED_PTS;
-                 ft->stream_id= stream_id;
-                 ft->size_mul=1;
-                 if(is_audio)
-                     ft->header_idx= find_header_idx(s, codec, -1, key_frame);
+         for (key_frame = 0; key_frame < 2; key_frame++) {
+             if (!intra_only || !keyframe_0_esc || key_frame != 0) {
+                 FrameCode *ft = &nut->frame_code[start2];
+                 ft->flags     = FLAG_KEY * key_frame;
+                 ft->flags    |= FLAG_SIZE_MSB | FLAG_CODED_PTS;
+                 ft->stream_id = stream_id;
+                 ft->size_mul  = 1;
+                 if (is_audio)
+                     ft->header_idx = find_header_idx(s, codec, -1, key_frame);
                  start2++;
              }
          }
  
-         key_frame= intra_only;
+         key_frame = intra_only;
 +#if 1
-         if(is_audio){
-             int frame_bytes= codec->frame_size*(int64_t)codec->bit_rate / (8*codec->sample_rate);
+         if (is_audio) {
+             int frame_bytes = codec->frame_size * (int64_t)codec->bit_rate /
+                               (8 * codec->sample_rate);
              int pts;
-             for(pts=0; pts<2; pts++){
-                 for(pred=0; pred<2; pred++){
-                     FrameCode *ft= &nut->frame_code[start2];
-                     ft->flags= FLAG_KEY*key_frame;
-                     ft->stream_id= stream_id;
-                     ft->size_mul=frame_bytes + 2;
-                     ft->size_lsb=frame_bytes + pred;
-                     ft->pts_delta=pts * frame_size;
-                     ft->header_idx= find_header_idx(s, codec, frame_bytes + pred, key_frame);
 -            for (pts = 0; pts < 2; pts++)
++            for (pts = 0; pts < 2; pts++) {
+                 for (pred = 0; pred < 2; pred++) {
 -                    FrameCode *ft = &nut->frame_code[start2];
++                    FrameCode *ft  = &nut->frame_code[start2];
+                     ft->flags      = FLAG_KEY * key_frame;
+                     ft->stream_id  = stream_id;
+                     ft->size_mul   = frame_bytes + 2;
+                     ft->size_lsb   = frame_bytes + pred;
 -                    ft->pts_delta  = pts;
++                    ft->pts_delta  = pts * frame_size;
+                     ft->header_idx = find_header_idx(s, codec, frame_bytes + pred, key_frame);
                      start2++;
                  }
-         }else{
-             FrameCode *ft= &nut->frame_code[start2];
-             ft->flags= FLAG_KEY | FLAG_SIZE_MSB;
-             ft->stream_id= stream_id;
-             ft->size_mul=1;
-             ft->pts_delta=frame_size;
 +            }
 -            ft->pts_delta = 1;
+         } else {
+             FrameCode *ft = &nut->frame_code[start2];
+             ft->flags     = FLAG_KEY | FLAG_SIZE_MSB;
+             ft->stream_id = stream_id;
+             ft->size_mul  = 1;
++            ft->pts_delta = frame_size;
              start2++;
          }
 +#endif
  
-         if(codec->has_b_frames){
-             pred_count=5;
-             pred_table[0]=-2;
-             pred_table[1]=-1;
-             pred_table[2]=1;
-             pred_table[3]=3;
-             pred_table[4]=4;
-         }else if(codec->codec_id == AV_CODEC_ID_VORBIS){
-             pred_count=3;
-             pred_table[0]=2;
-             pred_table[1]=9;
-             pred_table[2]=16;
-         }else{
-             pred_count=1;
-             pred_table[0]=1;
+         if (codec->has_b_frames) {
+             pred_count    = 5;
+             pred_table[0] = -2;
+             pred_table[1] = -1;
+             pred_table[2] = 1;
+             pred_table[3] = 3;
+             pred_table[4] = 4;
+         } else if (codec->codec_id == AV_CODEC_ID_VORBIS) {
+             pred_count    = 3;
+             pred_table[0] = 2;
+             pred_table[1] = 9;
+             pred_table[2] = 16;
+         } else {
+             pred_count    = 1;
+             pred_table[0] = 1;
          }
  
-         for(pred=0; pred<pred_count; pred++){
-             int start3= start2 + (end2-start2)*pred / pred_count;
-             int end3  = start2 + (end2-start2)*(pred+1) / pred_count;
+         for (pred = 0; pred < pred_count; pred++) {
+             int start3 = start2 + (end2 - start2) * pred / pred_count;
+             int end3   = start2 + (end2 - start2) * (pred + 1) / pred_count;
  
-             for(index=start3; index<end3; index++){
-                 FrameCode *ft= &nut->frame_code[index];
-                 ft->flags= FLAG_KEY*key_frame;
-                 ft->flags|= FLAG_SIZE_MSB;
-                 ft->stream_id= stream_id;
 +            pred_table[pred] *= frame_size;
 +
+             for (index = start3; index < end3; index++) {
+                 FrameCode *ft = &nut->frame_code[index];
+                 ft->flags     = FLAG_KEY * key_frame;
+                 ft->flags    |= FLAG_SIZE_MSB;
+                 ft->stream_id = stream_id;
  //FIXME use single byte size and pred from last
-                 ft->size_mul= end3-start3;
-                 ft->size_lsb= index - start3;
-                 ft->pts_delta= pred_table[pred];
-                 if(is_audio)
-                     ft->header_idx= find_header_idx(s, codec, -1, key_frame);
+                 ft->size_mul  = end3 - start3;
+                 ft->size_lsb  = index - start3;
+                 ft->pts_delta = pred_table[pred];
+                 if (is_audio)
+                     ft->header_idx = find_header_idx(s, codec, -1, key_frame);
              }
          }
      }
-     memmove(&nut->frame_code['N'+1], &nut->frame_code['N'], sizeof(FrameCode)*(255-'N'));
-     nut->frame_code[  0].flags=
-     nut->frame_code[255].flags=
-     nut->frame_code['N'].flags= FLAG_INVALID;
 -    memmove(&nut->frame_code['N' + 1], &nut->frame_code['N'],
 -            sizeof(FrameCode) * (255 - 'N'));
++    memmove(&nut->frame_code['N' + 1], &nut->frame_code['N'], sizeof(FrameCode) * (255 - 'N'));
+     nut->frame_code[0].flags       =
+         nut->frame_code[255].flags =
+         nut->frame_code['N'].flags = FLAG_INVALID;
  }
  
- static void put_tt(NUTContext *nut, AVRational *time_base, AVIOContext *bc, uint64_t val){
 -static void put_tt(NUTContext *nut, AVRational *time_base, AVIOContext *bc,
 -                   uint64_t val)
++static void put_tt(NUTContext *nut, AVRational *time_base, AVIOContext *bc, uint64_t val)
+ {
      val *= nut->time_base_count;
      val += time_base - nut->time_base;
      ff_put_v(bc, val);
@@@ -285,8 -281,8 +294,7 @@@ static inline void ff_put_v_trace(AVIOC
      ff_put_v(bc, v);
  }
  
--static inline void put_s_trace(AVIOContext *bc, int64_t v, const char *file,
--                               const char *func, int line)
++static inline void put_s_trace(AVIOContext *bc, int64_t v, const char *file, const char *func, int line)
  {
      av_log(NULL, AV_LOG_DEBUG, "put_s %5"PRId64" / %"PRIX64" in %s %s:%d\n", v, v, file, func, line);
  
@@@ -332,32 -332,37 +344,32 @@@ static void write_mainheader(NUTContex
          ff_put_v(bc, nut->time_base[i].den);
      }
  
-     tmp_pts=0;
-     tmp_mul=1;
-     tmp_stream=0;
-     tmp_match= 1-(1LL<<62);
-     tmp_head_idx= 0;
-     for(i=0; i<256;){
-         tmp_fields=0;
-         tmp_size=0;
+     tmp_pts      = 0;
+     tmp_mul      = 1;
+     tmp_stream   = 0;
+     tmp_match    = 1 - (1LL << 62);
+     tmp_head_idx = 0;
+     for (i = 0; i < 256; ) {
+         tmp_fields = 0;
+         tmp_size   = 0;
  //        tmp_res=0;
-         if(tmp_pts    != nut->frame_code[i].pts_delta) tmp_fields=1;
-         if(tmp_mul    != nut->frame_code[i].size_mul ) tmp_fields=2;
-         if(tmp_stream != nut->frame_code[i].stream_id) tmp_fields=3;
-         if(tmp_size   != nut->frame_code[i].size_lsb ) tmp_fields=4;
 -        if (tmp_pts != nut->frame_code[i].pts_delta)
 -            tmp_fields = 1;
 -        if (tmp_mul != nut->frame_code[i].size_mul)
 -            tmp_fields = 2;
 -        if (tmp_stream != nut->frame_code[i].stream_id)
 -            tmp_fields = 3;
 -        if (tmp_size != nut->frame_code[i].size_lsb)
 -            tmp_fields = 4;
--//        if(tmp_res    != nut->frame_code[i].res            ) tmp_fields=5;
-         if(tmp_head_idx!=nut->frame_code[i].header_idx)tmp_fields=8;
-         tmp_pts   = nut->frame_code[i].pts_delta;
-         tmp_flags = nut->frame_code[i].flags;
-         tmp_stream= nut->frame_code[i].stream_id;
-         tmp_mul   = nut->frame_code[i].size_mul;
-         tmp_size  = nut->frame_code[i].size_lsb;
 -        if (tmp_head_idx != nut->frame_code[i].header_idx)
 -            tmp_fields = 8;
++        if (tmp_pts      != nut->frame_code[i].pts_delta ) tmp_fields = 1;
++        if (tmp_mul      != nut->frame_code[i].size_mul  ) tmp_fields = 2;
++        if (tmp_stream   != nut->frame_code[i].stream_id ) tmp_fields = 3;
++        if (tmp_size     != nut->frame_code[i].size_lsb  ) tmp_fields = 4;
++//        if (tmp_res    != nut->frame_code[i].res            ) tmp_fields=5;
++        if (tmp_head_idx != nut->frame_code[i].header_idx) tmp_fields = 8;
+         tmp_pts    = nut->frame_code[i].pts_delta;
+         tmp_flags  = nut->frame_code[i].flags;
+         tmp_stream = nut->frame_code[i].stream_id;
+         tmp_mul    = nut->frame_code[i].size_mul;
+         tmp_size   = nut->frame_code[i].size_lsb;
  //        tmp_res   = nut->frame_code[i].res;
-         tmp_head_idx= nut->frame_code[i].header_idx;
+         tmp_head_idx = nut->frame_code[i].header_idx;
  
-         for(j=0; i<256; j++,i++){
-             if(i == 'N'){
+         for (j = 0; i < 256; j++, i++) {
+             if (i == 'N') {
                  j--;
                  continue;
              }
  
          ff_put_v(bc, tmp_flags);
          ff_put_v(bc, tmp_fields);
-         if(tmp_fields>0) put_s(bc, tmp_pts);
-         if(tmp_fields>1) ff_put_v(bc, tmp_mul);
-         if(tmp_fields>2) ff_put_v(bc, tmp_stream);
-         if(tmp_fields>3) ff_put_v(bc, tmp_size);
-         if(tmp_fields>4) ff_put_v(bc, 0 /*tmp_res*/);
-         if(tmp_fields>5) ff_put_v(bc, j);
-         if(tmp_fields>6) ff_put_v(bc, tmp_match);
-         if(tmp_fields>7) ff_put_v(bc, tmp_head_idx);
 -        if (tmp_fields > 0)
 -            put_s(bc, tmp_pts);
 -        if (tmp_fields > 1)
 -            ff_put_v(bc, tmp_mul);
 -        if (tmp_fields > 2)
 -            ff_put_v(bc, tmp_stream);
 -        if (tmp_fields > 3)
 -            ff_put_v(bc, tmp_size);
 -        if (tmp_fields > 4)
 -            ff_put_v(bc, 0 /*tmp_res*/);
 -        if (tmp_fields > 5)
 -            ff_put_v(bc, j);
 -        if (tmp_fields > 6)
 -            ff_put_v(bc, tmp_match);
 -        if (tmp_fields > 7)
 -            ff_put_v(bc, tmp_head_idx);
++        if (tmp_fields > 0) put_s(bc, tmp_pts);
++        if (tmp_fields > 1) ff_put_v(bc, tmp_mul);
++        if (tmp_fields > 2) ff_put_v(bc, tmp_stream);
++        if (tmp_fields > 3) ff_put_v(bc, tmp_size);
++        if (tmp_fields > 4) ff_put_v(bc, 0 /*tmp_res*/);
++        if (tmp_fields > 5) ff_put_v(bc, j);
++        if (tmp_fields > 6) ff_put_v(bc, tmp_match);
++        if (tmp_fields > 7) ff_put_v(bc, tmp_head_idx);
      }
-     ff_put_v(bc, nut->header_count-1);
-     for(i=1; i<nut->header_count; i++){
+     ff_put_v(bc, nut->header_count - 1);
+     for (i = 1; i < nut->header_count; i++) {
          ff_put_v(bc, nut->header_len[i]);
          avio_write(bc, nut->header[i], nut->header_len[i]);
      }
  }
  
- static int write_streamheader(AVFormatContext *avctx, AVIOContext *bc, AVStream *st, int i){
-     NUTContext *nut = avctx->priv_data;
+ static int write_streamheader(AVFormatContext *avctx, AVIOContext *bc,
+                               AVStream *st, int i)
+ {
+     NUTContext *nut       = avctx->priv_data;
      AVCodecContext *codec = st->codec;
 -    unsigned codec_tag    = av_codec_get_tag(ff_nut_codec_tags, codec->codec_id);
      ff_put_v(bc, i);
-     switch(codec->codec_type){
-     case AVMEDIA_TYPE_VIDEO: ff_put_v(bc, 0); break;
-     case AVMEDIA_TYPE_AUDIO: ff_put_v(bc, 1); break;
+     switch (codec->codec_type) {
 -    case AVMEDIA_TYPE_VIDEO:
 -        ff_put_v(bc, 0);
 -        break;
 -    case AVMEDIA_TYPE_AUDIO:
 -        ff_put_v(bc, 1);
 -        break;
 -    case AVMEDIA_TYPE_SUBTITLE:
 -        ff_put_v(bc, 2);
 -        break;
 -    default:
 -        ff_put_v(bc, 3);
 -        break;
++    case AVMEDIA_TYPE_VIDEO:    ff_put_v(bc, 0); break;
++    case AVMEDIA_TYPE_AUDIO:    ff_put_v(bc, 1); break;
 +    case AVMEDIA_TYPE_SUBTITLE: ff_put_v(bc, 2); break;
-     default              : ff_put_v(bc, 3); break;
++    default:                    ff_put_v(bc, 3); break;
      }
      ff_put_v(bc, 4);
-     if (codec->codec_tag){
 -
 -    if (!codec_tag || codec->codec_id == AV_CODEC_ID_RAWVIDEO)
 -        codec_tag = codec->codec_tag;
 -
 -    if (codec_tag) {
 -        avio_wl32(bc, codec_tag);
++    if (codec->codec_tag) {
 +        avio_wl32(bc, codec->codec_tag);
      } else {
          av_log(avctx, AV_LOG_ERROR, "No codec tag defined for stream %d\n", i);
          return AVERROR(EINVAL);
@@@ -474,19 -508,16 +494,19 @@@ static int write_globalinfo(NUTContext 
      return 0;
  }
  
--static int write_streaminfo(NUTContext *nut, AVIOContext *bc, int stream_id){
++static int write_streaminfo(NUTContext *nut, AVIOContext *bc, int stream_id) {
      AVFormatContext *s= nut->avf;
      AVStream* st = s->streams[stream_id];
 +    AVDictionaryEntry *t = NULL;
      AVIOContext *dyn_bc;
      uint8_t *dyn_buf=NULL;
      int count=0, dyn_size, i;
      int ret = avio_open_dyn_buf(&dyn_bc);
--    if(ret < 0)
++    if (ret < 0)
          return ret;
  
 +    while ((t = av_dict_get(st->metadata, "", t, AV_DICT_IGNORE_SUFFIX)))
 +        count += add_info(dyn_bc, t->key, t->value);
      for (i=0; ff_nut_dispositions[i].flag; ++i) {
          if (st->disposition & ff_nut_dispositions[i].flag)
              count += add_info(dyn_bc, "Disposition", ff_nut_dispositions[i].str);
@@@ -541,52 -567,8 +561,53 @@@ static int write_chapter(NUTContext *nu
      return 0;
  }
  
- static int write_index(NUTContext *nut, AVIOContext *bc){
++static int write_index(NUTContext *nut, AVIOContext *bc) {
 +    int i;
 +    Syncpoint dummy= { .pos= 0 };
 +    Syncpoint *next_node[2] = { NULL };
 +    int64_t startpos = avio_tell(bc);
 +    int64_t payload_size;
 +
 +    put_tt(nut, nut->max_pts_tb, bc, nut->max_pts);
 +
 +    ff_put_v(bc, nut->sp_count);
 +
-     for(i=0; i<nut->sp_count; i++){
++    for (i=0; i<nut->sp_count; i++) {
 +        av_tree_find(nut->syncpoints, &dummy, (void *) ff_nut_sp_pos_cmp, (void**)next_node);
 +        ff_put_v(bc, (next_node[1]->pos >> 4) - (dummy.pos>>4));
 +        dummy.pos = next_node[1]->pos;
 +    }
 +
-     for(i=0; i<nut->avf->nb_streams; i++){
++    for (i=0; i<nut->avf->nb_streams; i++) {
 +        StreamContext *nus= &nut->stream[i];
 +        int64_t last_pts= -1;
 +        int j, k;
-         for(j=0; j<nut->sp_count; j++){
++        for (j=0; j<nut->sp_count; j++) {
 +            int flag = (nus->keyframe_pts[j] != AV_NOPTS_VALUE) ^ (j+1 == nut->sp_count);
 +            int n = 0;
-             for(; j<nut->sp_count && (nus->keyframe_pts[j] != AV_NOPTS_VALUE) == flag; j++)
++            for (; j<nut->sp_count && (nus->keyframe_pts[j] != AV_NOPTS_VALUE) == flag; j++)
 +                n++;
 +
 +            ff_put_v(bc, 1 + 2*flag + 4*n);
-             for(k= j - n; k<=j && k<nut->sp_count; k++) {
-                 if(nus->keyframe_pts[k] == AV_NOPTS_VALUE)
++            for (k= j - n; k<=j && k<nut->sp_count; k++) {
++                if (nus->keyframe_pts[k] == AV_NOPTS_VALUE)
 +                    continue;
 +                av_assert0(nus->keyframe_pts[k] > last_pts);
 +                ff_put_v(bc, nus->keyframe_pts[k] - last_pts);
 +                last_pts = nus->keyframe_pts[k];
 +            }
 +        }
 +    }
 +
 +    payload_size = avio_tell(bc) - startpos + 8 + 4;
 +
 +    avio_wb64(bc, 8 + payload_size + av_log2(payload_size) / 7 + 1 + 4*(payload_size > 4096));
 +
 +    return 0;
 +}
 +
- static int write_headers(AVFormatContext *avctx, AVIOContext *bc){
+ static int write_headers(AVFormatContext *avctx, AVIOContext *bc)
+ {
      NUTContext *nut = avctx->priv_data;
      AVIOContext *dyn_bc;
      int i, ret;
@@@ -654,30 -638,26 +677,30 @@@ static int nut_write_header(AVFormatCon
      AVIOContext *bc = s->pb;
      int i, j, ret;
  
-     nut->avf= s;
+     nut->avf = s;
  
-     nut->stream   = av_mallocz(sizeof(StreamContext)*s->nb_streams);
-     nut->chapter  = av_mallocz(sizeof(ChapterContext)*s->nb_chapters);
-     nut->time_base= av_mallocz(sizeof(AVRational   )*(s->nb_streams +
 -    nut->stream = av_mallocz(sizeof(StreamContext) * s->nb_streams);
 -    if (s->nb_chapters)
 -        nut->chapter = av_mallocz(sizeof(ChapterContext) * s->nb_chapters);
 -    nut->time_base = av_mallocz(sizeof(AVRational) * (s->nb_streams +
--                                                      s->nb_chapters));
 -    if (!nut->stream || (s->nb_chapters && !nut->chapter) || !nut->time_base) {
++    nut->stream   = av_mallocz(sizeof(StreamContext ) * s->nb_streams);
++    nut->chapter  = av_mallocz(sizeof(ChapterContext) * s->nb_chapters);
++    nut->time_base= av_mallocz(sizeof(AVRational    ) *(s->nb_streams +
++                                                        s->nb_chapters));
 +    if (!nut->stream || !nut->chapter || !nut->time_base) {
          av_freep(&nut->stream);
          av_freep(&nut->chapter);
          av_freep(&nut->time_base);
          return AVERROR(ENOMEM);
      }
  
-     for(i=0; i<s->nb_streams; i++){
-         AVStream *st= s->streams[i];
+     for (i = 0; i < s->nb_streams; i++) {
+         AVStream *st = s->streams[i];
          int ssize;
          AVRational time_base;
 -        ff_parse_specific_params(st->codec, &time_base.den, &ssize,
 -                                 &time_base.num);
 +        ff_parse_specific_params(st->codec, &time_base.den, &ssize, &time_base.num);
 +
-         if(st->codec->codec_type == AVMEDIA_TYPE_AUDIO && st->codec->sample_rate) {
-             time_base = (AVRational){1, st->codec->sample_rate};
++        if (st->codec->codec_type == AVMEDIA_TYPE_AUDIO && st->codec->sample_rate) {
++            time_base = (AVRational) {1, st->codec->sample_rate};
 +        } else {
 +            time_base = ff_choose_timebase(s, st, 48000);
 +        }
  
          avpriv_set_pts_info(st, 64, time_base.num, time_base.den);
  
@@@ -767,31 -754,31 +798,32 @@@ static int find_best_header_idx(NUTCont
      return best_i;
  }
  
- static int nut_write_packet(AVFormatContext *s, AVPacket *pkt){
-     NUTContext *nut = s->priv_data;
-     StreamContext *nus= &nut->stream[pkt->stream_index];
-     AVIOContext *bc = s->pb, *dyn_bc;
+ static int nut_write_packet(AVFormatContext *s, AVPacket *pkt)
+ {
+     NUTContext *nut    = s->priv_data;
+     StreamContext *nus = &nut->stream[pkt->stream_index];
+     AVIOContext *bc    = s->pb, *dyn_bc;
      FrameCode *fc;
      int64_t coded_pts;
-     int best_length, frame_code, flags, needed_flags, i, header_idx, best_header_idx;
 -    int best_length, frame_code, flags, needed_flags, i, header_idx,
 -        best_header_idx;
++    int best_length, frame_code, flags, needed_flags, i, header_idx;
++    int best_header_idx;
      int key_frame = !!(pkt->flags & AV_PKT_FLAG_KEY);
-     int store_sp=0;
+     int store_sp  = 0;
      int ret;
  
 -    if (pkt->pts < 0)
 -        return -1;
 +    if (pkt->pts < 0) {
 +        av_log(s, AV_LOG_ERROR, "Invalid negative packet pts %"PRId64" in input\n", pkt->pts);
 +        return AVERROR(EINVAL);
 +    }
  
-     if(1LL<<(20+3*nut->header_count) <= avio_tell(bc))
+     if (1LL << (20 + 3 * nut->header_count) <= avio_tell(bc))
          write_headers(s, bc);
  
-     if(key_frame && !(nus->last_flags & FLAG_KEY))
-         store_sp= 1;
+     if (key_frame && !(nus->last_flags & FLAG_KEY))
+         store_sp = 1;
  
-     if(pkt->size + 30/*FIXME check*/ + avio_tell(bc) >= nut->last_syncpoint_pos + nut->max_distance)
-         store_sp= 1;
 -    if (pkt->size + 30 /*FIXME check*/ + avio_tell(bc) >=
 -        nut->last_syncpoint_pos + nut->max_distance)
++    if (pkt->size + 30 /*FIXME check*/ + avio_tell(bc) >= nut->last_syncpoint_pos + nut->max_distance)
+         store_sp = 1;
  
  //FIXME: Ensure store_sp is 1 in the first place.
  
                  nus->time_base->num * (int64_t)nut->stream[i].time_base->den,
                  nus->time_base->den * (int64_t)nut->stream[i].time_base->num,
                  AV_ROUND_DOWN);
-             int index= av_index_search_timestamp(st, dts_tb, AVSEEK_FLAG_BACKWARD);
-             if(index>=0) dummy.pos= FFMIN(dummy.pos, st->index_entries[index].pos);
+             int index = av_index_search_timestamp(st, dts_tb,
+                                                   AVSEEK_FLAG_BACKWARD);
+             if (index >= 0)
+                 dummy.pos = FFMIN(dummy.pos, st->index_entries[index].pos);
          }
-         if(dummy.pos == INT64_MAX)
-             dummy.pos= 0;
-         sp= av_tree_find(nut->syncpoints, &dummy, (void *) ff_nut_sp_pos_cmp,
-                          NULL);
+         if (dummy.pos == INT64_MAX)
+             dummy.pos = 0;
+         sp = av_tree_find(nut->syncpoints, &dummy, (void *)ff_nut_sp_pos_cmp,
+                           NULL);
  
-         nut->last_syncpoint_pos= avio_tell(bc);
-         ret = avio_open_dyn_buf(&dyn_bc);
-         if(ret < 0)
+         nut->last_syncpoint_pos = avio_tell(bc);
+         ret                     = avio_open_dyn_buf(&dyn_bc);
+         if (ret < 0)
              return ret;
          put_tt(nut, nus->time_base, dyn_bc, pkt->dts);
-         ff_put_v(dyn_bc, sp ? (nut->last_syncpoint_pos - sp->pos)>>4 : 0);
+         ff_put_v(dyn_bc, sp ? (nut->last_syncpoint_pos - sp->pos) >> 4 : 0);
          put_packet(nut, bc, dyn_bc, 1, SYNCPOINT_STARTCODE);
  
-         ff_nut_add_sp(nut, nut->last_syncpoint_pos, 0/*unused*/, pkt->dts);
+         ff_nut_add_sp(nut, nut->last_syncpoint_pos, 0 /*unused*/, pkt->dts);
 +
-         if((1ll<<60) % nut->sp_count == 0)
-             for(i=0; i<s->nb_streams; i++){
++        if ((1ll<<60) % nut->sp_count == 0)
++            for (i=0; i<s->nb_streams; i++) {
 +                int j;
 +                StreamContext *nus = &nut->stream[i];
 +                nus->keyframe_pts = av_realloc(nus->keyframe_pts, 2*nut->sp_count*sizeof(*nus->keyframe_pts));
-                 if(!nus->keyframe_pts)
++                if (!nus->keyframe_pts)
 +                    return AVERROR(ENOMEM);
-                 for(j=nut->sp_count == 1 ? 0 : nut->sp_count; j<2*nut->sp_count; j++)
++                for (j=nut->sp_count == 1 ? 0 : nut->sp_count; j<2*nut->sp_count; j++)
 +                    nus->keyframe_pts[j] = AV_NOPTS_VALUE;
 +        }
      }
 -    assert(nus->last_pts != AV_NOPTS_VALUE);
 +    av_assert0(nus->last_pts != AV_NOPTS_VALUE);
  
-     coded_pts = pkt->pts & ((1<<nus->msb_pts_shift)-1);
-     if(ff_lsb2full(nus, coded_pts) != pkt->pts)
-         coded_pts= pkt->pts + (1<<nus->msb_pts_shift);
+     coded_pts = pkt->pts & ((1 << nus->msb_pts_shift) - 1);
+     if (ff_lsb2full(nus, coded_pts) != pkt->pts)
+         coded_pts = pkt->pts + (1 << nus->msb_pts_shift);
  
-     best_header_idx= find_best_header_idx(nut, pkt);
+     best_header_idx = find_best_header_idx(nut, pkt);
  
-     best_length=INT_MAX;
-     frame_code= -1;
-     for(i=0; i<256; i++){
-         int length= 0;
-         FrameCode *fc= &nut->frame_code[i];
-         int flags= fc->flags;
+     best_length = INT_MAX;
+     frame_code  = -1;
+     for (i = 0; i < 256; i++) {
+         int length    = 0;
+         FrameCode *fc = &nut->frame_code[i];
+         int flags     = fc->flags;
  
-         if(flags & FLAG_INVALID)
+         if (flags & FLAG_INVALID)
              continue;
-         needed_flags= get_needed_flags(nut, nus, fc, pkt);
+         needed_flags = get_needed_flags(nut, nus, fc, pkt);
  
-         if(flags & FLAG_CODED){
+         if (flags & FLAG_CODED) {
              length++;
              flags = needed_flags;
          }
  
-         if((flags & needed_flags) != needed_flags)
+         if ((flags & needed_flags) != needed_flags)
              continue;
  
-         if((flags ^ needed_flags) & FLAG_KEY)
+         if ((flags ^ needed_flags) & FLAG_KEY)
              continue;
  
-         if(flags & FLAG_STREAM_ID)
-             length+= ff_get_v_length(pkt->stream_index);
+         if (flags & FLAG_STREAM_ID)
+             length += ff_get_v_length(pkt->stream_index);
  
-         if(pkt->size % fc->size_mul != fc->size_lsb)
+         if (pkt->size % fc->size_mul != fc->size_lsb)
              continue;
-         if(flags & FLAG_SIZE_MSB)
+         if (flags & FLAG_SIZE_MSB)
              length += ff_get_v_length(pkt->size / fc->size_mul);
  
-         if(flags & FLAG_CHECKSUM)
-             length+=4;
+         if (flags & FLAG_CHECKSUM)
+             length += 4;
  
-         if(flags & FLAG_CODED_PTS)
+         if (flags & FLAG_CODED_PTS)
              length += ff_get_v_length(coded_pts);
  
-         if(   (flags & FLAG_CODED)
-            && nut->header_len[best_header_idx] > nut->header_len[fc->header_idx]+1){
 -        if ((flags & FLAG_CODED)
 -            && nut->header_len[best_header_idx] >
 -            nut->header_len[fc->header_idx] + 1) {
++        if (   (flags & FLAG_CODED)
++            && nut->header_len[best_header_idx] > nut->header_len[fc->header_idx] + 1) {
              flags |= FLAG_HEADER_IDX;
          }
  
              length -= nut->header_len[fc->header_idx];
          }
  
-         length*=4;
-         length+= !(flags & FLAG_CODED_PTS);
-         length+= !(flags & FLAG_CHECKSUM);
+         length *= 4;
+         length += !(flags & FLAG_CODED_PTS);
+         length += !(flags & FLAG_CHECKSUM);
  
-         if(length < best_length){
-             best_length= length;
-             frame_code=i;
+         if (length < best_length) {
+             best_length = length;
+             frame_code  = i;
          }
      }
 -    assert(frame_code != -1);
 +    av_assert0(frame_code != -1);
-     fc= &nut->frame_code[frame_code];
-     flags= fc->flags;
-     needed_flags= get_needed_flags(nut, nus, fc, pkt);
-     header_idx= fc->header_idx;
+     fc           = &nut->frame_code[frame_code];
+     flags        = fc->flags;
+     needed_flags = get_needed_flags(nut, nus, fc, pkt);
+     header_idx   = fc->header_idx;
  
      ffio_init_checksum(bc, ff_crc04C11DB7_update, 0);
      avio_w8(bc, frame_code);
-     if(flags & FLAG_CODED){
-         ff_put_v(bc, (flags^needed_flags) & ~(FLAG_CODED));
+     if (flags & FLAG_CODED) {
+         ff_put_v(bc, (flags ^ needed_flags) & ~(FLAG_CODED));
          flags = needed_flags;
      }
-     if(flags & FLAG_STREAM_ID)  ff_put_v(bc, pkt->stream_index);
-     if(flags & FLAG_CODED_PTS)  ff_put_v(bc, coded_pts);
-     if(flags & FLAG_SIZE_MSB)   ff_put_v(bc, pkt->size / fc->size_mul);
-     if(flags & FLAG_HEADER_IDX) ff_put_v(bc, header_idx= best_header_idx);
 -    if (flags & FLAG_STREAM_ID)
 -        ff_put_v(bc, pkt->stream_index);
 -    if (flags & FLAG_CODED_PTS)
 -        ff_put_v(bc, coded_pts);
 -    if (flags & FLAG_SIZE_MSB)
 -        ff_put_v(bc, pkt->size / fc->size_mul);
 -    if (flags & FLAG_HEADER_IDX)
 -        ff_put_v(bc, header_idx = best_header_idx);
 -
 -    if (flags & FLAG_CHECKSUM)
 -        avio_wl32(bc, ffio_get_checksum(bc));
 -    else
 -        ffio_get_checksum(bc);
++    if (flags & FLAG_STREAM_ID)  ff_put_v(bc, pkt->stream_index);
++    if (flags & FLAG_CODED_PTS)  ff_put_v(bc, coded_pts);
++    if (flags & FLAG_SIZE_MSB )  ff_put_v(bc, pkt->size / fc->size_mul);
++    if (flags & FLAG_HEADER_IDX) ff_put_v(bc, header_idx = best_header_idx);
  
-     if(flags & FLAG_CHECKSUM)   avio_wl32(bc, ffio_get_checksum(bc));
-     else                        ffio_get_checksum(bc);
 -    avio_write(bc, pkt->data + nut->header_len[header_idx],
 -               pkt->size - nut->header_len[header_idx]);
++    if (flags & FLAG_CHECKSUM)   avio_wl32(bc, ffio_get_checksum(bc));
++    else                         ffio_get_checksum(bc);
 +
 +    avio_write(bc, pkt->data + nut->header_len[header_idx], pkt->size - nut->header_len[header_idx]);
-     nus->last_flags= flags;
-     nus->last_pts= pkt->pts;
+     nus->last_flags = flags;
+     nus->last_pts   = pkt->pts;
  
      //FIXME just store one per syncpoint
-     if(flags & FLAG_KEY) {
 -    if (flags & FLAG_KEY)
++    if (flags & FLAG_KEY) {
          av_add_index_entry(
              s->streams[pkt->stream_index],
              nut->last_syncpoint_pos,
              0,
              0,
              AVINDEX_KEYFRAME);
-         if(nus->keyframe_pts && nus->keyframe_pts[nut->sp_count] == AV_NOPTS_VALUE)
++        if (nus->keyframe_pts && nus->keyframe_pts[nut->sp_count] == AV_NOPTS_VALUE)
 +            nus->keyframe_pts[nut->sp_count] = pkt->pts;
 +    }
 +
-     if(!nut->max_pts_tb || av_compare_ts(nut->max_pts, *nut->max_pts_tb, pkt->pts, *nus->time_base) < 0) {
++    if (!nut->max_pts_tb || av_compare_ts(nut->max_pts, *nut->max_pts_tb, pkt->pts, *nus->time_base) < 0) {
 +        nut->max_pts = pkt->pts;
 +        nut->max_pts_tb = nus->time_base;
 +    }
  
      return 0;
  }
  
- static int nut_write_trailer(AVFormatContext *s){
-     NUTContext *nut= s->priv_data;
+ static int nut_write_trailer(AVFormatContext *s)
+ {
+     NUTContext *nut = s->priv_data;
 -    AVIOContext *bc = s->pb;
 +    AVIOContext *bc = s->pb, *dyn_bc;
 +    int i, ret;
  
-     while(nut->header_count<3)
+     while (nut->header_count < 3)
          write_headers(s, bc);
  
-     if(ret >= 0) {
 +    ret = avio_open_dyn_buf(&dyn_bc);
++    if (ret >= 0) {
 +        write_index(nut, dyn_bc);
 +        put_packet(nut, bc, dyn_bc, 1, INDEX_STARTCODE);
 +    }
 +
      ff_nut_free_sp(nut);
-     for(i=0; i<s->nb_streams; i++)
++    for (i=0; i<s->nb_streams; i++)
 +        av_freep(&nut->stream[i].keyframe_pts);
 +
      av_freep(&nut->stream);
      av_freep(&nut->chapter);
      av_freep(&nut->time_base);
@@@ -216,47 -156,19 +216,46 @@@ static int yuv4_write_header(AVFormatCo
          return AVERROR(EIO);
  
      if (s->streams[0]->codec->codec_id != AV_CODEC_ID_RAWVIDEO) {
-         av_log(s, AV_LOG_ERROR,
-                "A non-rawvideo stream was selected, but yuv4mpeg only handles rawvideo streams\n");
-         return AVERROR(EINVAL);
+         av_log(s, AV_LOG_ERROR, "ERROR: Only rawvideo supported.\n");
+         return AVERROR_INVALIDDATA;
      }
  
 -    if (s->streams[0]->codec->pix_fmt == AV_PIX_FMT_YUV411P) {
 -        av_log(s, AV_LOG_ERROR, "Warning: generating rarely used 4:1:1 YUV "
 +    switch (s->streams[0]->codec->pix_fmt) {
 +    case AV_PIX_FMT_YUV411P:
 +        av_log(s, AV_LOG_WARNING, "Warning: generating rarely used 4:1:1 YUV "
                 "stream, some mjpegtools might not work.\n");
 -    } else if ((s->streams[0]->codec->pix_fmt != AV_PIX_FMT_YUV420P) &&
 -               (s->streams[0]->codec->pix_fmt != AV_PIX_FMT_YUV422P) &&
 -               (s->streams[0]->codec->pix_fmt != AV_PIX_FMT_GRAY8)   &&
 -               (s->streams[0]->codec->pix_fmt != AV_PIX_FMT_YUV444P)) {
 -        av_log(s, AV_LOG_ERROR, "ERROR: yuv4mpeg only handles yuv444p, "
 -               "yuv422p, yuv420p, yuv411p and gray pixel formats. "
 +        break;
 +    case AV_PIX_FMT_GRAY8:
 +    case AV_PIX_FMT_GRAY16:
 +    case AV_PIX_FMT_YUV420P:
 +    case AV_PIX_FMT_YUV422P:
 +    case AV_PIX_FMT_YUV444P:
 +        break;
 +    case AV_PIX_FMT_YUV420P9:
 +    case AV_PIX_FMT_YUV422P9:
 +    case AV_PIX_FMT_YUV444P9:
 +    case AV_PIX_FMT_YUV420P10:
 +    case AV_PIX_FMT_YUV422P10:
 +    case AV_PIX_FMT_YUV444P10:
 +    case AV_PIX_FMT_YUV420P16:
 +    case AV_PIX_FMT_YUV422P16:
 +    case AV_PIX_FMT_YUV444P16:
 +        if (s->streams[0]->codec->strict_std_compliance >= FF_COMPLIANCE_NORMAL) {
 +            av_log(s, AV_LOG_ERROR, "'%s' is not a official yuv4mpegpipe pixel format. "
 +                   "Use '-strict -1' to encode to this pixel format.\n",
 +                   av_get_pix_fmt_name(s->streams[0]->codec->pix_fmt));
 +            return AVERROR(EINVAL);
 +        }
 +        av_log(s, AV_LOG_WARNING, "Warning: generating non standart YUV stream. "
 +               "Mjpegtools will not work.\n");
 +        break;
 +    default:
 +        av_log(s, AV_LOG_ERROR, "ERROR: yuv4mpeg can only handle "
 +               "yuv444p, yuv422p, yuv420p, yuv411p and gray8 pixel formats. "
 +               "And using 'strict -1' also yuv444p9, yuv422p9, yuv420p9, "
 +               "yuv444p10, yuv422p10, yuv420p10, "
 +               "yuv444p16, yuv422p16, yuv420p16 "
 +               "and gray16 pixel formats. "
                 "Use -pix_fmt to select one.\n");
          return AVERROR(EIO);
      }