Merge commit '6689474b91049411b5b7986616d1c66b701ae11f'
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 7 Jan 2013 00:30:23 +0000 (01:30 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 7 Jan 2013 00:30:23 +0000 (01:30 +0100)
* commit '6689474b91049411b5b7986616d1c66b701ae11f':
  v210x: return meaningful error codes.
  r210dec: return meaningful error codes.
  vb: cosmetics, reformat

Conflicts:
libavcodec/vb.c

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

@@@ -40,11 -38,10 +40,11 @@@ static av_cold int decode_init(AVCodecC
  static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame,
                          AVPacket *avpkt)
  {
-     int h, w;
+     int h, w, ret;
      AVFrame *pic = avctx->coded_frame;
      const uint32_t *src = (const uint32_t *)avpkt->data;
 -    int aligned_width = FFALIGN(avctx->width, 64);
 +    int aligned_width = FFALIGN(avctx->width,
 +                                avctx->codec_id == AV_CODEC_ID_R10K ? 1 : 64);
      uint8_t *dst_line;
  
      if (pic->data[0])
Simple merge
diff --cc libavcodec/vb.c
@@@ -73,8 -73,8 +73,8 @@@ static void vb_decode_palette(VBDecCont
          av_log(c->avctx, AV_LOG_ERROR, "Palette data runs beyond chunk size\n");
          return;
      }
-     for(i = start; i <= start + size; i++)
+     for (i = start; i <= start + size; i++)
 -        c->pal[i] = bytestream2_get_be24(&c->stream);
 +        c->pal[i] = 0xFFU << 24 | bytestream2_get_be24(&c->stream);
  }
  
  static inline int check_pixel(uint8_t *buf, uint8_t *start, uint8_t *end)
@@@ -197,9 -197,9 +197,9 @@@ static int decode_frame(AVCodecContext 
  
      bytestream2_init(&c->stream, avpkt->data, avpkt->size);
  
-     if(c->pic.data[0])
+     if (c->pic.data[0])
          avctx->release_buffer(avctx, &c->pic);
 -    c->pic.reference = 1;
 +    c->pic.reference = 3;
      if ((ret = ff_get_buffer(avctx, &c->pic)) < 0) {
          av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
          return ret;
          j = (int16_t)bytestream2_get_le16(&c->stream);
          offset = i + j * avctx->width;
      }
-     if(flags & VB_HAS_VIDEO){
+     if (flags & VB_HAS_VIDEO) {
          size = bytestream2_get_le32(&c->stream);
 +        if(size > bytestream2_get_bytes_left(&c->stream)+4 || size<4){
 +            av_log(avctx, AV_LOG_ERROR, "Frame size invalid\n");
 +            return -1;
 +        }
          vb_decode_framedata(c, offset);
          bytestream2_skip(&c->stream, size - 4);
      }
@@@ -251,9 -247,8 +251,9 @@@ static av_cold int decode_init(AVCodecC
  {
      VBDecContext * const c = avctx->priv_data;
  
-     c->avctx = avctx;
+     c->avctx       = avctx;
      avctx->pix_fmt = AV_PIX_FMT_PAL8;
 +    avcodec_get_frame_defaults(&c->pic);
  
      c->frame      = av_mallocz(avctx->width * avctx->height);
      c->prev_frame = av_mallocz(avctx->width * avctx->height);