Merge commit 'b1bb8fb860b47e90dd67f0c5740698128fc82dcc'
authorMichael Niedermayer <michaelni@gmx.at>
Wed, 17 Apr 2013 13:35:41 +0000 (15:35 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Wed, 17 Apr 2013 13:40:47 +0000 (15:40 +0200)
* commit 'b1bb8fb860b47e90dd67f0c5740698128fc82dcc':
  svq1dec: check that the reference frame has the same dimensions as the current one

Conflicts:
libavcodec/svq1dec.c

See: 3b57bb478ff4455773378355e285877d757e151e

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

diff --combined libavcodec/svq1dec.c
@@@ -8,20 -8,20 +8,20 @@@
   *
   * SVQ1 Encoder (c) 2004 Mike Melanson <melanson@pcisys.net>
   *
 - * 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
   */
  
@@@ -33,8 -33,8 +33,8 @@@
   */
  
  #include "avcodec.h"
 -#include "dsputil.h"
  #include "get_bits.h"
 +#include "hpeldsp.h"
  #include "internal.h"
  #include "mathops.h"
  #include "svq1.h"
@@@ -58,7 -58,7 +58,7 @@@ typedef struct svq1_pmv_s 
  } svq1_pmv;
  
  typedef struct SVQ1Context {
 -    DSPContext dsp;
 +    HpelDSPContext hdsp;
      GetBitContext gb;
      AVFrame *prev;
      int width;
@@@ -111,11 -111,12 +111,11 @@@ static const uint8_t string_table[256] 
                  break;                                                  \
          }                                                               \
          /* divide block if next bit set */                              \
 -        if (get_bits1(bitbuf) == 0)                                     \
 +        if (!get_bits1(bitbuf))                                         \
              break;                                                      \
          /* add child nodes */                                           \
          list[n++] = list[i];                                            \
 -        list[n++] = list[i] +                                           \
 -                    (((level & 1) ? pitch : 1) << (level / 2 + 1));     \
 +        list[n++] = list[i] + (((level & 1) ? pitch : 1) << ((level >> 1) + 1));\
      }
  
  #define SVQ1_ADD_CODEBOOK()                                             \
                        16 * j) << (level + 1);                           \
      }                                                                   \
      mean -= stages * 128;                                               \
 -    n4    = mean + (mean >> 31) << 16 | (mean & 0xFFFF);
 +    n4    = (mean << 16) + mean;
  
  static int svq1_decode_block_intra(GetBitContext *bitbuf, uint8_t *pixels,
                                     int pitch)
