Merge commit 'd1d7678040cd60148f97b372cb4291bcc45b2e22'
authorTimothy Gu <timothygu99@gmail.com>
Wed, 3 Aug 2016 22:59:51 +0000 (15:59 -0700)
committerTimothy Gu <timothygu99@gmail.com>
Wed, 3 Aug 2016 22:59:51 +0000 (15:59 -0700)
* commit 'd1d7678040cd60148f97b372cb4291bcc45b2e22':
  h264: fix the check for mixed IDR/non-IDR slices

Conflicts:
libavcodec/h264_slice.c
libavcodec/h264dec.c

Merged-by: Timothy Gu <timothygu99@gmail.com>
1  2 
libavcodec/h264_slice.c
libavcodec/h264dec.c
libavcodec/h264dec.h

diff --combined libavcodec/h264_slice.c
@@@ -2,20 -2,20 +2,20 @@@
   * H.26L/H.264/AVC/JVT/14496-10/... decoder
   * Copyright (c) 2003 Michael Niedermayer <michaelni@gmx.at>
   *
 - * 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 "rectangle.h"
  #include "thread.h"
  
 -static const uint8_t field_scan[16] = {
 +static const uint8_t field_scan[16+1] = {
      0 + 0 * 4, 0 + 1 * 4, 1 + 0 * 4, 0 + 2 * 4,
      0 + 3 * 4, 1 + 1 * 4, 1 + 2 * 4, 1 + 3 * 4,
      2 + 0 * 4, 2 + 1 * 4, 2 + 2 * 4, 2 + 3 * 4,
      3 + 0 * 4, 3 + 1 * 4, 3 + 2 * 4, 3 + 3 * 4,
  };
  
 -static const uint8_t field_scan8x8[64] = {
 +static const uint8_t field_scan8x8[64+1] = {
      0 + 0 * 8, 0 + 1 * 8, 0 + 2 * 8, 1 + 0 * 8,
      1 + 1 * 8, 0 + 3 * 8, 0 + 4 * 8, 1 + 2 * 8,
      2 + 0 * 8, 1 + 3 * 8, 0 + 5 * 8, 0 + 6 * 8,
@@@ -73,7 -73,7 +73,7 @@@
      7 + 4 * 8, 7 + 5 * 8, 7 + 6 * 8, 7 + 7 * 8,
  };
  
 -static const uint8_t field_scan8x8_cavlc[64] = {
 +static const uint8_t field_scan8x8_cavlc[64+1] = {
      0 + 0 * 8, 1 + 1 * 8, 2 + 0 * 8, 0 + 7 * 8,
      2 + 2 * 8, 2 + 3 * 8, 2 + 4 * 8, 3 + 3 * 8,
      3 + 4 * 8, 4 + 3 * 8, 4 + 4 * 8, 5 + 3 * 8,
@@@ -93,7 -93,7 +93,7 @@@
  };
  
  // zigzag_scan8x8_cavlc[i] = zigzag_scan8x8[(i/4) + 16*(i%4)]
 -static const uint8_t zigzag_scan8x8_cavlc[64] = {
 +static const uint8_t zigzag_scan8x8_cavlc[64+1] = {
      0 + 0 * 8, 1 + 1 * 8, 1 + 2 * 8, 2 + 2 * 8,
      4 + 1 * 8, 0 + 5 * 8, 3 + 3 * 8, 7 + 0 * 8,
      3 + 4 * 8, 1 + 7 * 8, 5 + 3 * 8, 6 + 3 * 8,
@@@ -135,9 -135,9 +135,9 @@@ static int alloc_scratch_buffers(H264Sl
      // (= 21x21 for  H.264)
      av_fast_malloc(&sl->edge_emu_buffer, &sl->edge_emu_buffer_allocated, alloc_size * 2 * 21);
  
 -    av_fast_malloc(&sl->top_borders[0], &sl->top_borders_allocated[0],
 +    av_fast_mallocz(&sl->top_borders[0], &sl->top_borders_allocated[0],
                     h->mb_width * 16 * 3 * sizeof(uint8_t) * 2);
 -    av_fast_malloc(&sl->top_borders[1], &sl->top_borders_allocated[1],
 +    av_fast_mallocz(&sl->top_borders[1], &sl->top_borders_allocated[1],
                     h->mb_width * 16 * 3 * sizeof(uint8_t) * 2);
  
      if (!sl->bipred_scratchpad || !sl->edge_emu_buffer ||
@@@ -196,10 -196,6 +196,10 @@@ static int alloc_picture(H264Context *h
      if (ret < 0)
          goto fail;
  
 +    pic->crop     = h->ps.sps->crop;
 +    pic->crop_top = h->ps.sps->crop_top;
 +    pic->crop_left= h->ps.sps->crop_left;
 +
      if (h->avctx->hwaccel) {
          const AVHWAccel *hwaccel = h->avctx->hwaccel;
          av_assert0(!pic->hwaccel_picture_private);
              pic->hwaccel_picture_private = pic->hwaccel_priv_buf->data;
          }
      }
 +    if (CONFIG_GRAY && !h->avctx->hwaccel && h->flags & AV_CODEC_FLAG_GRAY && pic->f->data[2]) {
 +        int h_chroma_shift, v_chroma_shift;
 +        av_pix_fmt_get_chroma_sub_sample(pic->f->format,
 +                                         &h_chroma_shift, &v_chroma_shift);
 +
 +        for(i=0; i<AV_CEIL_RSHIFT(pic->f->height, v_chroma_shift); i++) {
 +            memset(pic->f->data[1] + pic->f->linesize[1]*i,
 +                   0x80, AV_CEIL_RSHIFT(pic->f->width, h_chroma_shift));
 +            memset(pic->f->data[2] + pic->f->linesize[2]*i,
 +                   0x80, AV_CEIL_RSHIFT(pic->f->width, h_chroma_shift));
 +        }
 +    }
  
      if (!h->qscale_table_pool) {
          ret = init_table_pools(h);
@@@ -274,13 -258,40 +274,13 @@@ static int find_unused_picture(H264Cont
      return i;
  }
  
 -static int initialize_cur_frame(H264Context *h)
 -{
 -    H264Picture *cur;
 -    int ret;
 -
 -    release_unused_pictures(h, 1);
 -    ff_h264_unref_picture(h, &h->cur_pic);
 -    h->cur_pic_ptr = NULL;
 -
 -    ret = find_unused_picture(h);
 -    if (ret < 0) {
 -        av_log(h->avctx, AV_LOG_ERROR, "no frame buffer available\n");
 -        return ret;
 -    }
 -    cur = &h->DPB[ret];
 -
 -    ret = alloc_picture(h, cur);
 -    if (ret < 0)
 -        return ret;
 -
 -    ret = ff_h264_ref_picture(h, &h->cur_pic, cur);
 -    if (ret < 0)
 -        return ret;
 -    h->cur_pic_ptr = cur;
 -
 -    return 0;
 -}
  
 -#define IN_RANGE(a, b, size) (((a) >= (b)) && ((a) < ((b) + (size))))
 +#define IN_RANGE(a, b, size) (((void*)(a) >= (void*)(b)) && ((void*)(a) < (void*)((b) + (size))))
  
  #define REBASE_PICTURE(pic, new_ctx, old_ctx)             \
 -    ((pic && pic >= old_ctx->DPB &&                       \
 -      pic < old_ctx->DPB + H264_MAX_PICTURE_COUNT) ?          \
 -     &new_ctx->DPB[pic - old_ctx->DPB] : NULL)
 +    (((pic) && (pic) >= (old_ctx)->DPB &&                       \
 +      (pic) < (old_ctx)->DPB + H264_MAX_PICTURE_COUNT) ?          \
 +     &(new_ctx)->DPB[(pic) - (old_ctx)->DPB] : NULL)
  
  static void copy_picture_range(H264Picture **to, H264Picture **from, int count,
                                 H264Context *new_base,
      int i;
  
      for (i = 0; i < count; i++) {
 -        assert((IN_RANGE(from[i], old_base, sizeof(*old_base)) ||
 -                IN_RANGE(from[i], old_base->DPB,
 -                         sizeof(H264Picture) * H264_MAX_PICTURE_COUNT) ||
 -                !from[i]));
 +        av_assert1(!from[i] ||
 +                   IN_RANGE(from[i], old_base, 1) ||
 +                   IN_RANGE(from[i], old_base->DPB, H264_MAX_PICTURE_COUNT));
          to[i] = REBASE_PICTURE(from[i], new_base, old_base);
      }
  }
@@@ -306,12 -318,11 +306,12 @@@ int ff_h264_update_thread_context(AVCod
      int need_reinit = 0;
      int i, ret;
  
 -    if (dst == src || !h1->context_initialized)
 +    if (dst == src)
          return 0;
  
 -    if (!h1->ps.sps)
 -        return AVERROR_INVALIDDATA;
 +    // We can't fail if SPS isn't set at it breaks current skip_frame code
 +    //if (!h1->ps.sps)
 +    //    return AVERROR_INVALIDDATA;
  
      if (inited &&
          (h->width                 != h1->width                 ||
          need_reinit = 1;
      }
  
 +    /* copy block_offset since frame_start may not be called */
 +    memcpy(h->block_offset, h1->block_offset, sizeof(h->block_offset));
 +
      // SPS/PPS
      for (i = 0; i < FF_ARRAY_ELEMS(h->ps.sps_list); i++) {
          av_buffer_unref(&h->ps.sps_list[i]);
          }
      }
  
 -    h->ps.sps = h1->ps.sps;
 +    av_buffer_unref(&h->ps.pps_ref);
 +    av_buffer_unref(&h->ps.sps_ref);
 +    h->ps.pps = NULL;
 +    h->ps.sps = NULL;
 +    if (h1->ps.pps_ref) {
 +        h->ps.pps_ref = av_buffer_ref(h1->ps.pps_ref);
 +        if (!h->ps.pps_ref)
 +            return AVERROR(ENOMEM);
 +        h->ps.pps = (const PPS*)h->ps.pps_ref->data;
 +    }
 +    if (h1->ps.sps_ref) {
 +        h->ps.sps_ref = av_buffer_ref(h1->ps.sps_ref);
 +        if (!h->ps.sps_ref)
 +            return AVERROR(ENOMEM);
 +        h->ps.sps = (const SPS*)h->ps.sps_ref->data;
 +    }
  
      if (need_reinit || !inited) {
          h->width     = h1->width;
          h->mb_stride = h1->mb_stride;
          h->b_stride  = h1->b_stride;
  
 -        if ((err = h264_slice_header_init(h)) < 0) {
 -            av_log(h->avctx, AV_LOG_ERROR, "h264_slice_header_init() failed");
 -            return err;
 +        if (h->context_initialized || h1->context_initialized) {
 +            if ((err = h264_slice_header_init(h)) < 0) {
 +                av_log(h->avctx, AV_LOG_ERROR, "h264_slice_header_init() failed");
 +                return err;
 +            }
          }
  
          /* copy block_offset since frame_start may not be called */
      h->picture_structure    = h1->picture_structure;
      h->mb_aff_frame         = h1->mb_aff_frame;
      h->droppable            = h1->droppable;
 +    h->backup_width         = h1->backup_width;
 +    h->backup_height        = h1->backup_height;
 +    h->backup_pix_fmt       = h1->backup_pix_fmt;
  
      for (i = 0; i < H264_MAX_PICTURE_COUNT; i++) {
          ff_h264_unref_picture(h, &h->DPB[i]);
      // extradata/NAL handling
      h->is_avc = h1->is_avc;
      h->nal_length_size = h1->nal_length_size;
 +    h->sei.unregistered.x264_build = h1->sei.unregistered.x264_build;
  
      memcpy(&h->poc,        &h1->poc,        sizeof(h->poc));
  
 +    memcpy(h->default_ref, h1->default_ref, sizeof(h->default_ref));
      memcpy(h->short_ref,   h1->short_ref,   sizeof(h->short_ref));
      memcpy(h->long_ref,    h1->long_ref,    sizeof(h->long_ref));
      memcpy(h->delayed_pic, h1->delayed_pic, sizeof(h->delayed_pic));
      copy_picture_range(h->delayed_pic, h1->delayed_pic,
                         MAX_DELAYED_PIC_COUNT + 2, h, h1);
  
 +    h->frame_recovered       = h1->frame_recovered;
 +
      if (!h->cur_pic_ptr)
          return 0;
  
      h->poc.prev_frame_num        = h->poc.frame_num;
  
      h->recovery_frame        = h1->recovery_frame;
 -    h->frame_recovered       = h1->frame_recovered;
  
      return err;
  }
@@@ -466,28 -451,12 +466,28 @@@ static int h264_frame_start(H264Contex
      H264Picture *pic;
      int i, ret;
      const int pixel_shift = h->pixel_shift;
 +    int c[4] = {
 +        1<<(h->ps.sps->bit_depth_luma-1),
 +        1<<(h->ps.sps->bit_depth_chroma-1),
 +        1<<(h->ps.sps->bit_depth_chroma-1),
 +        -1
 +    };
  
 -    ret = initialize_cur_frame(h);
 -    if (ret < 0)
 -        return ret;
 +    if (!ff_thread_can_start_frame(h->avctx)) {
 +        av_log(h->avctx, AV_LOG_ERROR, "Attempt to start a frame outside SETUP state\n");
 +        return -1;
 +    }
 +
 +    release_unused_pictures(h, 1);
 +    h->cur_pic_ptr = NULL;
 +
 +    i = find_unused_picture(h);
 +    if (i < 0) {
 +        av_log(h->avctx, AV_LOG_ERROR, "no frame buffer available\n");
 +        return i;
 +    }
 +    pic = &h->DPB[i];
  
 -    pic = h->cur_pic_ptr;
      pic->reference              = h->droppable ? 0 : h->picture_structure;
      pic->f->coded_picture_number = h->coded_picture_number++;
      pic->field_picture          = h->picture_structure != PICT_FRAME;
      pic->f->key_frame = 0;
      pic->mmco_reset  = 0;
      pic->recovered   = 0;
 +    pic->invalid_gap = 0;
 +    pic->sei_recovery_frame_cnt = h->sei.recovery_point.recovery_frame_cnt;
  
      pic->f->pict_type = h->slice_ctx[0].slice_type;
  
 -    if (CONFIG_ERROR_RESILIENCE && h->enable_er)
 +    if ((ret = alloc_picture(h, pic)) < 0)
 +        return ret;
 +    if(!h->frame_recovered && !h->avctx->hwaccel
 +#if FF_API_CAP_VDPAU
 +       && !(h->avctx->codec->capabilities & AV_CODEC_CAP_HWACCEL_VDPAU)
 +#endif
 +       )
 +        ff_color_frame(pic->f, c);
 +
 +    h->cur_pic_ptr = pic;
 +    ff_h264_unref_picture(h, &h->cur_pic);
 +    if (CONFIG_ERROR_RESILIENCE) {
 +        ff_h264_set_erpic(&h->slice_ctx[0].er.cur_pic, NULL);
 +    }
 +
 +    if ((ret = ff_h264_ref_picture(h, &h->cur_pic, h->cur_pic_ptr)) < 0)
 +        return ret;
 +
 +    for (i = 0; i < h->nb_slice_ctx; i++) {
 +        h->slice_ctx[i].linesize   = h->cur_pic_ptr->f->linesize[0];
 +        h->slice_ctx[i].uvlinesize = h->cur_pic_ptr->f->linesize[1];
 +    }
 +
 +    if (CONFIG_ERROR_RESILIENCE && h->enable_er) {
          ff_er_frame_start(&h->slice_ctx[0].er);
 +        ff_h264_set_erpic(&h->slice_ctx[0].er.last_pic, NULL);
 +        ff_h264_set_erpic(&h->slice_ctx[0].er.next_pic, NULL);
 +    }
  
      for (i = 0; i < 16; i++) {
          h->block_offset[i]           = (4 * ((scan8[i] - scan8[0]) & 7) << pixel_shift) + 4 * pic->f->linesize[0] * ((scan8[i] - scan8[0]) >> 3);
          h->block_offset[48 + 32 + i] = (4 * ((scan8[i] - scan8[0]) & 7) << pixel_shift) + 8 * pic->f->linesize[1] * ((scan8[i] - scan8[0]) >> 3);
      }
  
 -    /* Some macroblocks can be accessed before they're available in case
 -     * of lost slices, MBAFF or threading. */
 -    memset(h->slice_table, -1,
 -           (h->mb_height * h->mb_stride - 1) * sizeof(*h->slice_table));
 -
      /* We mark the current picture as non-reference after allocating it, so
       * that if we break out due to an error it can be released automatically
       * in the next ff_mpv_frame_start().
@@@ -682,7 -628,7 +682,7 @@@ static void implicit_weight_table(cons
              cur_poc = h->cur_pic_ptr->field_poc[h->picture_structure - 1];
          }
          if (sl->ref_count[0] == 1 && sl->ref_count[1] == 1 && !FRAME_MBAFF(h) &&
 -            sl->ref_list[0][0].poc + sl->ref_list[1][0].poc == 2 * cur_poc) {
 +            sl->ref_list[0][0].poc + (int64_t)sl->ref_list[1][0].poc == 2 * cur_poc) {
              sl->pwt.use_weight        = 0;
              sl->pwt.use_weight_chroma = 0;
              return;
      sl->pwt.chroma_log2_weight_denom = 5;
  
      for (ref0 = ref_start; ref0 < ref_count0; ref0++) {
 -        int poc0 = sl->ref_list[0][ref0].poc;
 +        int64_t poc0 = sl->ref_list[0][ref0].poc;
          for (ref1 = ref_start; ref1 < ref_count1; ref1++) {
              int w = 32;
              if (!sl->ref_list[0][ref0].parent->long_ref && !sl->ref_list[1][ref1].parent->long_ref) {
@@@ -734,13 -680,13 +734,13 @@@ static void init_scan_tables(H264Contex
  {
      int i;
      for (i = 0; i < 16; i++) {
 -#define TRANSPOSE(x) (x >> 2) | ((x << 2) & 0xF)
 +#define TRANSPOSE(x) ((x) >> 2) | (((x) << 2) & 0xF)
          h->zigzag_scan[i] = TRANSPOSE(ff_zigzag_scan[i]);
          h->field_scan[i]  = TRANSPOSE(field_scan[i]);
  #undef TRANSPOSE
      }
      for (i = 0; i < 64; i++) {
 -#define TRANSPOSE(x) (x >> 3) | ((x & 7) << 3)
 +#define TRANSPOSE(x) ((x) >> 3) | (((x) & 7) << 3)
          h->zigzag_scan8x8[i]       = TRANSPOSE(ff_zigzag_direct[i]);
          h->zigzag_scan8x8_cavlc[i] = TRANSPOSE(zigzag_scan8x8_cavlc[i]);
          h->field_scan8x8[i]        = TRANSPOSE(field_scan8x8[i]);
  #undef TRANSPOSE
      }
      if (h->ps.sps->transform_bypass) { // FIXME same ugly
 -        h->zigzag_scan_q0          = ff_zigzag_scan;
 -        h->zigzag_scan8x8_q0       = ff_zigzag_direct;
 -        h->zigzag_scan8x8_cavlc_q0 = zigzag_scan8x8_cavlc;
 -        h->field_scan_q0           = field_scan;
 -        h->field_scan8x8_q0        = field_scan8x8;
 -        h->field_scan8x8_cavlc_q0  = field_scan8x8_cavlc;
 +        memcpy(h->zigzag_scan_q0          , ff_zigzag_scan          , sizeof(h->zigzag_scan_q0         ));
 +        memcpy(h->zigzag_scan8x8_q0       , ff_zigzag_direct        , sizeof(h->zigzag_scan8x8_q0      ));
 +        memcpy(h->zigzag_scan8x8_cavlc_q0 , zigzag_scan8x8_cavlc    , sizeof(h->zigzag_scan8x8_cavlc_q0));
 +        memcpy(h->field_scan_q0           , field_scan              , sizeof(h->field_scan_q0          ));
 +        memcpy(h->field_scan8x8_q0        , field_scan8x8           , sizeof(h->field_scan8x8_q0       ));
 +        memcpy(h->field_scan8x8_cavlc_q0  , field_scan8x8_cavlc     , sizeof(h->field_scan8x8_cavlc_q0 ));
      } else {
 -        h->zigzag_scan_q0          = h->zigzag_scan;
 -        h->zigzag_scan8x8_q0       = h->zigzag_scan8x8;
 -        h->zigzag_scan8x8_cavlc_q0 = h->zigzag_scan8x8_cavlc;
 -        h->field_scan_q0           = h->field_scan;
 -        h->field_scan8x8_q0        = h->field_scan8x8;
 -        h->field_scan8x8_cavlc_q0  = h->field_scan8x8_cavlc;
 +        memcpy(h->zigzag_scan_q0          , h->zigzag_scan          , sizeof(h->zigzag_scan_q0         ));
 +        memcpy(h->zigzag_scan8x8_q0       , h->zigzag_scan8x8       , sizeof(h->zigzag_scan8x8_q0      ));
 +        memcpy(h->zigzag_scan8x8_cavlc_q0 , h->zigzag_scan8x8_cavlc , sizeof(h->zigzag_scan8x8_cavlc_q0));
 +        memcpy(h->field_scan_q0           , h->field_scan           , sizeof(h->field_scan_q0          ));
 +        memcpy(h->field_scan8x8_q0        , h->field_scan8x8        , sizeof(h->field_scan8x8_q0       ));
 +        memcpy(h->field_scan8x8_cavlc_q0  , h->field_scan8x8_cavlc  , sizeof(h->field_scan8x8_cavlc_q0 ));
      }
  }
  
 -static enum AVPixelFormat get_pixel_format(H264Context *h)
 +static enum AVPixelFormat get_pixel_format(H264Context *h, int force_callback)
  {
  #define HWACCEL_MAX (CONFIG_H264_DXVA2_HWACCEL + \
                       CONFIG_H264_D3D11VA_HWACCEL + \
                       CONFIG_H264_VAAPI_HWACCEL + \
                       (CONFIG_H264_VDA_HWACCEL * 2) + \
 +                     CONFIG_H264_VIDEOTOOLBOX_HWACCEL + \
                       CONFIG_H264_VDPAU_HWACCEL)
      enum AVPixelFormat pix_fmts[HWACCEL_MAX + 2], *fmt = pix_fmts;
      const enum AVPixelFormat *choices = pix_fmts;
 +    int i;
  
      switch (h->ps.sps->bit_depth_luma) {
      case 9:
          else
              *fmt++ = AV_PIX_FMT_YUV420P10;
          break;
 +    case 12:
 +        if (CHROMA444(h)) {
 +            if (h->avctx->colorspace == AVCOL_SPC_RGB) {
 +                *fmt++ = AV_PIX_FMT_GBRP12;
 +            } else
 +                *fmt++ = AV_PIX_FMT_YUV444P12;
 +        } else if (CHROMA422(h))
 +            *fmt++ = AV_PIX_FMT_YUV422P12;
 +        else
 +            *fmt++ = AV_PIX_FMT_YUV420P12;
 +        break;
 +    case 14:
 +        if (CHROMA444(h)) {
 +            if (h->avctx->colorspace == AVCOL_SPC_RGB) {
 +                *fmt++ = AV_PIX_FMT_GBRP14;
 +            } else
 +                *fmt++ = AV_PIX_FMT_YUV444P14;
 +        } else if (CHROMA422(h))
 +            *fmt++ = AV_PIX_FMT_YUV422P14;
 +        else
 +            *fmt++ = AV_PIX_FMT_YUV420P14;
 +        break;
      case 8:
  #if CONFIG_H264_VDPAU_HWACCEL
          *fmt++ = AV_PIX_FMT_VDPAU;
              *fmt++ = AV_PIX_FMT_VDA_VLD;
              *fmt++ = AV_PIX_FMT_VDA;
  #endif
 +#if CONFIG_H264_VIDEOTOOLBOX_HWACCEL
 +            *fmt++ = AV_PIX_FMT_VIDEOTOOLBOX;
 +#endif
              if (h->avctx->codec->pix_fmts)
                  choices = h->avctx->codec->pix_fmts;
              else if (h->avctx->color_range == AVCOL_RANGE_JPEG)
  
      *fmt = AV_PIX_FMT_NONE;
  
 -    return ff_get_format(h->avctx, choices);
 +    for (i=0; choices[i] != AV_PIX_FMT_NONE; i++)
 +        if (choices[i] == h->avctx->pix_fmt && !force_callback)
 +            return choices[i];
 +    return ff_thread_get_format(h->avctx, choices);
  }
  
  /* export coded and cropped frame dimensions to AVCodecContext */
  static int init_dimensions(H264Context *h)
  {
 -    SPS *sps = h->ps.sps;
 +    const SPS *sps = (const SPS*)h->ps.sps;
      int width  = h->width  - (sps->crop_right + sps->crop_left);
      int height = h->height - (sps->crop_top   + sps->crop_bottom);
 +    av_assert0(sps->crop_right + sps->crop_left < (unsigned)h->width);
 +    av_assert0(sps->crop_top + sps->crop_bottom < (unsigned)h->height);
  
      /* handle container cropping */
      if (FFALIGN(h->avctx->width,  16) == FFALIGN(width,  16) &&
 -        FFALIGN(h->avctx->height, 16) == FFALIGN(height, 16)) {
 +        FFALIGN(h->avctx->height, 16) == FFALIGN(height, 16) &&
 +        h->avctx->width  <= width &&
 +        h->avctx->height <= height
 +    ) {
          width  = h->avctx->width;
          height = h->avctx->height;
      }
  
 -    if (width <= 0 || height <= 0) {
 -        av_log(h->avctx, AV_LOG_ERROR, "Invalid cropped dimensions: %dx%d.\n",
 -               width, height);
 -        if (h->avctx->err_recognition & AV_EF_EXPLODE)
 -            return AVERROR_INVALIDDATA;
 -
 -        av_log(h->avctx, AV_LOG_WARNING, "Ignoring cropping information.\n");
 -        sps->crop_bottom =
 -        sps->crop_top    =
 -        sps->crop_right  =
 -        sps->crop_left   =
 -        sps->crop        = 0;
 -
 -        width  = h->width;
 -        height = h->height;
 -    }
 -
      h->avctx->coded_width  = h->width;
      h->avctx->coded_height = h->height;
      h->avctx->width        = width;
@@@ -917,7 -845,7 +917,7 @@@ static int h264_slice_header_init(H264C
          if (h->sei.unregistered.x264_build < 44U)
              den *= 2;
          av_reduce(&h->avctx->framerate.den, &h->avctx->framerate.num,
 -                  sps->num_units_in_tick, den, 1 << 30);
 +                  sps->num_units_in_tick * h->avctx->ticks_per_frame, den, 1 << 30);
      }
  
      ff_h264_free_tables(h);
      ret = ff_h264_alloc_tables(h);
      if (ret < 0) {
          av_log(h->avctx, AV_LOG_ERROR, "Could not allocate memory\n");
 -        return ret;
 +        goto fail;
 +    }
 +
 +#if FF_API_CAP_VDPAU
 +    if (h->avctx->codec &&
 +        h->avctx->codec->capabilities & AV_CODEC_CAP_HWACCEL_VDPAU &&
 +        (sps->bit_depth_luma != 8 || sps->chroma_format_idc > 1)) {
 +        av_log(h->avctx, AV_LOG_ERROR,
 +                "VDPAU decoding does not support video colorspace.\n");
 +        ret = AVERROR_INVALIDDATA;
 +        goto fail;
      }
 +#endif
  
 -    if (sps->bit_depth_luma < 8 || sps->bit_depth_luma > 10) {
 +    if (sps->bit_depth_luma < 8 || sps->bit_depth_luma > 14 ||
 +        sps->bit_depth_luma == 11 || sps->bit_depth_luma == 13
 +    ) {
          av_log(h->avctx, AV_LOG_ERROR, "Unsupported bit depth %d\n",
                 sps->bit_depth_luma);
 -        return AVERROR_INVALIDDATA;
 +        ret = AVERROR_INVALIDDATA;
 +        goto fail;
      }
  
 +    h->cur_bit_depth_luma         =
      h->avctx->bits_per_raw_sample = sps->bit_depth_luma;
 +    h->cur_chroma_format_idc      = sps->chroma_format_idc;
      h->pixel_shift                = sps->bit_depth_luma > 8;
      h->chroma_format_idc          = sps->chroma_format_idc;
      h->bit_depth_luma             = sps->bit_depth_luma;
          ret = ff_h264_slice_context_init(h, &h->slice_ctx[0]);
          if (ret < 0) {
              av_log(h->avctx, AV_LOG_ERROR, "context_init() failed.\n");
 -            return ret;
 +            goto fail;
          }
      } else {
          for (i = 0; i < h->nb_slice_ctx; i++) {
  
              if ((ret = ff_h264_slice_context_init(h, sl)) < 0) {
                  av_log(h->avctx, AV_LOG_ERROR, "context_init() failed.\n");
 -                return ret;
 +                goto fail;
              }
          }
      }
      h->context_initialized = 1;
  
      return 0;
 +fail:
 +    ff_h264_free_tables(h);
 +    h->context_initialized = 0;
 +    return ret;
 +}
 +
 +static enum AVPixelFormat non_j_pixfmt(enum AVPixelFormat a)
 +{
 +    switch (a) {
 +    case AV_PIX_FMT_YUVJ420P: return AV_PIX_FMT_YUV420P;
 +    case AV_PIX_FMT_YUVJ422P: return AV_PIX_FMT_YUV422P;
 +    case AV_PIX_FMT_YUVJ444P: return AV_PIX_FMT_YUV444P;
 +    default:
 +        return a;
 +    }
  }
  
 -static int h264_init_ps(H264Context *h, const H264SliceContext *sl)
 +static int h264_init_ps(H264Context *h, const H264SliceContext *sl, int first_slice)
  {
      const SPS *sps;
 -    int needs_reinit = 0, ret;
 +    int needs_reinit = 0, must_reinit, ret;
 +
 +    if (first_slice) {
 +        av_buffer_unref(&h->ps.pps_ref);
 +        h->ps.pps = NULL;
 +        h->ps.pps_ref = av_buffer_ref(h->ps.pps_list[sl->pps_id]);
 +        if (!h->ps.pps_ref)
 +            return AVERROR(ENOMEM);
 +        h->ps.pps = (const PPS*)h->ps.pps_ref->data;
 +    }
  
 -    h->ps.pps = (const PPS*)h->ps.pps_list[sl->pps_id]->data;
      if (h->ps.sps != (const SPS*)h->ps.sps_list[h->ps.pps->sps_id]->data) {
 -        h->ps.sps = (SPS*)h->ps.sps_list[h->ps.pps->sps_id]->data;
 +        av_buffer_unref(&h->ps.sps_ref);
 +        h->ps.sps = NULL;
 +        h->ps.sps_ref = av_buffer_ref(h->ps.sps_list[h->ps.pps->sps_id]);
 +        if (!h->ps.sps_ref)
 +            return AVERROR(ENOMEM);
 +        h->ps.sps = (const SPS*)h->ps.sps_ref->data;
 +
 +        if (h->mb_width  != h->ps.sps->mb_width ||
 +            h->mb_height != h->ps.sps->mb_height * (2 - h->ps.sps->frame_mbs_only_flag) ||
 +            h->cur_bit_depth_luma    != h->ps.sps->bit_depth_luma ||
 +            h->cur_chroma_format_idc != h->ps.sps->chroma_format_idc
 +        )
 +            needs_reinit = 1;
  
          if (h->bit_depth_luma    != h->ps.sps->bit_depth_luma ||
              h->chroma_format_idc != h->ps.sps->chroma_format_idc)
      }
      sps = h->ps.sps;
  
 -    h->avctx->profile = ff_h264_get_profile(sps);
 -    h->avctx->level   = sps->level_idc;
 -    h->avctx->refs    = sps->ref_frame_count;
 +    must_reinit = (h->context_initialized &&
 +                    (   16*sps->mb_width != h->avctx->coded_width
 +                     || 16*sps->mb_height * (2 - sps->frame_mbs_only_flag) != h->avctx->coded_height
 +                     || h->cur_bit_depth_luma    != sps->bit_depth_luma
 +                     || h->cur_chroma_format_idc != sps->chroma_format_idc
 +                     || h->mb_width  != sps->mb_width
 +                     || h->mb_height != sps->mb_height * (2 - sps->frame_mbs_only_flag)
 +                    ));
 +    if (h->avctx->pix_fmt == AV_PIX_FMT_NONE
 +        || (non_j_pixfmt(h->avctx->pix_fmt) != non_j_pixfmt(get_pixel_format(h, 0))))
 +        must_reinit = 1;
 +
 +    if (first_slice && av_cmp_q(sps->sar, h->avctx->sample_aspect_ratio))
 +        must_reinit = 1;
  
 -    if (h->mb_width  != sps->mb_width ||
 -        h->mb_height != sps->mb_height * (2 - sps->frame_mbs_only_flag))
 -        needs_reinit = 1;
 +    if (!h->setup_finished) {
 +        h->avctx->profile = ff_h264_get_profile(sps);
 +        h->avctx->level   = sps->level_idc;
 +        h->avctx->refs    = sps->ref_frame_count;
  
 -    h->mb_width  = sps->mb_width;
 -    h->mb_height = sps->mb_height * (2 - sps->frame_mbs_only_flag);
 -    h->mb_num    = h->mb_width * h->mb_height;
 -    h->mb_stride = h->mb_width + 1;
 +        h->mb_width  = sps->mb_width;
 +        h->mb_height = sps->mb_height * (2 - sps->frame_mbs_only_flag);
 +        h->mb_num    = h->mb_width * h->mb_height;
 +        h->mb_stride = h->mb_width + 1;
  
 -    h->b_stride = h->mb_width * 4;
 +        h->b_stride = h->mb_width * 4;
  
 -    h->chroma_y_shift = sps->chroma_format_idc <= 1; // 400 uses yuv420p
 +        h->chroma_y_shift = sps->chroma_format_idc <= 1; // 400 uses yuv420p
  
 -    h->width  = 16 * h->mb_width;
 -    h->height = 16 * h->mb_height;
 +        h->width  = 16 * h->mb_width;
 +        h->height = 16 * h->mb_height;
  
 -    ret = init_dimensions(h);
 -    if (ret < 0)
 -        return ret;
 +        ret = init_dimensions(h);
 +        if (ret < 0)
 +            return ret;
  
 -    if (sps->video_signal_type_present_flag) {
 -        h->avctx->color_range = sps->full_range ? AVCOL_RANGE_JPEG
 -            : AVCOL_RANGE_MPEG;
 -        if (sps->colour_description_present_flag) {
 -            if (h->avctx->colorspace != sps->colorspace)
 -                needs_reinit = 1;
 -            h->avctx->color_primaries = sps->color_primaries;
 -            h->avctx->color_trc       = sps->color_trc;
 -            h->avctx->colorspace      = sps->colorspace;
 +        if (sps->video_signal_type_present_flag) {
 +            h->avctx->color_range = sps->full_range > 0 ? AVCOL_RANGE_JPEG
 +                                                        : AVCOL_RANGE_MPEG;
 +            if (sps->colour_description_present_flag) {
 +                if (h->avctx->colorspace != sps->colorspace)
 +                    needs_reinit = 1;
 +                h->avctx->color_primaries = sps->color_primaries;
 +                h->avctx->color_trc       = sps->color_trc;
 +                h->avctx->colorspace      = sps->colorspace;
 +            }
          }
      }
  
 -    if (!h->context_initialized || needs_reinit) {
 +    if (!h->context_initialized || must_reinit || needs_reinit) {
 +        int flush_changes = h->context_initialized;
          h->context_initialized = 0;
          if (sl != h->slice_ctx) {
              av_log(h->avctx, AV_LOG_ERROR,
              return AVERROR_INVALIDDATA;
          }
  
 -        ff_h264_flush_change(h);
 +        av_assert1(first_slice);
 +
 +        if (flush_changes)
 +            ff_h264_flush_change(h);
  
 -        if ((ret = get_pixel_format(h)) < 0)
 +        if ((ret = get_pixel_format(h, 1)) < 0)
              return ret;
          h->avctx->pix_fmt = ret;
  
          av_log(h->avctx, AV_LOG_VERBOSE, "Reinit context to %dx%d, "
 -               "pix_fmt: %d\n", h->width, h->height, h->avctx->pix_fmt);
 +               "pix_fmt: %s\n", h->width, h->height, av_get_pix_fmt_name(h->avctx->pix_fmt));
  
          if ((ret = h264_slice_header_init(h)) < 0) {
              av_log(h->avctx, AV_LOG_ERROR,
@@@ -1185,7 -1045,7 +1185,7 @@@ static int h264_export_frame_props(H264
          /* Derive top_field_first from field pocs. */
          cur->f->top_field_first = cur->field_poc[0] < cur->field_poc[1];
      } else {
 -        if (cur->f->interlaced_frame || sps->pic_struct_present_flag) {
 +        if (sps->pic_struct_present_flag) {
              /* Use picture timing SEI information. Even if it is a
               * information of a past frame, better than nothing. */
              if (h->sei.picture_timing.pic_struct == SEI_PIC_STRUCT_TOP_BOTTOM ||
                  cur->f->top_field_first = 1;
              else
                  cur->f->top_field_first = 0;
 +        } else if (cur->f->interlaced_frame) {
 +            /* Default to top field first when pic_struct_present_flag
 +             * is not set but interlaced frame detected */
 +            cur->f->top_field_first = 1;
          } else {
              /* Most likely progressive */
              cur->f->top_field_first = 0;
      }
  
      if (h->sei.frame_packing.present &&
 -        h->sei.frame_packing.arrangement_type >= 0 &&
 -        h->sei.frame_packing.arrangement_type <= 6 &&
 +        h->sei.frame_packing.frame_packing_arrangement_type <= 6 &&
          h->sei.frame_packing.content_interpretation_type > 0 &&
          h->sei.frame_packing.content_interpretation_type < 3) {
          H264SEIFramePacking *fp = &h->sei.frame_packing;
          AVStereo3D *stereo = av_stereo3d_create_side_data(cur->f);
 -        if (!stereo)
 -            return AVERROR(ENOMEM);
 -
 -        switch (fp->arrangement_type) {
 +        if (stereo) {
 +        switch (fp->frame_packing_arrangement_type) {
          case 0:
              stereo->type = AV_STEREO3D_CHECKERBOARD;
              break;
              stereo->type = AV_STEREO3D_LINES;
              break;
          case 3:
 -            if (fp->quincunx_subsampling)
 +            if (fp->quincunx_sampling_flag)
                  stereo->type = AV_STEREO3D_SIDEBYSIDE_QUINCUNX;
              else
                  stereo->type = AV_STEREO3D_SIDEBYSIDE;
  
          if (fp->content_interpretation_type == 2)
              stereo->flags = AV_STEREO3D_FLAG_INVERT;
 +        }
      }
  
      if (h->sei.display_orientation.present &&
          AVFrameSideData *rotation = av_frame_new_side_data(cur->f,
                                                             AV_FRAME_DATA_DISPLAYMATRIX,
                                                             sizeof(int32_t) * 9);
 -        if (!rotation)
 -            return AVERROR(ENOMEM);
 -
 -        av_display_rotation_set((int32_t *)rotation->data, angle);
 -        av_display_matrix_flip((int32_t *)rotation->data,
 -                               o->hflip, o->vflip);
 +        if (rotation) {
 +            av_display_rotation_set((int32_t *)rotation->data, angle);
 +            av_display_matrix_flip((int32_t *)rotation->data,
 +                                   o->hflip, o->vflip);
 +        }
      }
  
      if (h->sei.afd.present) {
          AVFrameSideData *sd = av_frame_new_side_data(cur->f, AV_FRAME_DATA_AFD,
                                                       sizeof(uint8_t));
 -        if (!sd)
 -            return AVERROR(ENOMEM);
  
 -        *sd->data = h->sei.afd.active_format_description;
 -        h->sei.afd.present = 0;
 +        if (sd) {
 +            *sd->data = h->sei.afd.active_format_description;
 +            h->sei.afd.present = 0;
 +        }
      }
  
      if (h->sei.a53_caption.a53_caption) {
          AVFrameSideData *sd = av_frame_new_side_data(cur->f,
                                                       AV_FRAME_DATA_A53_CC,
                                                       a53->a53_caption_size);
 -        if (!sd)
 -            return AVERROR(ENOMEM);
 -
 -        memcpy(sd->data, a53->a53_caption, a53->a53_caption_size);
 +        if (sd)
 +            memcpy(sd->data, a53->a53_caption, a53->a53_caption_size);
          av_freep(&a53->a53_caption);
          a53->a53_caption_size = 0;
 +        h->avctx->properties |= FF_CODEC_PROPERTY_CLOSED_CAPTIONS;
      }
  
      return 0;
   * or a second field in a pair and does the necessary setup.
   */
  static int h264_field_start(H264Context *h, const H264SliceContext *sl,
 -                            const H2645NAL *nal)
 +                            const H2645NAL *nal, int first_slice)
  {
 +    int i;
      const SPS *sps;
  
      int last_pic_structure, last_pic_droppable, ret;
  
 -    ret = h264_init_ps(h, sl);
 +    ret = h264_init_ps(h, sl, first_slice);
      if (ret < 0)
          return ret;
  
       * Here, we're using that to see if we should mark previously
       * decode frames as "finished".
       * We have to do that before the "dummy" in-between frame allocation,
 -     * since that can modify s->current_picture_ptr. */
 +     * since that can modify h->cur_pic_ptr. */
      if (h->first_field) {
 -        assert(h->cur_pic_ptr);
 -        assert(h->cur_pic_ptr->f->buf[0]);
 +        av_assert0(h->cur_pic_ptr);
 +        av_assert0(h->cur_pic_ptr->f->buf[0]);
          assert(h->cur_pic_ptr->reference != DELAYED_PIC_REF);
  
 +        /* Mark old field/frame as completed */
 +        if (h->cur_pic_ptr->tf.owner == h->avctx) {
 +            ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
 +                                      last_pic_structure == PICT_BOTTOM_FIELD);
 +        }
 +
          /* figure out if we have a complementary field pair */
          if (!FIELD_PICTURE(h) || h->picture_structure == last_pic_structure) {
              /* Previous field is unmatched. Don't display it, but let it
               * remain for reference if marked as such. */
 -            if (!last_pic_droppable && last_pic_structure != PICT_FRAME) {
 +            if (last_pic_structure != PICT_FRAME) {
                  ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
                                            last_pic_structure == PICT_TOP_FIELD);
              }
                   * different frame_nums. Consider this field first in
                   * pair. Throw away previous field except for reference
                   * purposes. */
 -                if (!last_pic_droppable && last_pic_structure != PICT_FRAME) {
 +                if (last_pic_structure != PICT_FRAME) {
                      ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
                                                last_pic_structure == PICT_TOP_FIELD);
                  }
          }
      }
  
 -    while (h->poc.frame_num != h->poc.prev_frame_num &&
 +    while (h->poc.frame_num != h->poc.prev_frame_num && !h->first_field &&
             h->poc.frame_num != (h->poc.prev_frame_num + 1) % (1 << sps->log2_max_frame_num)) {
          H264Picture *prev = h->short_ref_count ? h->short_ref[0] : NULL;
          av_log(h->avctx, AV_LOG_DEBUG, "Frame num gap %d %d\n",
                 h->poc.frame_num, h->poc.prev_frame_num);
 -        ret = initialize_cur_frame(h);
 +        if (!sps->gaps_in_frame_num_allowed_flag)
 +            for(i=0; i<FF_ARRAY_ELEMS(h->last_pocs); i++)
 +                h->last_pocs[i] = INT_MIN;
 +        ret = h264_frame_start(h);
          if (ret < 0) {
              h->first_field = 0;
              return ret;
          h->poc.prev_frame_num++;
          h->poc.prev_frame_num        %= 1 << sps->log2_max_frame_num;
          h->cur_pic_ptr->frame_num = h->poc.prev_frame_num;
 +        h->cur_pic_ptr->invalid_gap = !sps->gaps_in_frame_num_allowed_flag;
          ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX, 0);
          ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX, 1);
  
                                (const uint8_t **)prev->f->data,
                                prev->f->linesize,
                                prev->f->format,
 -                              h->mb_width  * 16,
 -                              h->mb_height * 16);
 +                              prev->f->width,
 +                              prev->f->height);
                  h->short_ref[0]->poc = prev->poc + 2;
              }
              h->short_ref[0]->frame_num = h->poc.prev_frame_num;
       * We're using that to see whether to continue decoding in that
       * frame, or to allocate a new one. */
      if (h->first_field) {
 -        assert(h->cur_pic_ptr);
 -        assert(h->cur_pic_ptr->f->buf[0]);
 +        av_assert0(h->cur_pic_ptr);
 +        av_assert0(h->cur_pic_ptr->f->buf[0]);
          assert(h->cur_pic_ptr->reference != DELAYED_PIC_REF);
  
          /* figure out if we have a complementary field pair */
          if (!FIELD_PICTURE(h) || h->picture_structure == last_pic_structure) {
              /* Previous field is unmatched. Don't display it, but let it
               * remain for reference if marked as such. */
 +            h->missing_fields ++;
              h->cur_pic_ptr = NULL;
              h->first_field = FIELD_PICTURE(h);
          } else {
 +            h->missing_fields = 0;
              if (h->cur_pic_ptr->frame_num != h->poc.frame_num) {
 +                ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
 +                                          h->picture_structure==PICT_BOTTOM_FIELD);
                  /* This and the previous field had different frame_nums.
                   * Consider this field first in pair. Throw away previous
                   * one except for reference purposes. */
      } else {
          release_unused_pictures(h, 0);
      }
 +    /* Some macroblocks can be accessed before they're available in case
 +    * of lost slices, MBAFF or threading. */
 +    if (FIELD_PICTURE(h)) {
 +        for(i = (h->picture_structure == PICT_BOTTOM_FIELD); i<h->mb_height; i++)
 +            memset(h->slice_table + i*h->mb_stride, -1, (h->mb_stride - (i+1==h->mb_height)) * sizeof(*h->slice_table));
 +    } else {
 +        memset(h->slice_table, -1,
 +            (h->mb_height * h->mb_stride - 1) * sizeof(*h->slice_table));
 +    }
  
      ff_h264_init_poc(h->cur_pic_ptr->field_poc, &h->cur_pic_ptr->poc,
                       h->ps.sps, &h->poc, h->picture_structure, nal->ref_idc);
      h->nb_mmco = sl->nb_mmco;
      h->explicit_ref_marking = sl->explicit_ref_marking;
  
+     h->picture_idr = nal->type == H264_NAL_IDR_SLICE;
      /* Set the frame properties/side data. Only done for the second field in
       * field coded frames, since some SEI information is present for each field
       * and is merged by the SEI parsing code. */
      return 0;
  }
  
 -static int h264_slice_header_parse(H264SliceContext *sl, const H2645NAL *nal,
 -                                   const H264ParamSets *ps, AVCodecContext *avctx)
 +static int h264_slice_header_parse(const H264Context *h, H264SliceContext *sl,
 +                                   const H2645NAL *nal)
  {
      const SPS *sps;
      const PPS *pps;
      int ret;
      unsigned int slice_type, tmp, i;
      int field_pic_flag, bottom_field_flag;
 -    int droppable, picture_structure;
 +    int first_slice = sl == h->slice_ctx && !h->current_slice;
 +    int picture_structure;
  
 -    sl->first_mb_addr = get_ue_golomb(&sl->gb);
 +    if (first_slice)
 +        av_assert0(!h->setup_finished);
 +
 +    sl->first_mb_addr = get_ue_golomb_long(&sl->gb);
  
      slice_type = get_ue_golomb_31(&sl->gb);
      if (slice_type > 9) {
 -        av_log(avctx, AV_LOG_ERROR,
 +        av_log(h->avctx, AV_LOG_ERROR,
                 "slice type %d too large at %d\n",
                 slice_type, sl->first_mb_addr);
          return AVERROR_INVALIDDATA;
  
      if (nal->type  == H264_NAL_IDR_SLICE &&
          sl->slice_type_nos != AV_PICTURE_TYPE_I) {
 -        av_log(avctx, AV_LOG_ERROR, "A non-intra slice in an IDR NAL unit.\n");
 +        av_log(h->avctx, AV_LOG_ERROR, "A non-intra slice in an IDR NAL unit.\n");
          return AVERROR_INVALIDDATA;
      }
  
      sl->pps_id = get_ue_golomb(&sl->gb);
      if (sl->pps_id >= MAX_PPS_COUNT) {
 -        av_log(avctx, AV_LOG_ERROR, "pps_id %u out of range\n", sl->pps_id);
 +        av_log(h->avctx, AV_LOG_ERROR, "pps_id %u out of range\n", sl->pps_id);
          return AVERROR_INVALIDDATA;
      }
 -    if (!ps->pps_list[sl->pps_id]) {
 -        av_log(avctx, AV_LOG_ERROR,
 +    if (!h->ps.pps_list[sl->pps_id]) {
 +        av_log(h->avctx, AV_LOG_ERROR,
                 "non-existing PPS %u referenced\n",
                 sl->pps_id);
          return AVERROR_INVALIDDATA;
      }
 -    pps = (const PPS*)ps->pps_list[sl->pps_id]->data;
 +    pps = (const PPS*)h->ps.pps_list[sl->pps_id]->data;
  
 -    if (!ps->sps_list[pps->sps_id]) {
 -        av_log(avctx, AV_LOG_ERROR,
 +    if (!h->ps.sps_list[pps->sps_id]) {
 +        av_log(h->avctx, AV_LOG_ERROR,
                 "non-existing SPS %u referenced\n", pps->sps_id);
          return AVERROR_INVALIDDATA;
      }
 -    sps = (const SPS*)ps->sps_list[pps->sps_id]->data;
 +    sps = (const SPS*)h->ps.sps_list[pps->sps_id]->data;
  
      sl->frame_num = get_bits(&sl->gb, sps->log2_max_frame_num);
 +    if (!first_slice) {
 +        if (h->poc.frame_num != sl->frame_num) {
 +            av_log(h->avctx, AV_LOG_ERROR, "Frame num change from %d to %d\n",
 +                   h->poc.frame_num, sl->frame_num);
 +            return AVERROR_INVALIDDATA;
 +        }
 +    }
  
      sl->mb_mbaff       = 0;
  
 -    droppable = nal->ref_idc == 0;
      if (sps->frame_mbs_only_flag) {
          picture_structure = PICT_FRAME;
      } else {
 +        if (!sps->direct_8x8_inference_flag && slice_type == AV_PICTURE_TYPE_B) {
 +            av_log(h->avctx, AV_LOG_ERROR, "This stream was generated by a broken encoder, invalid 8x8 inference\n");
 +            return -1;
 +        }
          field_pic_flag = get_bits1(&sl->gb);
          if (field_pic_flag) {
              bottom_field_flag = get_bits1(&sl->gb);
      }
  
      if (nal->type == H264_NAL_IDR_SLICE)
 -        get_ue_golomb(&sl->gb); /* idr_pic_id */
 +        get_ue_golomb_long(&sl->gb); /* idr_pic_id */
  
      if (sps->poc_type == 0) {
          sl->poc_lsb = get_bits(&sl->gb, sps->log2_max_poc_lsb);
  
      ret = ff_h264_parse_ref_count(&sl->list_count, sl->ref_count,
                                    &sl->gb, pps, sl->slice_type_nos,
 -                                  picture_structure);
 +                                  picture_structure, h->avctx);
      if (ret < 0)
          return ret;
  
      if (sl->slice_type_nos != AV_PICTURE_TYPE_I) {
 -       ret = ff_h264_decode_ref_pic_list_reordering(sl, avctx);
 +       ret = ff_h264_decode_ref_pic_list_reordering(sl, h->avctx);
         if (ret < 0) {
             sl->ref_count[1] = sl->ref_count[0] = 0;
             return ret;
          (pps->weighted_bipred_idc == 1 &&
           sl->slice_type_nos == AV_PICTURE_TYPE_B))
          ff_h264_pred_weight_table(&sl->gb, sps, sl->ref_count,
 -                                  sl->slice_type_nos, &sl->pwt);
 +                                  sl->slice_type_nos, &sl->pwt, h->avctx);
  
      sl->explicit_ref_marking = 0;
      if (nal->ref_idc) {
 -        ret = ff_h264_decode_ref_pic_marking(sl, &sl->gb, nal, avctx);
 -        if (ret < 0 && (avctx->err_recognition & AV_EF_EXPLODE))
 +        ret = ff_h264_decode_ref_pic_marking(sl, &sl->gb, nal, h->avctx);
 +        if (ret < 0 && (h->avctx->err_recognition & AV_EF_EXPLODE))
              return AVERROR_INVALIDDATA;
      }
  
      if (sl->slice_type_nos != AV_PICTURE_TYPE_I && pps->cabac) {
          tmp = get_ue_golomb_31(&sl->gb);
          if (tmp > 2) {
 -            av_log(avctx, AV_LOG_ERROR, "cabac_init_idc %u overflow\n", tmp);
 +            av_log(h->avctx, AV_LOG_ERROR, "cabac_init_idc %u overflow\n", tmp);
              return AVERROR_INVALIDDATA;
          }
          sl->cabac_init_idc = tmp;
      sl->last_qscale_diff = 0;
      tmp = pps->init_qp + get_se_golomb(&sl->gb);
      if (tmp > 51 + 6 * (sps->bit_depth_luma - 8)) {
 -        av_log(avctx, AV_LOG_ERROR, "QP %u out of range\n", tmp);
 +        av_log(h->avctx, AV_LOG_ERROR, "QP %u out of range\n", tmp);
          return AVERROR_INVALIDDATA;
      }
      sl->qscale       = tmp;
      if (pps->deblocking_filter_parameters_present) {
          tmp = get_ue_golomb_31(&sl->gb);
          if (tmp > 2) {
 -            av_log(avctx, AV_LOG_ERROR,
 +            av_log(h->avctx, AV_LOG_ERROR,
                     "deblocking_filter_idc %u out of range\n", tmp);
              return AVERROR_INVALIDDATA;
          }
                  sl->slice_alpha_c0_offset < -12 ||
                  sl->slice_beta_offset >  12     ||
                  sl->slice_beta_offset < -12) {
 -                av_log(avctx, AV_LOG_ERROR,
 +                av_log(h->avctx, AV_LOG_ERROR,
                         "deblocking filter parameters %d %d out of range\n",
                         sl->slice_alpha_c0_offset, sl->slice_beta_offset);
                  return AVERROR_INVALIDDATA;
@@@ -1729,92 -1553,40 +1731,92 @@@ int ff_h264_decode_slice_header(H264Con
                                  const H2645NAL *nal)
  {
      int i, j, ret = 0;
 +    int first_slice = sl == h->slice_ctx && !h->current_slice;
  
 -    ret = h264_slice_header_parse(sl, nal, &h->ps, h->avctx);
 +    ret = h264_slice_header_parse(h, sl, nal);
      if (ret < 0)
          return ret;
  
 -    if (!h->setup_finished) {
 -        if (sl->first_mb_addr == 0) { // FIXME better field boundary detection
 -            if (h->current_slice && h->cur_pic_ptr && FIELD_PICTURE(h)) {
 -                ff_h264_field_end(h, sl, 1);
 -            }
 +    if (sl->first_mb_addr == 0 || !h->current_slice) {
 +        if (h->setup_finished) {
 +            av_log(h->avctx, AV_LOG_ERROR, "Too many fields\n");
 +            return AVERROR_INVALIDDATA;
 +        }
 +    }
  
 -            h->current_slice = 0;
 -            if (!h->first_field) {
 -                if (h->cur_pic_ptr && !h->droppable) {
 -                    ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
 -                                              h->picture_structure == PICT_BOTTOM_FIELD);
 +    if (sl->first_mb_addr == 0) { // FIXME better field boundary detection
 +        if (h->current_slice) {
 +            if (h->max_contexts > 1) {
 +                if (!h->single_decode_warning) {
 +                    av_log(h->avctx, AV_LOG_WARNING, "Cannot decode multiple access units as slice threads\n");
 +                    h->single_decode_warning = 1;
                  }
 +                h->max_contexts = 1;
 +                return SLICE_SINGLETHREAD;
 +            }
 +
 +            if (h->cur_pic_ptr && FIELD_PICTURE(h) && h->first_field) {
 +                ret = ff_h264_field_end(h, h->slice_ctx, 1);
 +                h->current_slice = 0;
 +                if (ret < 0)
 +                    return ret;
 +            } else if (h->cur_pic_ptr && !FIELD_PICTURE(h) && !h->first_field && h->nal_unit_type  == H264_NAL_IDR_SLICE) {
 +                av_log(h, AV_LOG_WARNING, "Broken frame packetizing\n");
 +                ret = ff_h264_field_end(h, h->slice_ctx, 1);
 +                h->current_slice = 0;
 +                ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX, 0);
 +                ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX, 1);
                  h->cur_pic_ptr = NULL;
 +                if (ret < 0)
 +                    return ret;
 +            } else
 +                return AVERROR_INVALIDDATA;
 +        }
 +
 +        if (!h->first_field) {
 +            if (h->cur_pic_ptr && !h->droppable) {
 +                ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
 +                                          h->picture_structure == PICT_BOTTOM_FIELD);
              }
 +            h->cur_pic_ptr = NULL;
          }
 +    }
  
 -        if (h->current_slice == 0) {
 -            ret = h264_field_start(h, sl, nal);
 -            if (ret < 0)
 -                return ret;
 +    if (!h->current_slice)
 +        av_assert0(sl == h->slice_ctx);
 +
 +    if (h->current_slice == 0 && !h->first_field) {
 +        if (
 +            (h->avctx->skip_frame >= AVDISCARD_NONREF && !h->nal_ref_idc) ||
 +            (h->avctx->skip_frame >= AVDISCARD_BIDIR  && sl->slice_type_nos == AV_PICTURE_TYPE_B) ||
 +            (h->avctx->skip_frame >= AVDISCARD_NONINTRA && sl->slice_type_nos != AV_PICTURE_TYPE_I) ||
 +            (h->avctx->skip_frame >= AVDISCARD_NONKEY && h->nal_unit_type != H264_NAL_IDR_SLICE && h->sei.recovery_point.recovery_frame_cnt < 0) ||
 +            h->avctx->skip_frame >= AVDISCARD_ALL) {
 +            return SLICE_SKIPED;
          }
      }
  
 -    if (h->current_slice > 0) {
 -        if (h->ps.pps != (const PPS*)h->ps.pps_list[sl->pps_id]->data) {
 +    if (!first_slice) {
 +        const PPS *pps = (const PPS*)h->ps.pps_list[sl->pps_id]->data;
 +
 +        if (h->ps.pps->sps_id != pps->sps_id ||
 +            h->ps.pps->transform_8x8_mode != pps->transform_8x8_mode /*||
 +            (h->setup_finished && h->ps.pps != pps)*/) {
              av_log(h->avctx, AV_LOG_ERROR, "PPS changed between slices\n");
              return AVERROR_INVALIDDATA;
          }
 +        if (h->ps.sps != (const SPS*)h->ps.sps_list[h->ps.pps->sps_id]->data) {
 +            av_log(h->avctx, AV_LOG_ERROR,
 +               "SPS changed in the middle of the frame\n");
 +            return AVERROR_INVALIDDATA;
 +        }
 +    }
  
 +    if (h->current_slice == 0) {
 +        ret = h264_field_start(h, sl, nal, first_slice);
 +        if (ret < 0)
 +            return ret;
 +    } else {
          if (h->picture_structure != sl->picture_structure ||
              h->droppable         != (nal->ref_idc == 0)) {
              av_log(h->avctx, AV_LOG_ERROR,
          }
      }
  
 -    assert(h->mb_num == h->mb_width * h->mb_height);
+     if (h->picture_idr && nal->type != H264_NAL_IDR_SLICE) {
+         av_log(h->avctx, AV_LOG_ERROR, "Invalid mix of IDR and non-IDR slices\n");
+         return AVERROR_INVALIDDATA;
+     }
 +    av_assert1(h->mb_num == h->mb_width * h->mb_height);
      if (sl->first_mb_addr << FIELD_OR_MBAFF_PICTURE(h) >= h->mb_num ||
          sl->first_mb_addr >= h->mb_num) {
          av_log(h->avctx, AV_LOG_ERROR, "first_mb_in_slice overflow\n");
                                   FIELD_OR_MBAFF_PICTURE(h);
      if (h->picture_structure == PICT_BOTTOM_FIELD)
          sl->resync_mb_y = sl->mb_y = sl->mb_y + 1;
 -    assert(sl->mb_y < h->mb_height);
 +    av_assert1(sl->mb_y < h->mb_height);
  
      ret = ff_h264_build_ref_list(h, sl);
      if (ret < 0)
  
      if (h->avctx->skip_loop_filter >= AVDISCARD_ALL ||
          (h->avctx->skip_loop_filter >= AVDISCARD_NONKEY &&
 +         h->nal_unit_type != H264_NAL_IDR_SLICE) ||
 +        (h->avctx->skip_loop_filter >= AVDISCARD_NONINTRA &&
           sl->slice_type_nos != AV_PICTURE_TYPE_I) ||
          (h->avctx->skip_loop_filter >= AVDISCARD_BIDIR  &&
           sl->slice_type_nos == AV_PICTURE_TYPE_B) ||
           nal->ref_idc == 0))
          sl->deblocking_filter = 0;
  
 -    if (sl->deblocking_filter == 1 && h->nb_slice_ctx > 1) {
 +    if (sl->deblocking_filter == 1 && h->max_contexts > 1) {
          if (h->avctx->flags2 & AV_CODEC_FLAG2_FAST) {
              /* Cheat slightly for speed:
               * Do not bother to deblock across slices. */
                     6 * (h->ps.sps->bit_depth_luma - 8);
  
      sl->slice_num       = ++h->current_slice;
 -    if (sl->slice_num >= MAX_SLICES) {
 -        av_log(h->avctx, AV_LOG_ERROR,
 -               "Too many slices, increase MAX_SLICES and recompile\n");
 +
 +    if (sl->slice_num)
 +        h->slice_row[(sl->slice_num-1)&(MAX_SLICES-1)]= sl->resync_mb_y;
 +    if (   h->slice_row[sl->slice_num&(MAX_SLICES-1)] + 3 >= sl->resync_mb_y
 +        && h->slice_row[sl->slice_num&(MAX_SLICES-1)] <= sl->resync_mb_y
 +        && sl->slice_num >= MAX_SLICES) {
 +        //in case of ASO this check needs to be updated depending on how we decide to assign slice numbers in this case
 +        av_log(h->avctx, AV_LOG_WARNING, "Possibly too many slices (%d >= %d), increase MAX_SLICES and recompile if there are artifacts\n", sl->slice_num, MAX_SLICES);
      }
  
      for (j = 0; j < 2; j++) {
@@@ -2305,7 -2075,7 +2312,7 @@@ static void decode_finish_row(const H26
  
      ff_h264_draw_horiz_band(h, sl, top, height);
  
 -    if (h->droppable)
 +    if (h->droppable || sl->h264->slice_ctx[0].er.error_occurred)
          return;
  
      ff_thread_report_progress(&h->cur_pic_ptr->tf, top + height - 1,
@@@ -2316,14 -2086,15 +2323,14 @@@ static void er_add_slice(H264SliceConte
                           int startx, int starty,
                           int endx, int endy, int status)
  {
 -#if CONFIG_ERROR_RESILIENCE
 -    ERContext *er = &sl->er;
 -
      if (!sl->h264->enable_er)
          return;
  
 -    er->ref_count = sl->ref_count[0];
 -    ff_er_add_slice(er, startx, starty, endx, endy, status);
 -#endif
 +    if (CONFIG_ERROR_RESILIENCE) {
 +        ERContext *er = &sl->h264->slice_ctx[0].er;
 +
 +        ff_er_add_slice(er, startx, starty, endx, endy, status);
 +    }
  }
  
  static int decode_slice(struct AVCodecContext *avctx, void *arg)
  
      sl->mb_skip_run = -1;
  
 +    av_assert0(h->block_offset[15] == (4 * ((scan8[15] - scan8[0]) & 7) << h->pixel_shift) + 4 * sl->linesize * ((scan8[15] - scan8[0]) >> 3));
 +
      if (h->postpone_filter)
          sl->deblocking_filter = 0;
  
      sl->is_complex = FRAME_MBAFF(h) || h->picture_structure != PICT_FRAME ||
                       (CONFIG_GRAY && (h->flags & AV_CODEC_FLAG_GRAY));
  
 +    if (!(h->avctx->active_thread_type & FF_THREAD_SLICE) && h->picture_structure == PICT_FRAME && h->slice_ctx[0].er.error_status_table) {
 +        const int start_i  = av_clip(sl->resync_mb_x + sl->resync_mb_y * h->mb_width, 0, h->mb_num - 1);
 +        if (start_i) {
 +            int prev_status = h->slice_ctx[0].er.error_status_table[h->slice_ctx[0].er.mb_index2xy[start_i - 1]];
 +            prev_status &= ~ VP_START;
 +            if (prev_status != (ER_MV_END | ER_DC_END | ER_AC_END))
 +                h->slice_ctx[0].er.error_occurred = 1;
 +        }
 +    }
 +
      if (h->ps.pps->cabac) {
          /* realign */
          align_get_bits(&sl->gb);
  
          /* init cabac */
 -        ff_init_cabac_decoder(&sl->cabac,
 +        ret = ff_init_cabac_decoder(&sl->cabac,
                                sl->gb.buffer + get_bits_count(&sl->gb) / 8,
                                (get_bits_left(&sl->gb) + 7) / 8);
 +        if (ret < 0)
 +            return ret;
  
          ff_h264_init_cabac_states(h, sl);
  
          for (;;) {
              // START_TIMER
              int ret, eos;
 -
              if (sl->mb_x + sl->mb_y * h->mb_width >= sl->next_slice_idx) {
                  av_log(h->avctx, AV_LOG_ERROR, "Slice overlaps with next at %d\n",
                         sl->next_slice_idx);
 +                er_add_slice(sl, sl->resync_mb_x, sl->resync_mb_y, sl->mb_x,
 +                             sl->mb_y, ER_MB_ERROR);
                  return AVERROR_INVALIDDATA;
              }
  
                      loop_filter(h, sl, lf_x_start, sl->mb_x + 1);
                  goto finish;
              }
 -            if (ret < 0 || sl->cabac.bytestream > sl->cabac.bytestream_end + 2) {
 +            if (sl->cabac.bytestream > sl->cabac.bytestream_end + 2 )
 +                av_log(h->avctx, AV_LOG_DEBUG, "bytestream overread %"PTRDIFF_SPECIFIER"\n", sl->cabac.bytestream_end - sl->cabac.bytestream);
 +            if (ret < 0 || sl->cabac.bytestream > sl->cabac.bytestream_end + 4) {
                  av_log(h->avctx, AV_LOG_ERROR,
 -                       "error while decoding MB %d %d, bytestream %td\n",
 +                       "error while decoding MB %d %d, bytestream %"PTRDIFF_SPECIFIER"\n",
                         sl->mb_x, sl->mb_y,
                         sl->cabac.bytestream_end - sl->cabac.bytestream);
                  er_add_slice(sl, sl->resync_mb_x, sl->resync_mb_y, sl->mb_x,
              if (sl->mb_x + sl->mb_y * h->mb_width >= sl->next_slice_idx) {
                  av_log(h->avctx, AV_LOG_ERROR, "Slice overlaps with next at %d\n",
                         sl->next_slice_idx);
 +                er_add_slice(sl, sl->resync_mb_x, sl->resync_mb_y, sl->mb_x,
 +                             sl->mb_y, ER_MB_ERROR);
                  return AVERROR_INVALIDDATA;
              }
  
                      ff_tlog(h->avctx, "slice end %d %d\n",
                              get_bits_count(&sl->gb), sl->gb.size_in_bits);
  
 -                    if (get_bits_left(&sl->gb) == 0) {
 +                    if (   get_bits_left(&sl->gb) == 0
 +                        || get_bits_left(&sl->gb) > 0 && !(h->avctx->err_recognition & AV_EF_AGGRESSIVE)) {
                          er_add_slice(sl, sl->resync_mb_x, sl->resync_mb_y,
                                       sl->mb_x - 1, sl->mb_y, ER_MB_END);
  
                          goto finish;
                      } else {
                          er_add_slice(sl, sl->resync_mb_x, sl->resync_mb_y,
 -                                     sl->mb_x - 1, sl->mb_y, ER_MB_END);
 +                                     sl->mb_x, sl->mb_y, ER_MB_END);
  
                          return AVERROR_INVALIDDATA;
                      }
@@@ -2547,15 -2298,7 +2554,15 @@@ int ff_h264_execute_decode_slices(H264C
      H264SliceContext *sl;
      int i, j;
  
 -    if (h->avctx->hwaccel)
 +    av_assert0(context_count && h->slice_ctx[context_count - 1].mb_y < h->mb_height);
 +
 +    h->slice_ctx[0].next_slice_idx = INT_MAX;
 +
 +    if (h->avctx->hwaccel
 +#if FF_API_CAP_VDPAU
 +        || h->avctx->codec->capabilities & AV_CODEC_CAP_HWACCEL_VDPAU
 +#endif
 +        )
          return 0;
      if (context_count == 1) {
          int ret;
          h->mb_y = h->slice_ctx[0].mb_y;
          return ret;
      } else {
 +        av_assert0(context_count > 0);
          for (i = 0; i < context_count; i++) {
              int next_slice_idx = h->mb_width * h->mb_height;
              int slice_idx;
  
              sl                 = &h->slice_ctx[i];
 -            sl->er.error_count = 0;
 +            if (CONFIG_ERROR_RESILIENCE) {
 +                sl->er.error_count = 0;
 +            }
  
              /* make sure none of those slices overlap */
              slice_idx = sl->mb_y * h->mb_width + sl->mb_x;
          /* pull back stuff from slices to master context */
          sl                   = &h->slice_ctx[context_count - 1];
          h->mb_y              = sl->mb_y;
 -        for (i = 1; i < context_count; i++)
 -            h->slice_ctx[0].er.error_count += h->slice_ctx[i].er.error_count;
 +        if (CONFIG_ERROR_RESILIENCE) {
 +            for (i = 1; i < context_count; i++)
 +                h->slice_ctx[0].er.error_count += h->slice_ctx[i].er.error_count;
 +        }
  
          if (h->postpone_filter) {
              h->postpone_filter = 0;
diff --combined libavcodec/h264dec.c
@@@ -2,20 -2,20 +2,20 @@@
   * H.26L/H.264/AVC/JVT/14496-10/... decoder
   * Copyright (c) 2003 Michael Niedermayer <michaelni@gmx.at>
   *
 - * 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
   */
  
@@@ -25,9 -25,6 +25,9 @@@
   * @author Michael Niedermayer <michaelni@gmx.at>
   */
  
 +#define UNCHECKED_BITSTREAM_READER 1
 +
 +#include "libavutil/avassert.h"
  #include "libavutil/display.h"
  #include "libavutil/imgutils.h"
  #include "libavutil/opt.h"
  #include "profiles.h"
  #include "rectangle.h"
  #include "thread.h"
 +#include "vdpau_compat.h"
  
 -#include <assert.h>
 +static int h264_decode_end(AVCodecContext *avctx);
  
  const uint16_t ff_h264_mb_sizes[4] = { 256, 384, 512, 768 };
  
 +int avpriv_h264_has_num_reorder_frames(AVCodecContext *avctx)
 +{
 +    H264Context *h = avctx->priv_data;
 +    return h && h->ps.sps ? h->ps.sps->num_reorder_frames : 0;
 +}
 +
  static void h264_er_decode_mb(void *opaque, int ref, int mv_dir, int mv_type,
                                int (*mv)[2][4][2],
                                int mb_x, int mb_y, int mb_intra, int mb_skipped)
      sl->mb_y = mb_y;
      sl->mb_xy = mb_x + mb_y * h->mb_stride;
      memset(sl->non_zero_count_cache, 0, sizeof(sl->non_zero_count_cache));
 -    assert(ref >= 0);
 +    av_assert1(ref >= 0);
      /* FIXME: It is possible albeit uncommon that slice references
       * differ between slices. We take the easy approach and ignore
       * it for now. If this turns out to have any relevance in
       * practice then correct remapping should be added. */
      if (ref >= sl->ref_count[0])
          ref = 0;
 +    if (!sl->ref_list[0][ref].data[0]) {
 +        av_log(h->avctx, AV_LOG_DEBUG, "Reference not available for error concealing\n");
 +        ref = 0;
 +    }
 +    if ((sl->ref_list[0][ref].reference&3) != 3) {
 +        av_log(h->avctx, AV_LOG_DEBUG, "Reference invalid\n");
 +        return;
 +    }
      fill_rectangle(&h->cur_pic.ref_index[0][4 * sl->mb_xy],
                     2, 2, 2, ref, 1);
      fill_rectangle(&sl->ref_cache[0][scan8[0]], 4, 4, 8, ref, 1);
      fill_rectangle(sl->mv_cache[0][scan8[0]], 4, 4, 8,
                     pack16to32((*mv)[0][0][0], (*mv)[0][0][1]), 4);
 -    assert(!FRAME_MBAFF(h));
 +    sl->mb_mbaff =
 +    sl->mb_field_decoding_flag = 0;
      ff_h264_hl_decode_mb(h, &h->slice_ctx[0]);
  }
  
@@@ -182,11 -163,11 +182,11 @@@ void ff_h264_free_tables(H264Context *h
  int ff_h264_alloc_tables(H264Context *h)
  {
      const int big_mb_num = h->mb_stride * (h->mb_height + 1);
 -    const int row_mb_num = h->mb_stride * 2 * h->nb_slice_ctx;
 +    const int row_mb_num = 2*h->mb_stride*FFMAX(h->nb_slice_ctx, 1);
      int x, y;
  
 -    FF_ALLOCZ_OR_GOTO(h->avctx, h->intra4x4_pred_mode,
 -                      row_mb_num * 8 * sizeof(uint8_t), fail)
 +    FF_ALLOCZ_ARRAY_OR_GOTO(h->avctx, h->intra4x4_pred_mode,
 +                      row_mb_num, 8 * sizeof(uint8_t), fail)
      h->slice_ctx[0].intra4x4_pred_mode = h->intra4x4_pred_mode;
  
      FF_ALLOCZ_OR_GOTO(h->avctx, h->non_zero_count,
                        big_mb_num * sizeof(uint16_t), fail)
      FF_ALLOCZ_OR_GOTO(h->avctx, h->chroma_pred_mode_table,
                        big_mb_num * sizeof(uint8_t), fail)
 -    FF_ALLOCZ_OR_GOTO(h->avctx, h->mvd_table[0],
 -                      16 * row_mb_num * sizeof(uint8_t), fail);
 -    FF_ALLOCZ_OR_GOTO(h->avctx, h->mvd_table[1],
 -                      16 * row_mb_num * sizeof(uint8_t), fail);
 +    FF_ALLOCZ_ARRAY_OR_GOTO(h->avctx, h->mvd_table[0],
 +                      row_mb_num, 16 * sizeof(uint8_t), fail);
 +    FF_ALLOCZ_ARRAY_OR_GOTO(h->avctx, h->mvd_table[1],
 +                      row_mb_num, 16 * sizeof(uint8_t), fail);
      h->slice_ctx[0].mvd_table[0] = h->mvd_table[0];
      h->slice_ctx[0].mvd_table[1] = h->mvd_table[1];
  
@@@ -253,11 -234,7 +253,11 @@@ int ff_h264_slice_context_init(H264Cont
      sl->ref_cache[1][scan8[7]  + 1] =
      sl->ref_cache[1][scan8[13] + 1] = PART_NOT_AVAILABLE;
  
 +    if (sl != h->slice_ctx) {
 +        memset(er, 0, sizeof(*er));
 +    } else
      if (CONFIG_ERROR_RESILIENCE) {
 +
          /* init ER */
          er->avctx          = h->avctx;
          er->decode_mb      = h264_er_decode_mb;
@@@ -307,10 -284,6 +307,10 @@@ static int h264_init_context(AVCodecCon
      int i;
  
      h->avctx                 = avctx;
 +    h->backup_width          = -1;
 +    h->backup_height         = -1;
 +    h->backup_pix_fmt        = AV_PIX_FMT_NONE;
 +    h->cur_chroma_format_idc = -1;
  
      h->picture_structure     = PICT_FRAME;
      h->workaround_bugs       = avctx->workaround_bugs;
      h->poc.prev_poc_msb      = 1 << 16;
      h->recovery_frame        = -1;
      h->frame_recovered       = 0;
 +    h->poc.prev_frame_num    = -1;
 +    h->sei.frame_packing.frame_packing_arrangement_cancel_flag = -1;
 +    h->sei.unregistered.x264_build = -1;
  
      h->next_outputed_poc = INT_MIN;
      for (i = 0; i < MAX_DELAYED_PIC_COUNT; i++)
      if (!h->cur_pic.f)
          return AVERROR(ENOMEM);
  
 +    h->last_pic_for_ec.f = av_frame_alloc();
 +    if (!h->last_pic_for_ec.f)
 +        return AVERROR(ENOMEM);
 +
      for (i = 0; i < h->nb_slice_ctx; i++)
          h->slice_ctx[i].h264 = h;
  
@@@ -362,29 -328,28 +362,29 @@@ static av_cold int h264_decode_end(AVCo
      H264Context *h = avctx->priv_data;
      int i;
  
 +    ff_h264_remove_all_refs(h);
      ff_h264_free_tables(h);
  
      for (i = 0; i < H264_MAX_PICTURE_COUNT; i++) {
          ff_h264_unref_picture(h, &h->DPB[i]);
          av_frame_free(&h->DPB[i].f);
      }
 +    memset(h->delayed_pic, 0, sizeof(h->delayed_pic));
  
      h->cur_pic_ptr = NULL;
  
      av_freep(&h->slice_ctx);
      h->nb_slice_ctx = 0;
  
 -    for (i = 0; i < MAX_SPS_COUNT; i++)
 -        av_buffer_unref(&h->ps.sps_list[i]);
 -
 -    for (i = 0; i < MAX_PPS_COUNT; i++)
 -        av_buffer_unref(&h->ps.pps_list[i]);
 +    ff_h264_sei_uninit(&h->sei);
 +    ff_h264_ps_uninit(&h->ps);
  
      ff_h2645_packet_uninit(&h->pkt);
  
      ff_h264_unref_picture(h, &h->cur_pic);
      av_frame_free(&h->cur_pic.f);
 +    ff_h264_unref_picture(h, &h->last_pic_for_ec);
 +    av_frame_free(&h->last_pic_for_ec.f);
  
      return 0;
  }
@@@ -406,22 -371,18 +406,22 @@@ av_cold int ff_h264_decode_init(AVCodec
          return AVERROR_UNKNOWN;
      }
  
 -    if (avctx->ticks_per_frame == 1)
 -        h->avctx->framerate.num *= 2;
 +    if (avctx->ticks_per_frame == 1) {
 +        if(h->avctx->time_base.den < INT_MAX/2) {
 +            h->avctx->time_base.den *= 2;
 +        } else
 +            h->avctx->time_base.num /= 2;
 +    }
      avctx->ticks_per_frame = 2;
  
      if (avctx->extradata_size > 0 && avctx->extradata) {
 -       ret = ff_h264_decode_extradata(avctx->extradata, avctx->extradata_size,
 -                                      &h->ps, &h->is_avc, &h->nal_length_size,
 -                                      avctx->err_recognition, avctx);
 -       if (ret < 0) {
 -           h264_decode_end(avctx);
 -           return ret;
 -       }
 +        ret = ff_h264_decode_extradata(avctx->extradata, avctx->extradata_size,
 +                                       &h->ps, &h->is_avc, &h->nal_length_size,
 +                                       avctx->err_recognition, avctx);
 +        if (ret < 0) {
 +            h264_decode_end(avctx);
 +            return ret;
 +        }
      }
  
      if (h->ps.sps && h->ps.sps->bitstream_restriction_flag &&
  
      avctx->internal->allocate_progress = 1;
  
 -    if (h->enable_er) {
 +    ff_h264_flush_change(h);
 +
 +    if (h->enable_er < 0 && (avctx->active_thread_type & FF_THREAD_SLICE))
 +        h->enable_er = 0;
 +
 +    if (h->enable_er && (avctx->active_thread_type & FF_THREAD_SLICE)) {
          av_log(avctx, AV_LOG_WARNING,
 -               "Error resilience is enabled. It is unsafe and unsupported and may crash. "
 +               "Error resilience with slice threads is enabled. It is unsafe and unsupported and may crash. "
                 "Use it at your own risk\n");
      }
  
      return 0;
  }
  
 +#if HAVE_THREADS
  static int decode_init_thread_copy(AVCodecContext *avctx)
  {
      H264Context *h = avctx->priv_data;
  
      return 0;
  }
 +#endif
  
  /**
   * Run setup operations that must be run after slice header decoding.
@@@ -480,6 -434,7 +480,6 @@@ static void decode_postinit(H264Contex
      H264Picture *out = h->cur_pic_ptr;
      H264Picture *cur = h->cur_pic_ptr;
      int i, pics, out_of_order, out_idx;
 -    int invalid = 0, cnt = 0;
  
      if (h->next_output_pic)
          return;
           * yet, so we assume the worst for now. */
          // if (setup_finished)
          //    ff_thread_finish_setup(h->avctx);
 -        return;
 +        if (cur->field_poc[0] == INT_MAX && cur->field_poc[1] == INT_MAX)
 +            return;
 +        if (h->avctx->hwaccel || h->missing_fields <=1)
 +            return;
      }
  
 +    cur->mmco_reset = h->mmco_reset;
 +    h->mmco_reset = 0;
 +
      // FIXME do something with unavailable reference frames
  
      /* Sort B-frames into display order */
      if (sps->bitstream_restriction_flag ||
 -        h->avctx->strict_std_compliance >= FF_COMPLIANCE_NORMAL) {
 +        h->avctx->strict_std_compliance >= FF_COMPLIANCE_STRICT) {
          h->avctx->has_b_frames = FFMAX(h->avctx->has_b_frames, sps->num_reorder_frames);
      }
  
 +    for (i = 0; 1; i++) {
 +        if(i == MAX_DELAYED_PIC_COUNT || cur->poc < h->last_pocs[i]){
 +            if(i)
 +                h->last_pocs[i-1] = cur->poc;
 +            break;
 +        } else if(i) {
 +            h->last_pocs[i-1]= h->last_pocs[i];
 +        }
 +    }
 +    out_of_order = MAX_DELAYED_PIC_COUNT - i;
 +    if(   cur->f->pict_type == AV_PICTURE_TYPE_B
 +       || (h->last_pocs[MAX_DELAYED_PIC_COUNT-2] > INT_MIN && h->last_pocs[MAX_DELAYED_PIC_COUNT-1] - h->last_pocs[MAX_DELAYED_PIC_COUNT-2] > 2))
 +        out_of_order = FFMAX(out_of_order, 1);
 +    if (out_of_order == MAX_DELAYED_PIC_COUNT) {
 +        av_log(h->avctx, AV_LOG_VERBOSE, "Invalid POC %d<%d\n", cur->poc, h->last_pocs[0]);
 +        for (i = 1; i < MAX_DELAYED_PIC_COUNT; i++)
 +            h->last_pocs[i] = INT_MIN;
 +        h->last_pocs[0] = cur->poc;
 +        cur->mmco_reset = 1;
 +    } else if(h->avctx->has_b_frames < out_of_order && !sps->bitstream_restriction_flag){
 +        av_log(h->avctx, AV_LOG_INFO, "Increasing reorder buffer to %d\n", out_of_order);
 +        h->avctx->has_b_frames = out_of_order;
 +    }
 +
      pics = 0;
      while (h->delayed_pic[pics])
          pics++;
  
 -    assert(pics <= MAX_DELAYED_PIC_COUNT);
 +    av_assert0(pics <= MAX_DELAYED_PIC_COUNT);
  
      h->delayed_pic[pics++] = cur;
      if (cur->reference == 0)
          cur->reference = DELAYED_PIC_REF;
  
 -    /* Frame reordering. This code takes pictures from coding order and sorts
 -     * them by their incremental POC value into display order. It supports POC
 -     * gaps, MMCO reset codes and random resets.
 -     * A "display group" can start either with a IDR frame (f.key_frame = 1),
 -     * and/or can be closed down with a MMCO reset code. In sequences where
 -     * there is no delay, we can't detect that (since the frame was already
 -     * output to the user), so we also set h->mmco_reset to detect the MMCO
 -     * reset code.
 -     * FIXME: if we detect insufficient delays (as per h->avctx->has_b_frames),
 -     * we increase the delay between input and output. All frames affected by
 -     * the lag (e.g. those that should have been output before another frame
 -     * that we already returned to the user) will be dropped. This is a bug
 -     * that we will fix later. */
 -    for (i = 0; i < MAX_DELAYED_PIC_COUNT; i++) {
 -        cnt     += out->poc < h->last_pocs[i];
 -        invalid += out->poc == INT_MIN;
 -    }
 -    if (!h->mmco_reset && !cur->f->key_frame &&
 -        cnt + invalid == MAX_DELAYED_PIC_COUNT && cnt > 0) {
 -        h->mmco_reset = 2;
 -        if (pics > 1)
 -            h->delayed_pic[pics - 2]->mmco_reset = 2;
 -    }
 -    if (h->mmco_reset || cur->f->key_frame) {
 -        for (i = 0; i < MAX_DELAYED_PIC_COUNT; i++)
 -            h->last_pocs[i] = INT_MIN;
 -        cnt     = 0;
 -        invalid = MAX_DELAYED_PIC_COUNT;
 -    }
      out     = h->delayed_pic[0];
      out_idx = 0;
 -    for (i = 1; i < MAX_DELAYED_PIC_COUNT &&
 -                h->delayed_pic[i] &&
 -                !h->delayed_pic[i - 1]->mmco_reset &&
 -                !h->delayed_pic[i]->f->key_frame;
 +    for (i = 1; h->delayed_pic[i] &&
 +                !h->delayed_pic[i]->f->key_frame &&
 +                !h->delayed_pic[i]->mmco_reset;
           i++)
          if (h->delayed_pic[i]->poc < out->poc) {
              out     = h->delayed_pic[i];
              out_idx = i;
          }
      if (h->avctx->has_b_frames == 0 &&
 -        (h->delayed_pic[0]->f->key_frame || h->mmco_reset))
 +        (h->delayed_pic[0]->f->key_frame || h->delayed_pic[0]->mmco_reset))
          h->next_outputed_poc = INT_MIN;
 -    out_of_order = !out->f->key_frame && !h->mmco_reset &&
 -                   (out->poc < h->next_outputed_poc);
 -
 -    if (sps->bitstream_restriction_flag &&
 -        h->avctx->has_b_frames >= sps->num_reorder_frames) {
 -    } else if (out_of_order && pics - 1 == h->avctx->has_b_frames &&
 -               h->avctx->has_b_frames < MAX_DELAYED_PIC_COUNT) {
 -        if (invalid + cnt < MAX_DELAYED_PIC_COUNT) {
 -            h->avctx->has_b_frames = FFMAX(h->avctx->has_b_frames, cnt);
 -        }
 -    } else if (!h->avctx->has_b_frames &&
 -               ((h->next_outputed_poc != INT_MIN &&
 -                 out->poc > h->next_outputed_poc + 2) ||
 -                cur->f->pict_type == AV_PICTURE_TYPE_B)) {
 -        h->avctx->has_b_frames++;
 -    }
 +    out_of_order = out->poc < h->next_outputed_poc;
  
 -    if (pics > h->avctx->has_b_frames) {
 +    if (out_of_order || pics > h->avctx->has_b_frames) {
          out->reference &= ~DELAYED_PIC_REF;
          for (i = out_idx; h->delayed_pic[i]; i++)
              h->delayed_pic[i] = h->delayed_pic[i + 1];
      }
 -    memmove(h->last_pocs, &h->last_pocs[1],
 -            sizeof(*h->last_pocs) * (MAX_DELAYED_PIC_COUNT - 1));
 -    h->last_pocs[MAX_DELAYED_PIC_COUNT - 1] = cur->poc;
      if (!out_of_order && pics > h->avctx->has_b_frames) {
          h->next_output_pic = out;
 -        if (out->mmco_reset) {
 -            if (out_idx > 0) {
 -                h->next_outputed_poc                    = out->poc;
 -                h->delayed_pic[out_idx - 1]->mmco_reset = out->mmco_reset;
 -            } else {
 -                h->next_outputed_poc = INT_MIN;
 -            }
 -        } else {
 -            if (out_idx == 0 && pics > 1 && h->delayed_pic[0]->f->key_frame) {
 -                h->next_outputed_poc = INT_MIN;
 -            } else {
 -                h->next_outputed_poc = out->poc;
 -            }
 -        }
 -        h->mmco_reset = 0;
 +        if (out_idx == 0 && h->delayed_pic[0] && (h->delayed_pic[0]->f->key_frame || h->delayed_pic[0]->mmco_reset)) {
 +            h->next_outputed_poc = INT_MIN;
 +        } else
 +            h->next_outputed_poc = out->poc;
      } else {
 -        av_log(h->avctx, AV_LOG_DEBUG, "no picture\n");
 +        av_log(h->avctx, AV_LOG_DEBUG, "no picture %s\n", out_of_order ? "ooo" : "");
      }
  
      if (h->next_output_pic) {
   */
  static void idr(H264Context *h)
  {
 +    int i;
      ff_h264_remove_all_refs(h);
      h->poc.prev_frame_num        =
 -    h->poc.prev_frame_num_offset =
 -    h->poc.prev_poc_msb          =
 +    h->poc.prev_frame_num_offset = 0;
 +    h->poc.prev_poc_msb          = 1<<16;
      h->poc.prev_poc_lsb          = 0;
 +    for (i = 0; i < MAX_DELAYED_PIC_COUNT; i++)
 +        h->last_pocs[i] = INT_MIN;
  }
  
  /* forget old pics after a seek */
  void ff_h264_flush_change(H264Context *h)
  {
 -    int i;
 -    for (i = 0; i < MAX_DELAYED_PIC_COUNT; i++)
 -        h->last_pocs[i] = INT_MIN;
 +    int i, j;
 +
      h->next_outputed_poc = INT_MIN;
      h->prev_interlaced_frame = 1;
      idr(h);
 -    if (h->cur_pic_ptr)
 +
 +    h->poc.prev_frame_num = -1;
 +    if (h->cur_pic_ptr) {
          h->cur_pic_ptr->reference = 0;
 +        for (j=i=0; h->delayed_pic[i]; i++)
 +            if (h->delayed_pic[i] != h->cur_pic_ptr)
 +                h->delayed_pic[j++] = h->delayed_pic[i];
 +        h->delayed_pic[j] = NULL;
 +    }
 +    ff_h264_unref_picture(h, &h->last_pic_for_ec);
 +
      h->first_field = 0;
      ff_h264_sei_uninit(&h->sei);
      h->recovery_frame = -1;
      h->frame_recovered = 0;
 +    h->current_slice = 0;
 +    h->mmco_reset = 1;
  }
  
  /* forget old pics after a seek */
@@@ -652,16 -623,10 +652,16 @@@ static void flush_dpb(AVCodecContext *a
      h->context_initialized = 0;
  }
  
 +#if FF_API_CAP_VDPAU
 +static const uint8_t start_code[] = { 0x00, 0x00, 0x01 };
 +#endif
 +
  static int get_last_needed_nal(H264Context *h)
  {
      int nals_needed = 0;
 +    int first_slice = 0;
      int i;
 +    int ret;
  
      for (i = 0; i < h->pkt.nb_nals; i++) {
          H2645NAL *nal = &h->pkt.nals[i];
          case H264_NAL_DPA:
          case H264_NAL_IDR_SLICE:
          case H264_NAL_SLICE:
 -            init_get_bits(&gb, nal->data + 1, (nal->size - 1) * 8);
 -            if (!get_ue_golomb(&gb))
 +            ret = init_get_bits8(&gb, nal->data + 1, (nal->size - 1));
 +            if (ret < 0)
 +                return ret;
 +            if (!get_ue_golomb_long(&gb) ||  // first_mb_in_slice
 +                !first_slice ||
 +                first_slice != nal->type)
                  nals_needed = i;
 +            if (!first_slice)
 +                first_slice = nal->type;
          }
      }
  
      return nals_needed;
  }
  
 +static void debug_green_metadata(const H264SEIGreenMetaData *gm, void *logctx)
 +{
 +    av_log(logctx, AV_LOG_DEBUG, "Green Metadata Info SEI message\n");
 +    av_log(logctx, AV_LOG_DEBUG, "  green_metadata_type: %d\n", gm->green_metadata_type);
 +
 +    if (gm->green_metadata_type == 0) {
 +        av_log(logctx, AV_LOG_DEBUG, "  green_metadata_period_type: %d\n", gm->period_type);
 +
 +        if (gm->period_type == 2)
 +            av_log(logctx, AV_LOG_DEBUG, "  green_metadata_num_seconds: %d\n", gm->num_seconds);
 +        else if (gm->period_type == 3)
 +            av_log(logctx, AV_LOG_DEBUG, "  green_metadata_num_pictures: %d\n", gm->num_pictures);
 +
 +        av_log(logctx, AV_LOG_DEBUG, "  SEI GREEN Complexity Metrics: %f %f %f %f\n",
 +               (float)gm->percent_non_zero_macroblocks/255,
 +               (float)gm->percent_intra_coded_macroblocks/255,
 +               (float)gm->percent_six_tap_filtering/255,
 +               (float)gm->percent_alpha_point_deblocking_instance/255);
 +
 +    } else if (gm->green_metadata_type == 1) {
 +        av_log(logctx, AV_LOG_DEBUG, "  xsd_metric_type: %d\n", gm->xsd_metric_type);
 +
 +        if (gm->xsd_metric_type == 0)
 +            av_log(logctx, AV_LOG_DEBUG, "  xsd_metric_value: %f\n",
 +                   (float)gm->xsd_metric_value/100);
 +    }
 +}
 +
  static int decode_nal_units(H264Context *h, const uint8_t *buf, int buf_size)
  {
      AVCodecContext *const avctx = h->avctx;
      unsigned context_count = 0;
      int nals_needed = 0; ///< number of NALs that need decoding before the next frame thread starts
 +    int idr_cleared=0;
      int i, ret = 0;
  
 +    h->nal_unit_type= 0;
 +
 +    h->max_contexts = h->nb_slice_ctx;
      if (!(avctx->flags2 & AV_CODEC_FLAG2_CHUNKS)) {
          h->current_slice = 0;
          if (!h->first_field)
          ff_h264_sei_uninit(&h->sei);
      }
  
 +    if (h->nal_length_size == 4) {
 +        if (buf_size > 8 && AV_RB32(buf) == 1 && AV_RB32(buf+5) > (unsigned)buf_size) {
 +            h->is_avc = 0;
 +        }else if(buf_size > 3 && AV_RB32(buf) > 1 && AV_RB32(buf) <= (unsigned)buf_size)
 +            h->is_avc = 1;
 +    }
 +
      ret = ff_h2645_packet_split(&h->pkt, buf, buf_size, avctx, h->is_avc,
                                  h->nal_length_size, avctx->codec_id);
      if (ret < 0) {
  
      if (avctx->active_thread_type & FF_THREAD_FRAME)
          nals_needed = get_last_needed_nal(h);
 +    if (nals_needed < 0)
 +        return nals_needed;
  
      for (i = 0; i < h->pkt.nb_nals; i++) {
          H2645NAL *nal = &h->pkt.nals[i];
              nal->ref_idc == 0 && nal->type != H264_NAL_SEI)
              continue;
  
 +again:
          // FIXME these should stop being context-global variables
          h->nal_ref_idc   = nal->ref_idc;
          h->nal_unit_type = nal->type;
          err = 0;
          switch (nal->type) {
          case H264_NAL_IDR_SLICE:
 -            idr(h); // FIXME ensure we don't lose some frames if there is reordering
 +            if ((nal->data[1] & 0xFC) == 0x98) {
 +                av_log(h->avctx, AV_LOG_ERROR, "Invalid inter IDR frame\n");
 +                h->next_outputed_poc = INT_MIN;
 +                ret = -1;
 +                goto end;
 +            }
-             if (nal->type != H264_NAL_IDR_SLICE) {
-                 av_log(h->avctx, AV_LOG_ERROR,
-                        "Invalid mix of idr and non-idr slices\n");
-                 ret = -1;
-                 goto end;
-             }
 +            if(!idr_cleared) {
 +                if (h->current_slice && (avctx->active_thread_type & FF_THREAD_SLICE)) {
 +                    av_log(h, AV_LOG_ERROR, "invalid mixed IDR / non IDR frames cannot be decoded in slice multithreading mode\n");
 +                    ret = AVERROR_INVALIDDATA;
 +                    goto end;
 +                }
 +                idr(h); // FIXME ensure we don't lose some frames if there is reordering
 +            }
 +            idr_cleared = 1;
 +            h->has_recovery_point = 1;
          case H264_NAL_SLICE:
              sl->gb = nal->gb;
  
              if ((err = ff_h264_decode_slice_header(h, sl, nal)))
                  break;
  
 -            if (h->sei.recovery_point.recovery_frame_cnt >= 0 && h->recovery_frame < 0) {
 -                h->recovery_frame = (h->poc.frame_num + h->sei.recovery_point.recovery_frame_cnt) &
 -                                    ((1 << h->ps.sps->log2_max_frame_num) - 1);
 +            if (h->sei.recovery_point.recovery_frame_cnt >= 0) {
 +                const int sei_recovery_frame_cnt = h->sei.recovery_point.recovery_frame_cnt;
 +
 +                if (h->poc.frame_num != sei_recovery_frame_cnt || sl->slice_type_nos != AV_PICTURE_TYPE_I)
 +                    h->valid_recovery_point = 1;
 +
 +                if (   h->recovery_frame < 0
 +                    || av_mod_uintp2(h->recovery_frame - h->poc.frame_num, h->ps.sps->log2_max_frame_num) > sei_recovery_frame_cnt) {
 +                    h->recovery_frame = av_mod_uintp2(h->poc.frame_num + sei_recovery_frame_cnt, h->ps.sps->log2_max_frame_num);
 +
 +                    if (!h->valid_recovery_point)
 +                        h->recovery_frame = h->poc.frame_num;
 +                }
              }
  
 -            h->cur_pic_ptr->f->key_frame |=
 -                (nal->type == H264_NAL_IDR_SLICE) || (h->sei.recovery_point.recovery_frame_cnt >= 0);
 +            h->cur_pic_ptr->f->key_frame |= (nal->type == H264_NAL_IDR_SLICE);
  
 -            if (nal->type == H264_NAL_IDR_SLICE || h->recovery_frame == h->poc.frame_num) {
 +            if (nal->type == H264_NAL_IDR_SLICE ||
 +                (h->recovery_frame == h->poc.frame_num && nal->ref_idc)) {
                  h->recovery_frame         = -1;
                  h->cur_pic_ptr->recovered = 1;
              }
              // "recovered".
              if (nal->type == H264_NAL_IDR_SLICE)
                  h->frame_recovered |= FRAME_RECOVERED_IDR;
 +#if 1
 +            h->cur_pic_ptr->recovered |= h->frame_recovered;
 +#else
              h->cur_pic_ptr->recovered |= !!(h->frame_recovered & FRAME_RECOVERED_IDR);
 +#endif
  
              if (h->current_slice == 1) {
                  if (!(avctx->flags2 & AV_CODEC_FLAG2_CHUNKS))
                      decode_postinit(h, i >= nals_needed);
  
                  if (h->avctx->hwaccel &&
 -                    (ret = h->avctx->hwaccel->start_frame(h->avctx, NULL, 0)) < 0)
 -                    return ret;
 +                    (ret = h->avctx->hwaccel->start_frame(h->avctx, buf, buf_size)) < 0)
 +                    goto end;
 +#if FF_API_CAP_VDPAU
 +                if (CONFIG_H264_VDPAU_DECODER &&
 +                    h->avctx->codec->capabilities & AV_CODEC_CAP_HWACCEL_VDPAU)
 +                    ff_vdpau_h264_picture_start(h);
 +#endif
              }
  
 -            if (sl->redundant_pic_count == 0 &&
 -                (avctx->skip_frame < AVDISCARD_NONREF || nal->ref_idc) &&
 -                (avctx->skip_frame < AVDISCARD_BIDIR  ||
 -                 sl->slice_type_nos != AV_PICTURE_TYPE_B) &&
 -                (avctx->skip_frame < AVDISCARD_NONKEY ||
 -                 h->cur_pic_ptr->f->key_frame) &&
 -                avctx->skip_frame < AVDISCARD_ALL) {
 +            if (sl->redundant_pic_count == 0) {
                  if (avctx->hwaccel) {
 -                    ret = avctx->hwaccel->decode_slice(avctx, nal->raw_data, nal->raw_size);
 +                    ret = avctx->hwaccel->decode_slice(avctx,
 +                                                       nal->raw_data,
 +                                                       nal->raw_size);
                      if (ret < 0)
 -                        return ret;
 +                        goto end;
 +#if FF_API_CAP_VDPAU
 +                } else if (CONFIG_H264_VDPAU_DECODER &&
 +                           h->avctx->codec->capabilities & AV_CODEC_CAP_HWACCEL_VDPAU) {
 +                    ff_vdpau_add_data_chunk(h->cur_pic_ptr->f->data[0],
 +                                            start_code,
 +                                            sizeof(start_code));
 +                    ff_vdpau_add_data_chunk(h->cur_pic_ptr->f->data[0],
 +                                            nal->raw_data,
 +                                            nal->raw_size);
 +#endif
                  } else
                      context_count++;
              }
          case H264_NAL_DPB:
          case H264_NAL_DPC:
              avpriv_request_sample(avctx, "data partitioning");
 -            ret = AVERROR(ENOSYS);
 -            goto end;
              break;
          case H264_NAL_SEI:
              ret = ff_h264_sei_decode(&h->sei, &nal->gb, &h->ps, avctx);
 +            h->has_recovery_point = h->has_recovery_point || h->sei.recovery_point.recovery_frame_cnt != -1;
 +            if (avctx->debug & FF_DEBUG_GREEN_MD)
 +                debug_green_metadata(&h->sei.green_metadata, h->avctx);
 +#if FF_API_AFD
 +FF_DISABLE_DEPRECATION_WARNINGS
 +            h->avctx->dtg_active_format = h->sei.afd.active_format_description;
 +FF_ENABLE_DEPRECATION_WARNINGS
 +#endif /* FF_API_AFD */
              if (ret < 0 && (h->avctx->err_recognition & AV_EF_EXPLODE))
                  goto end;
              break;
 -        case H264_NAL_SPS:
 -            ret = ff_h264_decode_seq_parameter_set(&nal->gb, avctx, &h->ps);
 -            if (ret < 0 && (h->avctx->err_recognition & AV_EF_EXPLODE))
 -                goto end;
 +        case H264_NAL_SPS: {
 +            GetBitContext tmp_gb = nal->gb;
 +            if (ff_h264_decode_seq_parameter_set(&tmp_gb, avctx, &h->ps, 0) >= 0)
 +                break;
 +            av_log(h->avctx, AV_LOG_DEBUG,
 +                   "SPS decoding failure, trying again with the complete NAL\n");
 +            init_get_bits8(&tmp_gb, nal->raw_data + 1, nal->raw_size - 1);
 +            if (ff_h264_decode_seq_parameter_set(&tmp_gb, avctx, &h->ps, 0) >= 0)
 +                break;
 +            ff_h264_decode_seq_parameter_set(&nal->gb, avctx, &h->ps, 1);
              break;
 +        }
          case H264_NAL_PPS:
              ret = ff_h264_decode_picture_parameter_set(&nal->gb, avctx, &h->ps,
                                                         nal->size_bits);
                     nal->type, nal->size_bits);
          }
  
 -        if (context_count == h->nb_slice_ctx) {
 +        if (context_count == h->max_contexts) {
              ret = ff_h264_execute_decode_slices(h, context_count);
              if (ret < 0 && (h->avctx->err_recognition & AV_EF_EXPLODE))
                  goto end;
              context_count = 0;
          }
  
 -        if (err < 0) {
 -            av_log(h->avctx, AV_LOG_ERROR, "decode_slice_header error\n");
 +        if (err < 0 || err == SLICE_SKIPED) {
 +            if (err < 0)
 +                av_log(h->avctx, AV_LOG_ERROR, "decode_slice_header error\n");
              sl->ref_count[0] = sl->ref_count[1] = sl->list_count = 0;
 +        } else if (err == SLICE_SINGLETHREAD) {
 +            if (context_count > 0) {
 +                ret = ff_h264_execute_decode_slices(h, context_count);
 +                if (ret < 0 && (h->avctx->err_recognition & AV_EF_EXPLODE))
 +                    goto end;
 +                context_count = 0;
 +            }
 +            /* Slice could not be decoded in parallel mode, restart. */
 +            sl               = &h->slice_ctx[0];
 +            goto again;
          }
      }
      if (context_count) {
  
      ret = 0;
  end:
 +
 +#if CONFIG_ERROR_RESILIENCE
 +    /*
 +     * FIXME: Error handling code does not seem to support interlaced
 +     * when slices span multiple rows
 +     * The ff_er_add_slice calls don't work right for bottom
 +     * fields; they cause massive erroneous error concealing
 +     * Error marking covers both fields (top and bottom).
 +     * This causes a mismatched s->error_count
 +     * and a bad error table. Further, the error count goes to
 +     * INT_MAX when called for bottom field, because mb_y is
 +     * past end by one (callers fault) and resync_mb_y != 0
 +     * causes problems for the first MB line, too.
 +     */
 +    if (!FIELD_PICTURE(h) && h->current_slice &&
 +        h->ps.sps == (const SPS*)h->ps.sps_list[h->ps.pps->sps_id]->data &&
 +        h->enable_er) {
 +
 +        H264SliceContext *sl = h->slice_ctx;
 +        int use_last_pic = h->last_pic_for_ec.f->buf[0] && !sl->ref_count[0];
 +
 +        ff_h264_set_erpic(&sl->er.cur_pic, h->cur_pic_ptr);
 +
 +        if (use_last_pic) {
 +            ff_h264_set_erpic(&sl->er.last_pic, &h->last_pic_for_ec);
 +            sl->ref_list[0][0].parent = &h->last_pic_for_ec;
 +            memcpy(sl->ref_list[0][0].data, h->last_pic_for_ec.f->data, sizeof(sl->ref_list[0][0].data));
 +            memcpy(sl->ref_list[0][0].linesize, h->last_pic_for_ec.f->linesize, sizeof(sl->ref_list[0][0].linesize));
 +            sl->ref_list[0][0].reference = h->last_pic_for_ec.reference;
 +        } else if (sl->ref_count[0]) {
 +            ff_h264_set_erpic(&sl->er.last_pic, sl->ref_list[0][0].parent);
 +        } else
 +            ff_h264_set_erpic(&sl->er.last_pic, NULL);
 +
 +        if (sl->ref_count[1])
 +            ff_h264_set_erpic(&sl->er.next_pic, sl->ref_list[1][0].parent);
 +
 +        sl->er.ref_count = sl->ref_count[0];
 +
 +        ff_er_frame_end(&sl->er);
 +        if (use_last_pic)
 +            memset(&sl->ref_list[0][0], 0, sizeof(sl->ref_list[0][0]));
 +    }
 +#endif /* CONFIG_ERROR_RESILIENCE */
      /* clean up */
      if (h->cur_pic_ptr && !h->droppable) {
          ff_thread_report_progress(&h->cur_pic_ptr->tf, INT_MAX,
@@@ -1017,62 -820,26 +1011,62 @@@ static int get_consumed_bytes(int pos, 
      return pos;
  }
  
 -static int output_frame(H264Context *h, AVFrame *dst, AVFrame *src)
 +static int output_frame(H264Context *h, AVFrame *dst, H264Picture *srcp)
  {
 +    AVFrame *src = srcp->f;
 +    const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(src->format);
      int i;
      int ret = av_frame_ref(dst, src);
      if (ret < 0)
          return ret;
  
 -    if (!h->ps.sps || !h->ps.sps->crop)
 +    av_dict_set(&dst->metadata, "stereo_mode", ff_h264_sei_stereo_mode(&h->sei.frame_packing), 0);
 +
 +    h->backup_width   = h->avctx->width;
 +    h->backup_height  = h->avctx->height;
 +    h->backup_pix_fmt = h->avctx->pix_fmt;
 +
 +    h->avctx->width   = dst->width;
 +    h->avctx->height  = dst->height;
 +    h->avctx->pix_fmt = dst->format;
 +
 +    if (srcp->sei_recovery_frame_cnt == 0)
 +        dst->key_frame = 1;
 +    if (!srcp->crop)
          return 0;
  
 -    for (i = 0; i < 3; i++) {
 -        int hshift = (i > 0) ? h->chroma_x_shift : 0;
 -        int vshift = (i > 0) ? h->chroma_y_shift : 0;
 -        int off    = ((h->ps.sps->crop_left >> hshift) << h->pixel_shift) +
 -                     (h->ps.sps->crop_top >> vshift) * dst->linesize[i];
 +    for (i = 0; i < desc->nb_components; i++) {
 +        int hshift = (i > 0) ? desc->log2_chroma_w : 0;
 +        int vshift = (i > 0) ? desc->log2_chroma_h : 0;
 +        int off    = ((srcp->crop_left >> hshift) << h->pixel_shift) +
 +                      (srcp->crop_top  >> vshift) * dst->linesize[i];
          dst->data[i] += off;
      }
      return 0;
  }
  
 +static int is_extra(const uint8_t *buf, int buf_size)
 +{
 +    int cnt= buf[5]&0x1f;
 +    const uint8_t *p= buf+6;
 +    while(cnt--){
 +        int nalsize= AV_RB16(p) + 2;
 +        if(nalsize > buf_size - (p-buf) || (p[2] & 0x9F) != 7)
 +            return 0;
 +        p += nalsize;
 +    }
 +    cnt = *(p++);
 +    if(!cnt)
 +        return 0;
 +    while(cnt--){
 +        int nalsize= AV_RB16(p) + 2;
 +        if(nalsize > buf_size - (p-buf) || (p[2] & 0x9F) != 8)
 +            return 0;
 +        p += nalsize;
 +    }
 +    return 1;
 +}
 +
  static int h264_decode_frame(AVCodecContext *avctx, void *data,
                               int *got_frame, AVPacket *avpkt)
  {
      H264Context *h     = avctx->priv_data;
      AVFrame *pict      = data;
      int buf_index      = 0;
 +    H264Picture *out;
 +    int i, out_idx;
      int ret;
 -    const uint8_t *new_extradata;
 -    int new_extradata_size;
  
      h->flags = avctx->flags;
      h->setup_finished = 0;
  
 +    if (h->backup_width != -1) {
 +        avctx->width    = h->backup_width;
 +        h->backup_width = -1;
 +    }
 +    if (h->backup_height != -1) {
 +        avctx->height    = h->backup_height;
 +        h->backup_height = -1;
 +    }
 +    if (h->backup_pix_fmt != AV_PIX_FMT_NONE) {
 +        avctx->pix_fmt    = h->backup_pix_fmt;
 +        h->backup_pix_fmt = AV_PIX_FMT_NONE;
 +    }
 +
 +    ff_h264_unref_picture(h, &h->last_pic_for_ec);
 +
      /* end of stream, output what is still in the buffers */
 -out:
      if (buf_size == 0) {
 -        H264Picture *out;
 -        int i, out_idx;
 + out:
  
          h->cur_pic_ptr = NULL;
 +        h->first_field = 0;
  
          // FIXME factorize this with the output code below
          out     = h->delayed_pic[0];
              h->delayed_pic[i] = h->delayed_pic[i + 1];
  
          if (out) {
 -            ret = output_frame(h, pict, out->f);
 +            out->reference &= ~DELAYED_PIC_REF;
 +            ret = output_frame(h, pict, out);
              if (ret < 0)
                  return ret;
              *got_frame = 1;
  
          return buf_index;
      }
 -
 -    new_extradata_size = 0;
 -    new_extradata = av_packet_get_side_data(avpkt, AV_PKT_DATA_NEW_EXTRADATA,
 -                                            &new_extradata_size);
 -    if (new_extradata_size > 0 && new_extradata) {
 -        ret = ff_h264_decode_extradata(new_extradata, new_extradata_size,
 -                                       &h->ps, &h->is_avc, &h->nal_length_size,
 -                                       avctx->err_recognition, avctx);
 -        if (ret < 0)
 -            return ret;
 +    if (h->is_avc && av_packet_get_side_data(avpkt, AV_PKT_DATA_NEW_EXTRADATA, NULL)) {
 +        int side_size;
 +        uint8_t *side = av_packet_get_side_data(avpkt, AV_PKT_DATA_NEW_EXTRADATA, &side_size);
 +        if (is_extra(side, side_size))
 +            ff_h264_decode_extradata(side, side_size,
 +                                     &h->ps, &h->is_avc, &h->nal_length_size,
 +                                     avctx->err_recognition, avctx);
 +    }
 +    if(h->is_avc && buf_size >= 9 && buf[0]==1 && buf[2]==0 && (buf[4]&0xFC)==0xFC && (buf[5]&0x1F) && buf[8]==0x67){
 +        if (is_extra(buf, buf_size))
 +            return ff_h264_decode_extradata(buf, buf_size,
 +                                            &h->ps, &h->is_avc, &h->nal_length_size,
 +                                            avctx->err_recognition, avctx);
      }
  
      buf_index = decode_nal_units(h, buf, buf_size);
          return AVERROR_INVALIDDATA;
  
      if (!h->cur_pic_ptr && h->nal_unit_type == H264_NAL_END_SEQUENCE) {
 -        buf_size = 0;
 +        av_assert0(buf_index <= buf_size);
          goto out;
      }
  
      if (!(avctx->flags2 & AV_CODEC_FLAG2_CHUNKS) && !h->cur_pic_ptr) {
 -        if (avctx->skip_frame >= AVDISCARD_NONREF)
 -            return 0;
 +        if (avctx->skip_frame >= AVDISCARD_NONREF ||
 +            buf_size >= 4 && !memcmp("Q264", buf, 4))
 +            return buf_size;
          av_log(avctx, AV_LOG_ERROR, "no frame!\n");
          return AVERROR_INVALIDDATA;
      }
          if (avctx->flags2 & AV_CODEC_FLAG2_CHUNKS)
              decode_postinit(h, 1);
  
 -        ff_h264_field_end(h, &h->slice_ctx[0], 0);
 +        if ((ret = ff_h264_field_end(h, &h->slice_ctx[0], 0)) < 0)
 +            return ret;
  
 +        /* Wait for second field. */
          *got_frame = 0;
          if (h->next_output_pic && ((avctx->flags & AV_CODEC_FLAG_OUTPUT_CORRUPT) ||
 +                                   (avctx->flags2 & AV_CODEC_FLAG2_SHOW_ALL) ||
                                     h->next_output_pic->recovered)) {
              if (!h->next_output_pic->recovered)
                  h->next_output_pic->f->flags |= AV_FRAME_FLAG_CORRUPT;
  
 -            ret = output_frame(h, pict, h->next_output_pic->f);
 +            if (!h->avctx->hwaccel &&
 +                 (h->next_output_pic->field_poc[0] == INT_MAX ||
 +                  h->next_output_pic->field_poc[1] == INT_MAX)
 +            ) {
 +                int p;
 +                AVFrame *f = h->next_output_pic->f;
 +                int field = h->next_output_pic->field_poc[0] == INT_MAX;
 +                uint8_t *dst_data[4];
 +                int linesizes[4];
 +                const uint8_t *src_data[4];
 +
 +                av_log(h->avctx, AV_LOG_DEBUG, "Duplicating field %d to fill missing\n", field);
 +
 +                for (p = 0; p<4; p++) {
 +                    dst_data[p] = f->data[p] + (field^1)*f->linesize[p];
 +                    src_data[p] = f->data[p] +  field   *f->linesize[p];
 +                    linesizes[p] = 2*f->linesize[p];
 +                }
 +
 +                av_image_copy(dst_data, linesizes, src_data, linesizes,
 +                              f->format, f->width, f->height>>1);
 +            }
 +
 +            ret = output_frame(h, pict, h->next_output_pic);
              if (ret < 0)
                  return ret;
              *got_frame = 1;
 +            if (CONFIG_MPEGVIDEO) {
 +                ff_print_debug_info2(h->avctx, pict, NULL,
 +                                    h->next_output_pic->mb_type,
 +                                    h->next_output_pic->qscale_table,
 +                                    h->next_output_pic->motion_val,
 +                                    NULL,
 +                                    h->mb_width, h->mb_height, h->mb_stride, 1);
 +            }
          }
      }
  
 -    assert(pict->buf[0] || !*got_frame);
 +    av_assert0(pict->buf[0] || !*got_frame);
 +
 +    ff_h264_unref_picture(h, &h->last_pic_for_ec);
  
      return get_consumed_bytes(buf_index, buf_size);
  }
  #define OFFSET(x) offsetof(H264Context, x)
  #define VD AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_DECODING_PARAM
  static const AVOption h264_options[] = {
 -    { "enable_er", "Enable error resilience on damaged frames (unsafe)", OFFSET(enable_er), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, VD },
 +    {"is_avc", "is avc", offsetof(H264Context, is_avc), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, 0},
 +    {"nal_length_size", "nal_length_size", offsetof(H264Context, nal_length_size), AV_OPT_TYPE_INT, {.i64 = 0}, 0, 4, 0},
 +    { "enable_er", "Enable error resilience on damaged frames (unsafe)", OFFSET(enable_er), AV_OPT_TYPE_BOOL, { .i64 = -1 }, -1, 1, VD },
      { NULL },
  };
  
  static const AVClass h264_class = {
 -    .class_name = "h264",
 +    .class_name = "H264 Decoder",
      .item_name  = av_default_item_name,
      .option     = h264_options,
      .version    = LIBAVUTIL_VERSION_INT,
@@@ -1264,29 -974,3 +1258,29 @@@ AVCodec ff_h264_decoder = 
      .profiles              = NULL_IF_CONFIG_SMALL(ff_h264_profiles),
      .priv_class            = &h264_class,
  };
 +
 +#if CONFIG_H264_VDPAU_DECODER && FF_API_VDPAU
 +static const AVClass h264_vdpau_class = {
 +    .class_name = "H264 VDPAU Decoder",
 +    .item_name  = av_default_item_name,
 +    .option     = h264_options,
 +    .version    = LIBAVUTIL_VERSION_INT,
 +};
 +
 +AVCodec ff_h264_vdpau_decoder = {
 +    .name           = "h264_vdpau",
 +    .long_name      = NULL_IF_CONFIG_SMALL("H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10 (VDPAU acceleration)"),
 +    .type           = AVMEDIA_TYPE_VIDEO,
 +    .id             = AV_CODEC_ID_H264,
 +    .priv_data_size = sizeof(H264Context),
 +    .init           = ff_h264_decode_init,
 +    .close          = h264_decode_end,
 +    .decode         = h264_decode_frame,
 +    .capabilities   = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_DELAY | AV_CODEC_CAP_HWACCEL_VDPAU,
 +    .flush          = flush_dpb,
 +    .pix_fmts       = (const enum AVPixelFormat[]) { AV_PIX_FMT_VDPAU_H264,
 +                                                     AV_PIX_FMT_NONE},
 +    .profiles       = NULL_IF_CONFIG_SMALL(ff_h264_profiles),
 +    .priv_class     = &h264_vdpau_class,
 +};
 +#endif
diff --combined libavcodec/h264dec.h
@@@ -2,20 -2,20 +2,20 @@@
   * H.26L/H.264/AVC/JVT/14496-10/... encoder/decoder
   * Copyright (c) 2003 Michael Niedermayer <michaelni@gmx.at>
   *
 - * 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 "rectangle.h"
  #include "videodsp.h"
  
 -#define H264_MAX_PICTURE_COUNT 32
 +#define H264_MAX_PICTURE_COUNT 36
  
  #define MAX_MMCO_COUNT         66
  
  #define MAX_DELAYED_PIC_COUNT  16
  
 +#define MAX_MBPAIR_SIZE (256*1024) // a tighter bound could be calculated if someone cares about a few bytes
 +
  /* Compiling in interlaced support reduces the speed
   * of progressive decoding by about 2%. */
  #define ALLOW_INTERLACE
  #define MAX_SLICES 32
  
  #ifdef ALLOW_INTERLACE
 -#define MB_MBAFF(h)    h->mb_mbaff
 -#define MB_FIELD(h)    h->mb_field_decoding_flag
 -#define FRAME_MBAFF(h) h->mb_aff_frame
 -#define FIELD_PICTURE(h) (h->picture_structure != PICT_FRAME)
 +#define MB_MBAFF(h)    (h)->mb_mbaff
 +#define MB_FIELD(sl)  (sl)->mb_field_decoding_flag
 +#define FRAME_MBAFF(h) (h)->mb_aff_frame
 +#define FIELD_PICTURE(h) ((h)->picture_structure != PICT_FRAME)
  #define LEFT_MBS 2
  #define LTOP     0
  #define LBOT     1
  #define LEFT(i)  (i)
  #else
  #define MB_MBAFF(h)      0
 -#define MB_FIELD(h)      0
 +#define MB_FIELD(sl)     0
  #define FRAME_MBAFF(h)   0
  #define FIELD_PICTURE(h) 0
  #undef  IS_INTERLACED
  #define FIELD_OR_MBAFF_PICTURE(h) (FRAME_MBAFF(h) || FIELD_PICTURE(h))
  
  #ifndef CABAC
 -#define CABAC(h) h->ps.pps->cabac
 +#define CABAC(h) (h)->ps.pps->cabac
  #endif
  
 -#define CHROMA422(h) (h->ps.sps->chroma_format_idc == 2)
 -#define CHROMA444(h) (h->ps.sps->chroma_format_idc == 3)
 +#define CHROMA(h)    ((h)->ps.sps->chroma_format_idc)
 +#define CHROMA422(h) ((h)->ps.sps->chroma_format_idc == 2)
 +#define CHROMA444(h) ((h)->ps.sps->chroma_format_idc == 3)
  
  #define MB_TYPE_REF0       MB_TYPE_ACPRED // dirty but it fits in 16 bit
  #define MB_TYPE_8x8DCT     0x01000000
@@@ -154,19 -151,13 +154,19 @@@ typedef struct H264Picture 
      int pic_id;             /**< pic_num (short -> no wrap version of pic_num,
                                   pic_num & max_pic_num; long -> long_pic_num) */
      int long_ref;           ///< 1->long term reference 0->short term reference
 -    int ref_poc[2][2][32];  ///< POCs of the frames used as reference (FIXME need per slice)
 +    int ref_poc[2][2][32];  ///< POCs of the frames/fields used as reference (FIXME need per slice)
      int ref_count[2][2];    ///< number of entries in ref_poc         (FIXME need per slice)
      int mbaff;              ///< 1 -> MBAFF frame 0-> not MBAFF
      int field_picture;      ///< whether or not picture was encoded in separate fields
  
      int reference;
      int recovered;          ///< picture at IDR or recovery point + recovery count
 +    int invalid_gap;
 +    int sei_recovery_frame_cnt;
 +
 +    int crop;
 +    int crop_left;
 +    int crop_top;
  } H264Picture;
  
  typedef struct H264Ref {
@@@ -277,7 -268,7 +277,7 @@@ typedef struct H264SliceContext 
                                           *   according to picture reordering in slice header */
      struct {
          uint8_t op;
 -        uint8_t val;
 +        uint32_t val;
      } ref_modifications[2][32];
      int nb_ref_modifications[2];
  
@@@ -351,7 -342,6 +351,7 @@@ typedef struct H264Context 
      H264Picture DPB[H264_MAX_PICTURE_COUNT];
      H264Picture *cur_pic_ptr;
      H264Picture cur_pic;
 +    H264Picture last_pic_for_ec;
  
      H264SliceContext *slice_ctx;
      int            nb_slice_ctx;
      int width, height;
      int chroma_x_shift, chroma_y_shift;
  
 +    /**
 +     * Backup frame properties: needed, because they can be different
 +     * between returned frame and last decoded frame.
 +     **/
 +    int backup_width;
 +    int backup_height;
 +    enum AVPixelFormat backup_pix_fmt;
 +
      int droppable;
      int coded_picture_number;
  
       */
      int postpone_filter;
  
+     /*
+      * Set to 1 when the current picture is IDR, 0 otherwise.
+      */
+     int picture_idr;
      int8_t(*intra4x4_pred_mode);
      H264PredContext hpc;
  
      uint8_t field_scan[16];
      uint8_t field_scan8x8[64];
      uint8_t field_scan8x8_cavlc[64];
 -    const uint8_t *zigzag_scan_q0;
 -    const uint8_t *zigzag_scan8x8_q0;
 -    const uint8_t *zigzag_scan8x8_cavlc_q0;
 -    const uint8_t *field_scan_q0;
 -    const uint8_t *field_scan8x8_q0;
 -    const uint8_t *field_scan8x8_cavlc_q0;
 +    uint8_t zigzag_scan_q0[16];
 +    uint8_t zigzag_scan8x8_q0[64];
 +    uint8_t zigzag_scan8x8_cavlc_q0[64];
 +    uint8_t field_scan_q0[16];
 +    uint8_t field_scan8x8_q0[64];
 +    uint8_t field_scan8x8_cavlc_q0[64];
  
      int mb_y;
      int mb_height, mb_width;
  
      H264POCContext poc;
  
 +    H264Ref default_ref[2];
      H264Picture *short_ref[32];
      H264Picture *long_ref[32];
      H264Picture *delayed_pic[MAX_DELAYED_PIC_COUNT + 2]; // FIXME size?
       */
      int current_slice;
  
 +    /**
 +     * Max number of threads / contexts.
 +     * This is equal to AVCodecContext.thread_count unless
 +     * multithreaded decoding is impossible, in which case it is
 +     * reduced to 1.
 +     */
 +    int max_contexts;
 +
 +    /**
 +     *  1 if the single thread fallback warning has already been
 +     *  displayed, 0 otherwise.
 +     */
 +    int single_decode_warning;
 +
      /** @} */
  
      /**
      int prev_interlaced_frame;
  
      /**
 +     * Are the SEI recovery points looking valid.
 +     */
 +    int valid_recovery_point;
 +
 +    /**
       * recovery_frame is the frame_num at which the next frame should
       * be fully constructed.
       *
  
      int frame_recovered;    ///< Initial frame has been completely recovered
  
 +    int has_recovery_point;
 +
 +    int missing_fields;
 +
      /* for frame threading, this is set to 1
       * after finish_setup() has been called, so we cannot modify
       * some context properties (which are supposed to stay constant between
       * slices) anymore */
      int setup_finished;
  
 +    int cur_chroma_format_idc;
 +    int cur_bit_depth_luma;
 +    int16_t slice_row[MAX_SLICES]; ///< to detect when MAX_SLICES is too low
 +
      int enable_er;
  
      H264SEIContext sei;
@@@ -575,7 -534,7 +580,7 @@@ int ff_h264_get_slice_type(const H264Sl
  int ff_h264_alloc_tables(H264Context *h);
  
  int ff_h264_decode_ref_pic_list_reordering(H264SliceContext *sl, void *logctx);
 -int ff_h264_build_ref_list(const H264Context *h, H264SliceContext *sl);
 +int ff_h264_build_ref_list(H264Context *h, H264SliceContext *sl);
  void ff_h264_remove_all_refs(H264Context *h);
  
  /**
@@@ -604,6 -563,8 +609,6 @@@ int ff_h264_decode_mb_cabac(const H264C
  
  void ff_h264_init_cabac_states(const H264Context *h, H264SliceContext *sl);
  
 -void ff_h264_init_dequant_tables(H264Context *h);
 -
  void ff_h264_direct_dist_scale_factor(const H264Context *const h, H264SliceContext *sl);
  void ff_h264_direct_ref_list_init(const H264Context *const h, H264SliceContext *sl);
  void ff_h264_pred_direct_motion(const H264Context *const h, H264SliceContext *sl,
@@@ -666,7 -627,7 +671,7 @@@ static const uint8_t scan8[16 * 3 + 3] 
      0 +  0 * 8, 0 +  5 * 8, 0 + 10 * 8
  };
  
 -static av_always_inline uint32_t pack16to32(int a, int b)
 +static av_always_inline uint32_t pack16to32(unsigned a, unsigned b)
  {
  #if HAVE_BIGENDIAN
      return (b & 0xFFFF) + (a << 16);
  #endif
  }
  
 -static av_always_inline uint16_t pack8to16(int a, int b)
 +static av_always_inline uint16_t pack8to16(unsigned a, unsigned b)
  {
  #if HAVE_BIGENDIAN
      return (b & 0xFF) + (a << 8);
@@@ -822,16 -783,6 +827,16 @@@ static av_always_inline int get_dct8x8_
                    0x0001000100010001ULL));
  }
  
 +static inline int find_start_code(const uint8_t *buf, int buf_size,
 +                           int buf_index, int next_avc)
 +{
 +    uint32_t state = -1;
 +
 +    buf_index = avpriv_find_start_code(buf + buf_index, buf + next_avc + 1, &state) - buf - 1;
 +
 +    return FFMIN(buf_index, buf_size);
 +}
 +
  int ff_h264_field_end(H264Context *h, H264SliceContext *sl, int in_setup);
  
  int ff_h264_ref_picture(H264Context *h, H264Picture *dst, H264Picture *src);
@@@ -843,9 -794,6 +848,9 @@@ void ff_h264_draw_horiz_band(const H264
  
  int ff_h264_decode_slice_header(H264Context *h, H264SliceContext *sl,
                                  const H2645NAL *nal);
 +#define SLICE_SINGLETHREAD 1
 +#define SLICE_SKIPED 2
 +
  int ff_h264_execute_decode_slices(H264Context *h, unsigned context_count);
  int ff_h264_update_thread_context(AVCodecContext *dst,
                                    const AVCodecContext *src);
@@@ -854,6 -802,4 +859,6 @@@ void ff_h264_flush_change(H264Context *
  
  void ff_h264_free_tables(H264Context *h);
  
 +void ff_h264_set_erpic(ERPicture *dst, H264Picture *src);
 +
  #endif /* AVCODEC_H264DEC_H */