Merge commit '7b1fbd4729a52dd7c02622dbe7bb81a6a7ed12f8'
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 7 Jan 2013 00:11:14 +0000 (01:11 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 7 Jan 2013 00:11:14 +0000 (01:11 +0100)
* commit '7b1fbd4729a52dd7c02622dbe7bb81a6a7ed12f8':
  indeo2: check decoding errors.
  indeo2: return meaningful error codes
  rl2: cosmetics, reformat

Conflicts:
libavcodec/rl2.c

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

@@@ -146,13 -146,16 +146,13 @@@ static int ir2_decode_frame(AVCodecCont
      Ir2Context * const s = avctx->priv_data;
      AVFrame *picture = data;
      AVFrame * const p = &s->picture;
-     int start;
+     int start, ret;
  
 -    if(p->data[0])
 -        avctx->release_buffer(avctx, p);
 -
 -    p->reference = 1;
 +    p->reference = 3;
      p->buffer_hints = FF_BUFFER_HINTS_VALID | FF_BUFFER_HINTS_PRESERVE | FF_BUFFER_HINTS_REUSABLE;
-     if (avctx->reget_buffer(avctx, p)) {
+     if ((ret = avctx->reget_buffer(avctx, p)) < 0) {
          av_log(s->avctx, AV_LOG_ERROR, "reget_buffer() failed\n");
-         return -1;
+         return ret;
      }
  
      start = 48; /* hardcoded for now */
@@@ -135,12 -135,12 +135,13 @@@ static av_cold int rl2_decode_init(AVCo
      Rl2Context *s = avctx->priv_data;
      int back_size;
      int i;
-     s->avctx = avctx;
+     s->avctx       = avctx;
      avctx->pix_fmt = AV_PIX_FMT_PAL8;
 +    avcodec_get_frame_defaults(&s->frame);
  
      /** parse extra data */
-     if(!avctx->extradata || avctx->extradata_size < EXTRADATA1_SIZE){
+     if (!avctx->extradata || avctx->extradata_size < EXTRADATA1_SIZE) {
          av_log(avctx, AV_LOG_ERROR, "invalid extradata size\n");
          return AVERROR(EINVAL);
      }
      }
  
      /** initialize palette */
-     for(i=0;i<AVPALETTE_COUNT;i++)
+     for (i = 0; i < AVPALETTE_COUNT; i++)
 -        s->palette[i] = AV_RB24(&avctx->extradata[6 + i * 3]);
 +        s->palette[i] = 0xFFU << 24 | AV_RB24(&avctx->extradata[6 + i * 3]);
  
      /** decode background frame if present */
      back_size = avctx->extradata_size - EXTRADATA1_SIZE;