@@@ -319,7 -320,7 +319,7 @@@ static void svq1_skip_block(uint8_t *cu
      }
  }
  
 -static int svq1_motion_inter_block(DSPContext *dsp, GetBitContext *bitbuf,
 +static int svq1_motion_inter_block(HpelDSPContext *hdsp, GetBitContext *bitbuf,
                                     uint8_t *current, uint8_t *previous,
                                     int pitch, svq1_pmv *motion, int x, int y)
  {
      }
  
      result = svq1_decode_motion_vector(bitbuf, &mv, pmv);
 -
 -    if (result != 0)
 +    if (result)
          return result;
  
      motion[0].x         =
      src = &previous[(x + (mv.x >> 1)) + (y + (mv.y >> 1)) * pitch];
      dst = current;
  
 -    dsp->put_pixels_tab[0][(mv.y & 1) << 1 | (mv.x & 1)](dst, src, pitch, 16);
 +    hdsp->put_pixels_tab[0][(mv.y & 1) << 1 | (mv.x & 1)](dst, src, pitch, 16);
  
      return 0;
  }
  
 -static int svq1_motion_inter_4v_block(DSPContext *dsp, GetBitContext *bitbuf,
 +static int svq1_motion_inter_4v_block(HpelDSPContext *hdsp, GetBitContext *bitbuf,
                                        uint8_t *current, uint8_t *previous,
                                        int pitch, svq1_pmv *motion, int x, int y)
  {
      }
  
      result = svq1_decode_motion_vector(bitbuf, &mv, pmv);
 -
 -    if (result != 0)
 +    if (result)
          return result;
  
      /* predict and decode motion vector (1) */
          pmv[1] = &motion[(x / 8) + 3];
      }
      result = svq1_decode_motion_vector(bitbuf, &motion[0], pmv);
 -
 -    if (result != 0)
 +    if (result)
          return result;
  
      /* predict and decode motion vector (2) */
      pmv[2] = &motion[(x / 8) + 1];
  
      result = svq1_decode_motion_vector(bitbuf, &motion[(x / 8) + 2], pmv);
 -
 -    if (result != 0)
 +    if (result)
          return result;
  
      /* predict and decode motion vector (3) */
      pmv[3] = &motion[(x / 8) + 3];
  
      result = svq1_decode_motion_vector(bitbuf, pmv[3], pmv);
 -
 -    if (result != 0)
 +    if (result)
          return result;
  
      /* form predictions */
          src = &previous[(x + (mvx >> 1)) + (y + (mvy >> 1)) * pitch];
          dst = current;
  
 -        dsp->put_pixels_tab[1][((mvy & 1) << 1) | (mvx & 1)](dst, src, pitch, 8);
 +        hdsp->put_pixels_tab[1][((mvy & 1) << 1) | (mvx & 1)](dst, src, pitch, 8);
  
          /* select next block */
          if (i & 1)
      return 0;
  }
  
 -static int svq1_decode_delta_block(AVCodecContext *avctx, DSPContext *dsp,
 +static int svq1_decode_delta_block(AVCodecContext *avctx, HpelDSPContext *hdsp,
                                     GetBitContext *bitbuf,
                                     uint8_t *current, uint8_t *previous,
                                     int pitch, svq1_pmv *motion, int x, int y)
          break;
  
      case SVQ1_BLOCK_INTER:
 -        result = svq1_motion_inter_block(dsp, bitbuf, current, previous,
 +        result = svq1_motion_inter_block(hdsp, bitbuf, current, previous,
                                           pitch, motion, x, y);
  
          if (result != 0) {
          break;
  
      case SVQ1_BLOCK_INTER_4V:
 -        result = svq1_motion_inter_4v_block(dsp, bitbuf, current, previous,
 +        result = svq1_motion_inter_4v_block(hdsp, bitbuf, current, previous,
                                              pitch, motion, x, y);
  
          if (result != 0) {
@@@ -516,8 -522,6 +516,8 @@@ static int svq1_decode_frame_header(AVC
      SVQ1Context *s = avctx->priv_data;
      GetBitContext *bitbuf = &s->gb;
      int frame_size_code;
 +    int width  = s->width;
 +    int height = s->height;
  
      skip_bits(bitbuf, 8); /* temporal_reference */
  
  
          if (frame_size_code == 7) {
              /* load width, height (12 bits each) */
 -            s->width  = get_bits(bitbuf, 12);
 -            s->height = get_bits(bitbuf, 12);
 +            width  = get_bits(bitbuf, 12);
 +            height = get_bits(bitbuf, 12);
  
 -            if (!s->width || !s->height)
 +            if (!width || !height)
                  return AVERROR_INVALIDDATA;
          } else {
              /* get width, height from table */
 -            s->width  = ff_svq1_frame_size_table[frame_size_code][0];
 -            s->height = ff_svq1_frame_size_table[frame_size_code][1];
 +            width  = ff_svq1_frame_size_table[frame_size_code][0];
 +            height = ff_svq1_frame_size_table[frame_size_code][1];
          }
      }
  
      /* unknown fields */
 -    if (get_bits1(bitbuf) == 1) {
 +    if (get_bits1(bitbuf)) {
          skip_bits1(bitbuf);    /* use packet checksum if (1) */
          skip_bits1(bitbuf);    /* component checksums after image data if (1) */
  
              return AVERROR_INVALIDDATA;
      }
  
 -    if (get_bits1(bitbuf) == 1) {
 +    if (get_bits1(bitbuf)) {
          skip_bits1(bitbuf);
          skip_bits(bitbuf, 4);
          skip_bits1(bitbuf);
          skip_bits(bitbuf, 2);
  
 -        while (get_bits1(bitbuf) == 1)
 +        while (get_bits1(bitbuf))
              skip_bits(bitbuf, 8);
      }
  
 +    s->width  = width;
 +    s->height = height;
      return 0;
  }
  
@@@ -628,14 -630,12 +628,14 @@@ static int svq1_decode_frame(AVCodecCon
      if (s->frame_code != 0x20) {
          uint32_t *src = (uint32_t *)(buf + 4);
  
 +        if (buf_size < 36)
 +            return AVERROR_INVALIDDATA;
 +
          for (i = 0; i < 4; i++)
              src[i] = ((src[i] << 16) | (src[i] >> 16)) ^ src[7 - i];
      }
  
      result = svq1_decode_frame_header(avctx, cur);
 -
      if (result != 0) {
          av_dlog(avctx, "Error in svq1_decode_frame_header %i\n", result);
          return result;
                  for (x = 0; x < width; x += 16) {
                      result = svq1_decode_block_intra(&s->gb, &current[x],
                                                       linesize);
 -                    if (result != 0) {
 -                        av_log(avctx, AV_LOG_INFO,
 +                    if (result) {
 +                        av_log(avctx, AV_LOG_ERROR,
                                 "Error in svq1_decode_block %i (keyframe)\n",
                                 result);
                          goto err;
          } else {
              /* delta frame */
              uint8_t *previous = s->prev->data[i];
-             if (!previous || s->prev->width != cur->width || s->prev->height != cur->height) {
+             if (!previous ||
+                 s->prev->width != s->width || s->prev->height != s->height) {
                  av_log(avctx, AV_LOG_ERROR, "Missing reference frame.\n");
                  result = AVERROR_INVALIDDATA;
                  goto err;
  
              for (y = 0; y < height; y += 16) {
                  for (x = 0; x < width; x += 16) {
 -                    result = svq1_decode_delta_block(avctx, &s->dsp,
 +                    result = svq1_decode_delta_block(avctx, &s->hdsp,
                                                       &s->gb, &current[x],
                                                       previous, linesize,
                                                       pmv, x, y);
 -                    if (result != 0) {
 +                    if (result) {
                          av_dlog(avctx,
                                  "Error in svq1_decode_delta_block %i\n",
                                  result);
@@@ -748,7 -749,7 +749,7 @@@ static av_cold int svq1_decode_init(AVC
      s->height           = avctx->height + 3 & ~3;
      avctx->pix_fmt      = AV_PIX_FMT_YUV410P;
  
 -    ff_dsputil_init(&s->dsp, avctx);
 +    ff_hpeldsp_init(&s->hdsp, avctx->flags);
  
      INIT_VLC_STATIC(&svq1_block_type, 2, 4,
                      &ff_svq1_block_type_vlc[0][1], 2, 1,