Merge commit 'b16e9b9ac9db449cae2242767dd3c3fc309357c4'
authorClément Bœsch <clement@stupeflix.com>
Wed, 29 Jun 2016 13:03:12 +0000 (15:03 +0200)
committerClément Bœsch <clement@stupeflix.com>
Wed, 29 Jun 2016 13:03:12 +0000 (15:03 +0200)
* commit 'b16e9b9ac9db449cae2242767dd3c3fc309357c4':
  h264: move initialising the explicit pred weight table for MBAFF

Merged-by: Clément Bœsch <clement@stupeflix.com>
1  2 
libavcodec/h264_parse.c
libavcodec/h264_refs.c

diff --combined libavcodec/h264_parse.c
@@@ -1,18 -1,18 +1,18 @@@
  /*
 - * 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
   */
  
@@@ -24,9 -24,9 +24,9 @@@
  
  int ff_h264_pred_weight_table(GetBitContext *gb, const SPS *sps,
                                const int *ref_count, int slice_type_nos,
 -                              H264PredWeightTable *pwt)
 +                              H264PredWeightTable *pwt, void *logctx)
  {
-     int list, i;
+     int list, i, j;
      int luma_def, chroma_def;
  
      pwt->use_weight             = 0;
      pwt->luma_log2_weight_denom = get_ue_golomb(gb);
      if (sps->chroma_format_idc)
          pwt->chroma_log2_weight_denom = get_ue_golomb(gb);
 +
 +    if (pwt->luma_log2_weight_denom > 7U) {
 +        av_log(logctx, AV_LOG_ERROR, "luma_log2_weight_denom %d is out of range\n", pwt->luma_log2_weight_denom);
 +        pwt->luma_log2_weight_denom = 0;
 +    }
 +    if (pwt->chroma_log2_weight_denom > 7U) {
 +        av_log(logctx, AV_LOG_ERROR, "chroma_log2_weight_denom %d is out of range\n", pwt->chroma_log2_weight_denom);
 +        pwt->chroma_log2_weight_denom = 0;
 +    }
 +
      luma_def   = 1 << pwt->luma_log2_weight_denom;
      chroma_def = 1 << pwt->chroma_log2_weight_denom;
  
                      }
                  }
              }
+             // for MBAFF
+             pwt->luma_weight[16 + 2 * i][list][0] = pwt->luma_weight[16 + 2 * i + 1][list][0] = pwt->luma_weight[i][list][0];
+             pwt->luma_weight[16 + 2 * i][list][1] = pwt->luma_weight[16 + 2 * i + 1][list][1] = pwt->luma_weight[i][list][1];
+             for (j = 0; j < 2; j++) {
+                 pwt->chroma_weight[16 + 2 * i][list][j][0] = pwt->chroma_weight[16 + 2 * i + 1][list][j][0] = pwt->chroma_weight[i][list][j][0];
+                 pwt->chroma_weight[16 + 2 * i][list][j][1] = pwt->chroma_weight[16 + 2 * i + 1][list][j][1] = pwt->chroma_weight[i][list][j][1];
+             }
          }
          if (slice_type_nos != AV_PICTURE_TYPE_B)
              break;
