Merge commit 'f337c29017b10c98ccb4dce20efced4c74b665f6'
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 26 Jan 2013 13:47:08 +0000 (14:47 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sat, 26 Jan 2013 13:47:08 +0000 (14:47 +0100)
* commit 'f337c29017b10c98ccb4dce20efced4c74b665f6':
  eatgq: return meaningful error codes.
  eatgv: cosmetics, reformat

Conflicts:
libavcodec/eatgq.c
libavcodec/eatgv.c

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

diff --combined libavcodec/eatgq.c
@@@ -2,20 -2,20 +2,20 @@@
   * Electronic Arts TGQ Video Decoder
   * Copyright (c) 2007-2008 Peter Ross <pross@xvid.org>
   *
 - * 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 St, Fifth Floor, Boston, MA  02110-1301  USA
   */
  
@@@ -142,10 -142,7 +142,10 @@@ static void tgq_idct_put_mb_dconly(TgqC
      }
  }
  
 -static void tgq_decode_mb(TgqContext *s, int mb_y, int mb_x){
 +/**
 + * @return <0 on error
 + */
 +static int tgq_decode_mb(TgqContext *s, int mb_y, int mb_x){
      int mode;
      int i;
      int8_t dc[6];
      mode = bytestream2_get_byte(&s->gb);
      if (mode>12) {
          GetBitContext gb;
 -        init_get_bits(&gb, s->gb.buffer, FFMIN(s->gb.buffer_end - s->gb.buffer, mode) * 8);
 +        init_get_bits(&gb, s->gb.buffer, FFMIN(bytestream2_get_bytes_left(&s->gb), mode) * 8);
          for(i=0; i<6; i++)
              tgq_decode_block(s, s->block[i], &gb);
          tgq_idct_put_mb(s, s->block, mb_x, mb_y);
              }
          }else{
              av_log(s->avctx, AV_LOG_ERROR, "unsupported mb mode %i\n", mode);
 +            return -1;
          }
          tgq_idct_put_mb_dconly(s, mb_x, mb_y, dc);
      }
 +    return 0;
  }
  
  static void tgq_calculate_qtable(TgqContext *s, int quant){
@@@ -194,14 -189,13 +194,14 @@@ static int tgq_decode_frame(AVCodecCont
      const uint8_t *buf = avpkt->data;
      int buf_size = avpkt->size;
      TgqContext *s = avctx->priv_data;
-     int x,y;
+     int x, y, ret;
 -    int big_endian = AV_RL32(&buf[4]) > 0x000FFFFF;
 +    int big_endian;
  
      if (buf_size < 16) {
          av_log(avctx, AV_LOG_WARNING, "truncated header\n");
-         return -1;
+         return AVERROR_INVALIDDATA;
      }
 +    big_endian = AV_RL32(&buf[4]) > 0x000FFFFF;
      bytestream2_init(&s->gb, buf + 8, buf_size - 8);
      if (big_endian) {
          s->width  = bytestream2_get_be16u(&s->gb);
          s->frame.key_frame = 1;
          s->frame.pict_type = AV_PICTURE_TYPE_I;
          s->frame.buffer_hints = FF_BUFFER_HINTS_VALID;
-         if (ff_get_buffer(avctx, &s->frame)) {
+         if ((ret = ff_get_buffer(avctx, &s->frame)) < 0) {
              av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
-             return -1;
+             return ret;
          }
      }
  
      for (y = 0; y < FFALIGN(avctx->height, 16) >> 4; y++)
          for (x = 0; x < FFALIGN(avctx->width, 16) >> 4; x++)
 -            tgq_decode_mb(s, y, x);
 +            if (tgq_decode_mb(s, y, x) < 0)
 +                return AVERROR_INVALIDDATA;
  
      *got_frame = 1;
      *(AVFrame*)data = s->frame;
diff --combined libavcodec/eatgv.c
@@@ -2,20 -2,20 +2,20 @@@
   * Electronic Arts TGV Video Decoder
   * Copyright (c) 2007-2008 Peter Ross
   *
 - * 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 St, Fifth Floor, Boston, MA  02110-1301  USA
   */
  
@@@ -50,13 -50,12 +50,14 @@@ typedef struct TgvContext 
      int num_blocks_packed; ///< current length of block_codebook
  } TgvContext;
  
- static av_cold int tgv_decode_init(AVCodecContext *avctx){
+ static av_cold int tgv_decode_init(AVCodecContext *avctx)
+ {
      TgvContext *s = avctx->priv_data;
-     s->avctx = avctx;
+     s->avctx         = avctx;
      avctx->time_base = (AVRational){1, 15};
-     avctx->pix_fmt = AV_PIX_FMT_PAL8;
+     avctx->pix_fmt   = AV_PIX_FMT_PAL8;
 +    avcodec_get_frame_defaults(&s->frame);
 +    avcodec_get_frame_defaults(&s->last_frame);
      return 0;
  }
  
@@@ -64,7 -63,9 +65,9 @@@
   * Unpack buffer
   * @return 0 on success, -1 on critical buffer underflow
   */
- static int unpack(const uint8_t *src, const uint8_t *src_end, unsigned char *dst, int width, int height) {
+ static int unpack(const uint8_t *src, const uint8_t *src_end,
+                   unsigned char *dst, int width, int height)
+ {
      unsigned char *dst_end = dst + width*height;
      int size, size1, size2, offset, run;
      unsigned char *dst_start = dst;
      else
          src += 2;
  
 -    if (src + 3 > src_end)
 +    if (src_end - src < 3)
          return AVERROR_INVALIDDATA;
      size = AV_RB24(src);
      src += 3;
  
-     while(size>0 && src<src_end) {
+     while (size > 0 && src < src_end) {
  
          /* determine size1 and size2 */
          size1 = (src[0] & 3);
-         if ( src[0] & 0x80 ) {  // 1
+         if (src[0] & 0x80) {  // 1
              if (src[0] & 0x40 ) {  // 11
-                 if ( src[0] & 0x20 ) {  // 111
-                     if ( src[0] < 0xFC )  // !(111111)
+                 if (src[0] & 0x20) {  // 111
+                     if (src[0] < 0xFC)  // !(111111)
                          size1 = (((src[0] & 31) + 1) << 2);
                      src++;
                      size2 = 0;
                  } else {  // 110
                      offset = ((src[0] & 0x10) << 12) + AV_RB16(&src[1]) + 1;
-                     size2 = ((src[0] & 0xC) << 6) + src[3] + 5;
-                     src += 4;
+                     size2  = ((src[0] & 0xC) << 6) + src[3] + 5;
+                     src   += 4;
                  }
              } else {  // 10
-                 size1 = ( ( src[1] & 0xC0) >> 6 );
+                 size1  = ((src[1] & 0xC0) >> 6);
                  offset = (AV_RB16(&src[1]) & 0x3FFF) + 1;
-                 size2 = (src[0] & 0x3F) + 4;
-                 src += 3;
+                 size2  = (src[0] & 0x3F) + 4;
+                 src   += 3;
              }
          } else {  // 0
              offset = ((src[0] & 0x60) << 3) + src[1] + 1;
-             size2 = ((src[0] & 0x1C) >> 2) + 3;
-             src += 2;
+             size2  = ((src[0] & 0x1C) >> 2) + 3;
+             src   += 2;
          }
  
  
          /* fetch strip from src */
-         if (size1>src_end-src)
+         if (size1 > src_end - src)
              break;
  
-         if (size1>0) {
+         if (size1 > 0) {
              size -= size1;
-             run = FFMIN(size1, dst_end-dst);
+             run   = FFMIN(size1, dst_end - dst);
              memcpy(dst, src, run);
              dst += run;
              src += run;
          }
  
-         if (size2>0) {
-             if (dst-dst_start<offset)
+         if (size2 > 0) {
+             if (dst - dst_start < offset)
                  return 0;
              size -= size2;
-             run = FFMIN(size2, dst_end-dst);
+             run   = FFMIN(size2, dst_end - dst);
              av_memcpy_backptr(dst, offset, run);
              dst += run;
          }
   * Decode inter-frame
   * @return 0 on success, -1 on critical buffer underflow
   */
- static int tgv_decode_inter(TgvContext * s, const uint8_t *buf, const uint8_t *buf_end){
+ static int tgv_decode_inter(TgvContext *s, const uint8_t *buf,
+                             const uint8_t *buf_end)
+ {
      int num_mvs;
      int num_blocks_raw;
      int num_blocks_packed;
      int mvbits;
      const unsigned char *blocks_raw;
  
 -    if (buf + 12 > buf_end)
 +    if(buf_end - buf < 12)
          return AVERROR_INVALIDDATA;
  
      num_mvs           = AV_RL16(&buf[0]);
      }
  
      /* read motion vectors */
-     mvbits = (num_mvs*2*10+31) & ~31;
+     mvbits = (num_mvs * 2 * 10 + 31) & ~31;
  
-     if (buf_end - buf < (mvbits>>3)+16*num_blocks_raw+8*num_blocks_packed)
 -    if (buf + (mvbits >> 3) + 16 * num_blocks_raw + 8 * num_blocks_packed > buf_end)
++    if (buf_end - buf < (mvbits>>3) + 16*num_blocks_raw + 8*num_blocks_packed)
          return AVERROR_INVALIDDATA;
  
      init_get_bits(&gb, buf, mvbits);
-     for (i=0; i<num_mvs; i++) {
+     for (i = 0; i < num_mvs; i++) {
          s->mv_codebook[i][0] = get_sbits(&gb, 10);
          s->mv_codebook[i][1] = get_sbits(&gb, 10);
      }
-     buf += mvbits>>3;
+     buf += mvbits >> 3;
  
      /* note ptr to uncompressed blocks */
      blocks_raw = buf;
-     buf += num_blocks_raw*16;
+     buf       += num_blocks_raw * 16;
  
      /* read compressed blocks */
-     init_get_bits(&gb, buf, (buf_end-buf)<<3);
-     for (i=0; i<num_blocks_packed; i++) {
+     init_get_bits(&gb, buf, (buf_end - buf) << 3);
+     for (i = 0; i < num_blocks_packed; i++) {
          int tmp[4];
-         for(j=0; j<4; j++)
+         for (j = 0; j < 4; j++)
              tmp[j] = get_bits(&gb, 8);
-         for(j=0; j<16; j++)
+         for (j = 0; j < 16; j++)
              s->block_codebook[i][15-j] = tmp[get_bits(&gb, 2)];
      }
  
      if (get_bits_left(&gb) < vector_bits *
-         (s->avctx->height/4) * (s->avctx->width/4))
+         (s->avctx->height / 4) * (s->avctx->width / 4))
          return AVERROR_INVALIDDATA;
  
      /* read vectors and build frame */
-     for(y=0; y<s->avctx->height/4; y++)
-     for(x=0; x<s->avctx->width/4; x++) {
-         unsigned int vector = get_bits(&gb, vector_bits);
-         const unsigned char *src;
-         int src_stride;
-         if (vector < num_mvs) {
-             int mx = x * 4 + s->mv_codebook[vector][0];
-             int my = y * 4 + s->mv_codebook[vector][1];
-             if (   mx < 0 || mx + 4 > s->avctx->width
-                 || my < 0 || my + 4 > s->avctx->height) {
-                 av_log(s->avctx, AV_LOG_ERROR, "MV %d %d out of picture\n", mx, my);
-                 continue;
-             }
+     for (y = 0; y < s->avctx->height / 4; y++)
+         for (x = 0; x < s->avctx->width / 4; x++) {
+             unsigned int vector = get_bits(&gb, vector_bits);
+             const unsigned char *src;
+             int src_stride;
+             if (vector < num_mvs) {
+                 int mx = x * 4 + s->mv_codebook[vector][0];
+                 int my = y * 4 + s->mv_codebook[vector][1];
+                 if (mx < 0 || mx + 4 > s->avctx->width ||
 -                    my < 0 || my + 4 > s->avctx->height)
++                    my < 0 || my + 4 > s->avctx->height) {
++                    av_log(s->avctx, AV_LOG_ERROR, "MV %d %d out of picture\n", mx, my);
+                     continue;
++                }
  
-             src = s->last_frame.data[0] + mx + my * s->last_frame.linesize[0];
-             src_stride = s->last_frame.linesize[0];
-         }else{
-             int offset = vector - num_mvs;
-             if (offset<num_blocks_raw)
-                 src = blocks_raw + 16*offset;
-             else if (offset-num_blocks_raw<num_blocks_packed)
-                 src = s->block_codebook[offset-num_blocks_raw];
-             else
-                 continue;
-             src_stride = 4;
-         }
+                 src = s->last_frame.data[0] + mx + my * s->last_frame.linesize[0];
+                 src_stride = s->last_frame.linesize[0];
+             } else {
+                 int offset = vector - num_mvs;
+                 if (offset < num_blocks_raw)
+                     src = blocks_raw + 16*offset;
+                 else if (offset - num_blocks_raw < num_blocks_packed)
+                     src = s->block_codebook[offset - num_blocks_raw];
+                 else
+                     continue;
+                 src_stride = 4;
+             }
  
-         for(j=0; j<4; j++)
-         for(i=0; i<4; i++)
-             s->frame.data[0][ (y*4+j)*s->frame.linesize[0] + (x*4+i)  ] =
-                src[j*src_stride + i];
+             for (j = 0; j < 4; j++)
+                 for (i = 0; i < 4; i++)
+                     s->frame.data[0][(y * 4 + j) * s->frame.linesize[0] + (x * 4 + i)] =
+                         src[j * src_stride + i];
      }
  
      return 0;
@@@ -256,28 -257,25 +261,28 @@@ static int tgv_decode_frame(AVCodecCont
                              void *data, int *got_frame,
                              AVPacket *avpkt)
  {
-     const uint8_t *buf = avpkt->data;
-     int buf_size = avpkt->size;
-     TgvContext *s = avctx->priv_data;
+     const uint8_t *buf     = avpkt->data;
+     int buf_size           = avpkt->size;
+     TgvContext *s          = avctx->priv_data;
      const uint8_t *buf_end = buf + buf_size;
      int chunk_type, ret;
  
 +    if (buf_end - buf < EA_PREAMBLE_SIZE)
 +        return AVERROR_INVALIDDATA;
 +
      chunk_type = AV_RL32(&buf[0]);
-     buf += EA_PREAMBLE_SIZE;
+     buf       += EA_PREAMBLE_SIZE;
  
-     if (chunk_type==kVGT_TAG) {
+     if (chunk_type == kVGT_TAG) {
          int pal_count, i;
 -        if (buf + 12 > buf_end) {
 +        if(buf_end - buf < 12) {
              av_log(avctx, AV_LOG_WARNING, "truncated header\n");
              return AVERROR_INVALIDDATA;
          }
  
          s->width  = AV_RL16(&buf[0]);
          s->height = AV_RL16(&buf[2]);
-         if (s->avctx->width!=s->width || s->avctx->height!=s->height) {
+         if (s->avctx->width != s->width || s->avctx->height != s->height) {
              avcodec_set_dimensions(s->avctx, s->width, s->height);
              cond_release_buffer(&s->frame);
              cond_release_buffer(&s->last_frame);
  
          pal_count = AV_RL16(&buf[6]);
          buf += 12;
-         for(i=0; i<pal_count && i<AVPALETTE_COUNT && buf_end - buf >= 3; i++) {
 -        for (i = 0; i < pal_count && i < AVPALETTE_COUNT && buf + 2 < buf_end; i++) {
 -            s->palette[i] = AV_RB24(buf);
++        for(i = 0; i < pal_count && i < AVPALETTE_COUNT && buf_end - buf >= 3; i++) {
 +            s->palette[i] = 0xFFU << 24 | AV_RB24(buf);
              buf += 3;
          }
      }
      /* shuffle */
      FFSWAP(AVFrame, s->frame, s->last_frame);
      if (!s->frame.data[0]) {
 -        s->frame.reference = 1;
 +        s->frame.reference = 3;
          s->frame.buffer_hints = FF_BUFFER_HINTS_VALID;
          s->frame.linesize[0] = s->width;
  
      }
      memcpy(s->frame.data[1], s->palette, AVPALETTE_SIZE);
  
-     if(chunk_type==kVGT_TAG) {
+     if (chunk_type == kVGT_TAG) {
          s->frame.key_frame = 1;
          s->frame.pict_type = AV_PICTURE_TYPE_I;
-         if (unpack(buf, buf_end, s->frame.data[0], s->avctx->width, s->avctx->height)<0) {
+         if (unpack(buf, buf_end, s->frame.data[0], s->avctx->width, s->avctx->height) < 0) {
              av_log(avctx, AV_LOG_WARNING, "truncated intra frame\n");
              return AVERROR_INVALIDDATA;
          }
-     }else{
+     } else {
          if (!s->last_frame.data[0]) {
              av_log(avctx, AV_LOG_WARNING, "inter frame without corresponding intra frame\n");
              return buf_size;
          }
          s->frame.key_frame = 0;
          s->frame.pict_type = AV_PICTURE_TYPE_P;
-         if (tgv_decode_inter(s, buf, buf_end)<0) {
+         if (tgv_decode_inter(s, buf, buf_end) < 0) {
              av_log(avctx, AV_LOG_WARNING, "truncated inter frame\n");
              return AVERROR_INVALIDDATA;
          }