@@@ -116,7 -114,7 +124,7 @@@ int ff_h264_check_intra4x4_pred_mode(in
              int status = top[pred_mode_cache[scan8[0] + i]];
              if (status < 0) {
                  av_log(logctx, AV_LOG_ERROR,
 -                       "top block unavailable for requested intra4x4 mode %d\n",
 +                       "top block unavailable for requested intra mode %d\n",
                         status);
                  return AVERROR_INVALIDDATA;
              } else if (status) {
@@@ -172,17 -170,17 +180,17 @@@ int ff_h264_check_intra_pred_mode(void 
  
      if ((left_samples_available & 0x8080) != 0x8080) {
          mode = left[mode];
 +        if (mode < 0) {
 +            av_log(logctx, AV_LOG_ERROR,
 +                   "left block unavailable for requested intra mode\n");
 +            return AVERROR_INVALIDDATA;
 +        }
          if (is_chroma && (left_samples_available & 0x8080)) {
              // mad cow disease mode, aka MBAFF + constrained_intra_pred
              mode = ALZHEIMER_DC_L0T_PRED8x8 +
                     (!(left_samples_available & 0x8000)) +
                     2 * (mode == DC_128_PRED8x8);
          }
 -        if (mode < 0) {
 -            av_log(logctx, AV_LOG_ERROR,
 -                   "left block unavailable for requested intra mode\n");
 -            return AVERROR_INVALIDDATA;
 -        }
      }
  
      return mode;
  
  int ff_h264_parse_ref_count(int *plist_count, int ref_count[2],
                              GetBitContext *gb, const PPS *pps,
 -                            int slice_type_nos, int picture_structure)
 +                            int slice_type_nos, int picture_structure, void *logctx)
  {
      int list_count;
 -    int num_ref_idx_active_override_flag, max_refs;
 +    int num_ref_idx_active_override_flag;
  
      // set defaults, might be overridden a few lines later
      ref_count[0] = pps->ref_count[0];
      ref_count[1] = pps->ref_count[1];
  
      if (slice_type_nos != AV_PICTURE_TYPE_I) {
 +        unsigned max[2];
 +        max[0] = max[1] = picture_structure == PICT_FRAME ? 15 : 31;
 +
          num_ref_idx_active_override_flag = get_bits1(gb);
  
          if (num_ref_idx_active_override_flag) {
              ref_count[0] = get_ue_golomb(gb) + 1;
 -            if (ref_count[0] < 1)
 -                goto fail;
              if (slice_type_nos == AV_PICTURE_TYPE_B) {
                  ref_count[1] = get_ue_golomb(gb) + 1;
 -                if (ref_count[1] < 1)
 -                    goto fail;
 -            }
 +            } else
 +                // full range is spec-ok in this case, even for frames
 +                ref_count[1] = 1;
 +        }
 +
 +        if (ref_count[0] - 1 > max[0] || ref_count[1] - 1 > max[1]) {
 +            av_log(logctx, AV_LOG_ERROR, "reference overflow %u > %u or %u > %u\n",
 +                   ref_count[0] - 1, max[0], ref_count[1] - 1, max[1]);
 +            ref_count[0] = ref_count[1] = 0;
 +            *plist_count = 0;
 +            goto fail;
          }
  
          if (slice_type_nos == AV_PICTURE_TYPE_B)
          ref_count[0] = ref_count[1] = 0;
      }
  
 -    max_refs = picture_structure == PICT_FRAME ? 16 : 32;
 -
 -    if (ref_count[0] > max_refs || ref_count[1] > max_refs)
 -        goto fail;
 -
      *plist_count = list_count;
  
      return 0;
@@@ -328,16 -322,14 +336,16 @@@ static int decode_extradata_ps(const ui
      int i, ret = 0;
  
      ret = ff_h2645_packet_split(&pkt, data, size, logctx, is_avc, 2, AV_CODEC_ID_H264);
 -    if (ret < 0)
 +    if (ret < 0) {
 +        ret = 0;
          goto fail;
 +    }
  
      for (i = 0; i < pkt.nb_nals; i++) {
          H2645NAL *nal = &pkt.nals[i];
          switch (nal->type) {
          case NAL_SPS:
 -            ret = ff_h264_decode_seq_parameter_set(&nal->gb, logctx, ps);
 +            ret = ff_h264_decode_seq_parameter_set(&nal->gb, logctx, ps, 0);
              if (ret < 0)
                  goto fail;
              break;
@@@ -415,9 -407,6 +423,9 @@@ int ff_h264_decode_extradata(const uint
  {
      int ret;
  
 +    if (!data || size <= 0)
 +        return -1;
 +
      if (data[0] == 1) {
          int i, cnt, nalsize;
          const uint8_t *p = data;
          p  += 6;
          for (i = 0; i < cnt; i++) {
              nalsize = AV_RB16(p) + 2;
 -            if (p - data + nalsize > size)
 +            if (nalsize > size - (p - data))
                  return AVERROR_INVALIDDATA;
              ret = decode_extradata_ps_mp4(p, nalsize, ps, err_recognition, logctx);
              if (ret < 0) {
          cnt = *(p++); // Number of pps
          for (i = 0; i < cnt; i++) {
              nalsize = AV_RB16(p) + 2;
 -            if (p - data + nalsize > size)
 +            if (nalsize > size - (p - data))
                  return AVERROR_INVALIDDATA;
              ret = decode_extradata_ps_mp4(p, nalsize, ps, err_recognition, logctx);
              if (ret < 0) {
          if (ret < 0)
              return ret;
      }
 -    return 0;
 +    return size;
  }
  
  /**
diff --combined libavcodec/h264_refs.c
@@@ -2,20 -2,20 +2,20 @@@
   * H.26L/H.264/AVC/JVT/14496-10/... reference picture handling
   * 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
   */
  
@@@ -27,7 -27,6 +27,7 @@@
  
  #include <inttypes.h>
  
 +#include "libavutil/avassert.h"
  #include "internal.h"
  #include "avcodec.h"
  #include "h264.h"
@@@ -80,18 -79,16 +80,18 @@@ static int build_def_list(H264Ref *def
      int  i[2] = { 0 };
      int index = 0;
  
 -    while ((i[0] < len || i[1] < len) && index < def_len) {
 +    while (i[0] < len || i[1] < len) {
          while (i[0] < len && !(in[i[0]] && (in[i[0]]->reference & sel)))
              i[0]++;
          while (i[1] < len && !(in[i[1]] && (in[i[1]]->reference & (sel ^ 3))))
              i[1]++;
 -        if (i[0] < len && index < def_len) {
 +        if (i[0] < len) {
 +            av_assert0(index < def_len);
              in[i[0]]->pic_id = is_long ? i[0] : in[i[0]]->frame_num;
              split_field_copy(&def[index++], in[i[0]++], sel, 1);
          }
 -        if (i[1] < len && index < def_len) {
 +        if (i[1] < len) {
 +            av_assert0(index < def_len);
              in[i[1]]->pic_id = is_long ? i[1] : in[i[1]]->frame_num;
              split_field_copy(&def[index++], in[i[1]++], sel ^ 3, 0);
          }
@@@ -123,18 -120,9 +123,18 @@@ static int add_sorted(H264Picture **sor
      return out_i;
  }
  
 -static void h264_initialise_ref_list(const H264Context *h, H264SliceContext *sl)
 +static int mismatches_ref(const H264Context *h, const H264Picture *pic)
 +{
 +    const AVFrame *f = pic->f;
 +    return (h->cur_pic_ptr->f->width  != f->width ||
 +            h->cur_pic_ptr->f->height != f->height ||
 +            h->cur_pic_ptr->f->format != f->format);
 +}
 +
 +static void h264_initialise_ref_list(H264Context *h, H264SliceContext *sl)
  {
      int i, len;
 +    int j;
  
      if (sl->slice_type_nos == AV_PICTURE_TYPE_B) {
          H264Picture *sorted[32];
          for (list = 0; list < 2; list++) {
              len  = add_sorted(sorted,       h->short_ref, h->short_ref_count, cur_poc, 1 ^ list);
              len += add_sorted(sorted + len, h->short_ref, h->short_ref_count, cur_poc, 0 ^ list);
 -            assert(len <= 32);
 +            av_assert0(len <= 32);
  
              len  = build_def_list(sl->ref_list[list], FF_ARRAY_ELEMS(sl->ref_list[0]),
                                    sorted, len, 0, h->picture_structure);
              len += build_def_list(sl->ref_list[list] + len,
                                    FF_ARRAY_ELEMS(sl->ref_list[0]) - len,
                                    h->long_ref, 16, 1, h->picture_structure);
 +            av_assert0(len <= 32);
  
              if (len < sl->ref_count[list])
                  memset(&sl->ref_list[list][len], 0, sizeof(H264Ref) * (sl->ref_count[list] - len));
          len += build_def_list(sl->ref_list[0] + len,
                                FF_ARRAY_ELEMS(sl->ref_list[0]) - len,
                                h-> long_ref, 16, 1, h->picture_structure);
 +        av_assert0(len <= 32);
  
          if (len < sl->ref_count[0])
              memset(&sl->ref_list[0][len], 0, sizeof(H264Ref) * (sl->ref_count[0] - len));
      }
 +#ifdef TRACE
 +    for (i = 0; i < sl->ref_count[0]; i++) {
 +        ff_tlog(h->avctx, "List0: %s fn:%d 0x%p\n",
 +                (sl->ref_list[0][i].parent ? (sl->ref_list[0][i].parent->long_ref ? "LT" : "ST") : "??"),
 +                sl->ref_list[0][i].pic_id,
 +                sl->ref_list[0][i].data[0]);
 +    }
 +    if (sl->slice_type_nos == AV_PICTURE_TYPE_B) {
 +        for (i = 0; i < sl->ref_count[1]; i++) {
 +            ff_tlog(h->avctx, "List1: %s fn:%d 0x%p\n",
 +                    (sl->ref_list[1][i].parent ? (sl->ref_list[1][i].parent->long_ref ? "LT" : "ST") : "??"),
 +                    sl->ref_list[1][i].pic_id,
 +                    sl->ref_list[1][i].data[0]);
 +        }
 +    }
 +#endif
 +
 +    for (j = 0; j<1+(sl->slice_type_nos == AV_PICTURE_TYPE_B); j++) {
 +        for (i = 0; i < sl->ref_count[j]; i++) {
 +            if (sl->ref_list[j][i].parent) {
 +                if (mismatches_ref(h, sl->ref_list[j][i].parent)) {
 +                    av_log(h->avctx, AV_LOG_ERROR, "Discarding mismatching reference\n");
 +                    memset(&sl->ref_list[j][i], 0, sizeof(sl->ref_list[j][i]));
 +                }
 +            }
 +        }
 +    }
 +    for (i = 0; i < sl->list_count; i++)
 +        h->default_ref[i] = sl->ref_list[i][0];
  }
  
  /**
@@@ -270,7 -227,7 +270,7 @@@ static int pic_num_extract(const H264Co
      return pic_num;
  }
  
 -int ff_h264_decode_ref_pic_list_reordering(const H264Context *h, H264SliceContext *sl)
 +int ff_h264_decode_ref_pic_list_reordering(H264Context *h, H264SliceContext *sl)
  {
      int list, index, pic_structure;
  
                  switch (modification_of_pic_nums_idc) {
                  case 0:
                  case 1: {
 -                    const unsigned int abs_diff_pic_num = get_ue_golomb(&sl->gb) + 1;
 +                    const unsigned int abs_diff_pic_num = get_ue_golomb_long(&sl->gb) + 1;
                      int frame_num;
  
                      if (abs_diff_pic_num > h->max_pic_num) {
  
                      long_idx = pic_num_extract(h, pic_id, &pic_structure);
  
 -                    if (long_idx > 31) {
 +                    if (long_idx > 31U) {
                          av_log(h->avctx, AV_LOG_ERROR,
                                 "long_term_pic_idx overflow\n");
                          return AVERROR_INVALIDDATA;
                      }
                      ref = h->long_ref[long_idx];
                      assert(!(ref && !ref->reference));
 -                    if (ref && (ref->reference & pic_structure)) {
 +                    if (ref && (ref->reference & pic_structure) && !mismatches_ref(h, ref)) {
                          ref->pic_id = pic_id;
                          assert(ref->long_ref);
                          i = 0;
      }
      for (list = 0; list < sl->list_count; list++) {
          for (index = 0; index < sl->ref_count[list]; index++) {
 -            if (!sl->ref_list[list][index].parent) {
 -                av_log(h->avctx, AV_LOG_ERROR, "Missing reference picture\n");
 -                if (index == 0 || h->avctx->err_recognition & AV_EF_EXPLODE)
 -                    return AVERROR_INVALIDDATA;
 +            if (   !sl->ref_list[list][index].parent
 +                || (!FIELD_PICTURE(h) && (sl->ref_list[list][index].reference&3) != 3)) {
 +                int i;
 +                av_log(h->avctx, AV_LOG_ERROR, "Missing reference picture, default is %d\n", h->default_ref[list].poc);
 +                for (i = 0; i < FF_ARRAY_ELEMS(h->last_pocs); i++)
 +                    h->last_pocs[i] = INT_MIN;
 +                if (h->default_ref[list].parent
 +                    && !(!FIELD_PICTURE(h) && (h->default_ref[list].reference&3) != 3))
 +                    sl->ref_list[list][index] = h->default_ref[list];
                  else
 -                    sl->ref_list[list][index] = sl->ref_list[list][index - 1];
 +                    return -1;
              }
 +            av_assert0(av_buffer_get_ref_count(sl->ref_list[list][index].parent->f->buf[0]) > 0);
          }
      }
  
@@@ -422,13 -373,6 +422,6 @@@ void ff_h264_fill_mbaff_ref_list(H264Sl
                  field[1].data[j] += frame->parent->f->linesize[j];
              field[1].reference = PICT_BOTTOM_FIELD;
              field[1].poc       = field[1].parent->field_poc[1];
-             sl->pwt.luma_weight[16 + 2 * i][list][0] = sl->pwt.luma_weight[16 + 2 * i + 1][list][0] = sl->pwt.luma_weight[i][list][0];
-             sl->pwt.luma_weight[16 + 2 * i][list][1] = sl->pwt.luma_weight[16 + 2 * i + 1][list][1] = sl->pwt.luma_weight[i][list][1];
-             for (j = 0; j < 2; j++) {
-                 sl->pwt.chroma_weight[16 + 2 * i][list][j][0] = sl->pwt.chroma_weight[16 + 2 * i + 1][list][j][0] = sl->pwt.chroma_weight[i][list][j][0];
-                 sl->pwt.chroma_weight[16 + 2 * i][list][j][1] = sl->pwt.chroma_weight[16 + 2 * i + 1][list][j][1] = sl->pwt.chroma_weight[i][list][j][1];
-             }
          }
      }
  }
@@@ -549,24 -493,11 +542,24 @@@ void ff_h264_remove_all_refs(H264Contex
      }
      assert(h->long_ref_count == 0);
  
 +    if (h->short_ref_count && !h->last_pic_for_ec.f->data[0]) {
 +        ff_h264_unref_picture(h, &h->last_pic_for_ec);
 +        if (h->short_ref[0]->f->buf[0])
 +            ff_h264_ref_picture(h, &h->last_pic_for_ec, h->short_ref[0]);
 +    }
 +
      for (i = 0; i < h->short_ref_count; i++) {
          unreference_pic(h, h->short_ref[i], 0);
          h->short_ref[i] = NULL;
      }
      h->short_ref_count = 0;
 +
 +    memset(h->default_ref, 0, sizeof(h->default_ref));
 +    for (i = 0; i < h->nb_slice_ctx; i++) {
 +        H264SliceContext *sl = &h->slice_ctx[i];
 +        sl->list_count = sl->ref_count[0] = sl->ref_count[1] = 0;
 +        memset(sl->ref_list, 0, sizeof(sl->ref_list));
 +    }
  }
  
  static int check_opcodes(MMCO *mmco1, MMCO *mmco2, int n_mmcos)
      int i;
  
      for (i = 0; i < n_mmcos; i++) {
 -        if (mmco1[i].opcode != mmco2[i].opcode)
 +        if (mmco1[i].opcode != mmco2[i].opcode) {
 +            av_log(NULL, AV_LOG_ERROR, "MMCO opcode [%d, %d] at %d mismatches between slices\n",
 +                   mmco1[i].opcode, mmco2[i].opcode, i);
              return -1;
 +        }
      }
  
      return 0;
@@@ -589,8 -517,10 +582,8 @@@ int ff_generate_sliding_window_mmcos(H2
      MMCO mmco_temp[MAX_MMCO_COUNT], *mmco = first_slice ? h->mmco : mmco_temp;
      int mmco_index = 0, i = 0;
  
 -    assert(h->long_ref_count + h->short_ref_count <= h->ps.sps->ref_frame_count);
 -
      if (h->short_ref_count &&
 -        h->long_ref_count + h->short_ref_count == h->ps.sps->ref_frame_count &&
 +        h->long_ref_count + h->short_ref_count >= h->ps.sps->ref_frame_count &&
          !(FIELD_PICTURE(h) && !h->first_field && h->cur_pic_ptr->reference)) {
          mmco[0].opcode        = MMCO_SHORT2UNUSED;
          mmco[0].short_pic_num = h->short_ref[h->short_ref_count - 1]->frame_num;
                 (mmco_index != h->mmco_index ||
                  (i = check_opcodes(h->mmco, mmco_temp, mmco_index)))) {
          av_log(h->avctx, AV_LOG_ERROR,
 -               "Inconsistent MMCO state between slices [%d, %d, %d]\n",
 -               mmco_index, h->mmco_index, i);
 +               "Inconsistent MMCO state between slices [%d, %d]\n",
 +               mmco_index, h->mmco_index);
          return AVERROR_INVALIDDATA;
      }
      return 0;
  int ff_h264_execute_ref_pic_marking(H264Context *h, MMCO *mmco, int mmco_count)
  {
      int i, av_uninit(j);
 +    int pps_ref_count[2] = {0};
      int current_ref_assigned = 0, err = 0;
      H264Picture *av_uninit(pic);
  
                  if (mmco[i].opcode != MMCO_SHORT2LONG ||
                      !h->long_ref[mmco[i].long_arg]    ||
                      h->long_ref[mmco[i].long_arg]->frame_num != frame_num) {
 -                    av_log(h->avctx, AV_LOG_ERROR, "mmco: unref short failure\n");
 +                    av_log(h->avctx, h->short_ref_count ? AV_LOG_ERROR : AV_LOG_DEBUG, "mmco: unref short failure\n");
                      err = AVERROR_INVALIDDATA;
                  }
                  continue;
                       * Report the problem and keep the pair where it is,
                       * and mark this field valid.
                       */
 -            if (h->short_ref[0] == h->cur_pic_ptr)
 +            if (h->short_ref[0] == h->cur_pic_ptr) {
 +                av_log(h->avctx, AV_LOG_ERROR, "mmco: cannot assign current picture to short and long at the same time\n");
                  remove_short_at_index(h, 0);
 +            }
  
              /* make sure the current picture is not already assigned as a long ref */
              if (h->cur_pic_ptr->long_ref) {
                  for (j = 0; j < FF_ARRAY_ELEMS(h->long_ref); j++) {
 -                    if (h->long_ref[j] == h->cur_pic_ptr)
 +                    if (h->long_ref[j] == h->cur_pic_ptr) {
 +                        if (j != mmco[i].long_arg)
 +                            av_log(h->avctx, AV_LOG_ERROR, "mmco: cannot assign current picture to 2 long term references\n");
                          remove_long(h, j, 0);
 +                    }
                  }
              }
  
 -
              if (h->long_ref[mmco[i].long_arg] != h->cur_pic_ptr) {
 +                av_assert0(!h->cur_pic_ptr->long_ref);
                  remove_long(h, mmco[i].long_arg, 0);
  
                  h->long_ref[mmco[i].long_arg]           = h->cur_pic_ptr;
              h->poc.frame_num = h->cur_pic_ptr->frame_num = 0;
              h->mmco_reset = 1;
              h->cur_pic_ptr->mmco_reset = 1;
 +            for (j = 0; j < MAX_DELAYED_PIC_COUNT; j++)
 +                h->last_pocs[j] = INT_MIN;
              break;
          default: assert(0);
          }
           */
          if (h->short_ref_count && h->short_ref[0] == h->cur_pic_ptr) {
              /* Just mark the second field valid */
 -            h->cur_pic_ptr->reference = PICT_FRAME;
 +            h->cur_pic_ptr->reference |= h->picture_structure;
          } else if (h->cur_pic_ptr->long_ref) {
              av_log(h->avctx, AV_LOG_ERROR, "illegal short term reference "
                                             "assignment for second field "
          }
      }
  
 -    if (h->long_ref_count + h->short_ref_count -
 -        (h->short_ref[0] == h->cur_pic_ptr) > h->ps.sps->ref_frame_count) {
 +    if (h->long_ref_count + h->short_ref_count > FFMAX(h->ps.sps->ref_frame_count, 1)) {
  
          /* We have too many reference frames, probably due to corrupted
           * stream. Need to discard one frame. Prevents overrun of the
          }
      }
  
 +    for (i = 0; i<h->short_ref_count; i++) {
 +        pic = h->short_ref[i];
 +        if (pic->invalid_gap) {
 +            int d = av_mod_uintp2(h->cur_pic_ptr->frame_num - pic->frame_num, h->ps.sps->log2_max_frame_num);
 +            if (d > h->ps.sps->ref_frame_count)
 +                remove_short(h, pic->frame_num, 0);
 +        }
 +    }
 +
      print_short_term(h);
      print_long_term(h);
 +
 +    for (i = 0; i < FF_ARRAY_ELEMS(h->ps.pps_list); i++) {
 +        if (h->ps.pps_list[i]) {
 +            const PPS *pps = (const PPS *)h->ps.pps_list[i]->data;
 +            pps_ref_count[0] = FFMAX(pps_ref_count[0], pps->ref_count[0]);
 +            pps_ref_count[1] = FFMAX(pps_ref_count[1], pps->ref_count[1]);
 +        }
 +    }
 +
 +    if (   err >= 0
 +        && h->long_ref_count==0
 +        && (   h->short_ref_count<=2
 +            || pps_ref_count[0] <= 1 + (h->picture_structure != PICT_FRAME) && pps_ref_count[1] <= 1)
 +        && pps_ref_count[0]<=2 + (h->picture_structure != PICT_FRAME) + (2*!h->has_recovery_point)
 +        && h->cur_pic_ptr->f->pict_type == AV_PICTURE_TYPE_I){
 +        h->cur_pic_ptr->recovered |= 1;
 +        if(!h->avctx->has_b_frames)
 +            h->frame_recovered |= FRAME_RECOVERED_SEI;
 +    }
 +
      return (h->avctx->err_recognition & AV_EF_EXPLODE) ? err : 0;
  }
  
@@@ -829,7 -723,7 +822,7 @@@ int ff_h264_decode_ref_pic_marking(H264
                                     int first_slice)
  {
      int i, ret;
 -    MMCO mmco_temp[MAX_MMCO_COUNT], *mmco = first_slice ? h->mmco : mmco_temp;
 +    MMCO mmco_temp[MAX_MMCO_COUNT], *mmco = mmco_temp;
      int mmco_index = 0;
  
      if (h->nal_unit_type == NAL_IDR_SLICE) { // FIXME fields
                  mmco[i].opcode = opcode;
                  if (opcode == MMCO_SHORT2UNUSED || opcode == MMCO_SHORT2LONG) {
                      mmco[i].short_pic_num =
 -                        (h->curr_pic_num - get_ue_golomb(gb) - 1) &
 +                        (h->curr_pic_num - get_ue_golomb_long(gb) - 1) &
                              (h->max_pic_num - 1);
  #if 0
                      if (mmco[i].short_pic_num >= h->short_ref_count ||
      }
  
      if (first_slice && mmco_index != -1) {
 +        memcpy(h->mmco, mmco_temp, sizeof(h->mmco));
          h->mmco_index = mmco_index;
      } else if (!first_slice && mmco_index >= 0 &&
                 (mmco_index != h->mmco_index ||