Switch some ints to unsigned (they can only have positive values, this allows
[ffmpeg.git] / libavcodec / iff.c
index e3ce6c1..b5ee2e6 100644 (file)
@@ -1,6 +1,7 @@
 /*
  * IFF PBM/ILBM bitmap decoder
  * Copyright (c) 2010 Peter Ross <pross@xvid.org>
+ * Copyright (c) 2010 Sebastian Vater <cdgs.basty@googlemail.com>
  *
  * This file is part of FFmpeg.
  *
  */
 
 /**
- * @file libavcodec/iff.c
+ * @file
  * IFF PBM/ILBM bitmap decoder
  */
 
 #include "bytestream.h"
 #include "avcodec.h"
+#include "get_bits.h"
+#include "iff.h"
+
+typedef struct {
+    AVFrame frame;
+    unsigned planesize;
+    uint8_t * planebuf;
+} IffContext;
 
 /**
  * Convert CMAP buffer (stored in extradata) to lavc palette format
  */
 int ff_cmap_read_palette(AVCodecContext *avctx, uint32_t *pal)
 {
-    int count, i;
+    unsigned count, i;
 
     if (avctx->bits_per_coded_sample > 8) {
         av_log(avctx, AV_LOG_ERROR, "bit_per_coded_sample > 8 not supported\n");
@@ -45,51 +54,74 @@ int ff_cmap_read_palette(AVCodecContext *avctx, uint32_t *pal)
         return AVERROR_INVALIDDATA;
     }
     for (i=0; i < count; i++) {
-        pal[i] = AV_RB24( avctx->extradata + i*3 );
+        pal[i] = 0xFF000000 | AV_RB24( avctx->extradata + i*3 );
     }
     return 0;
 }
 
 static av_cold int decode_init(AVCodecContext *avctx)
 {
-    AVFrame *frame = avctx->priv_data;
+    IffContext *s = avctx->priv_data;
+    int err;
+
+    if (avctx->bits_per_coded_sample <= 8) {
+        avctx->pix_fmt = PIX_FMT_PAL8;
+    } else if (avctx->bits_per_coded_sample <= 32) {
+        avctx->pix_fmt = PIX_FMT_BGR32;
+    } else {
+        return AVERROR_INVALIDDATA;
+    }
 
-    avctx->pix_fmt = PIX_FMT_PAL8;
-    frame->reference = 1;
+    s->planesize = avctx->width >> 3;
+    s->planebuf = av_malloc(s->planesize + FF_INPUT_BUFFER_PADDING_SIZE);
+    if (!s->planebuf)
+        return AVERROR(ENOMEM);
 
-    if (avctx->get_buffer(avctx, frame) < 0) {
+    s->frame.reference = 1;
+    if ((err = avctx->get_buffer(avctx, &s->frame) < 0)) {
         av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
-        return AVERROR_UNKNOWN;
+        return err;
     }
-    return ff_cmap_read_palette(avctx, (uint32_t*)frame->data[1]);
+
+    return avctx->bits_per_coded_sample <= 8 ?
+       ff_cmap_read_palette(avctx, (uint32_t*)s->frame.data[1]) : 0;
 }
 
 /**
- * Interleaved memcpy
+ * Decode interleaved plane buffer up to 8bpp
+ * @param dst Destination buffer
+ * @param buf Source buffer
+ * @param buf_size
+ * @param bps bits_per_coded_sample (must be <= 8)
+ * @param plane plane number to decode as
  */
-static void imemcpy(uint8_t *dst, const uint8_t const *buf, int x, int bps, int plane, int length)
+static void decodeplane8(uint8_t *dst, const uint8_t *const buf, int buf_size, int bps, int plane)
 {
-    int i, b;
-    for(i = 0; i < length; i++) {
-        int value = buf[i];
-        for (b = 0; b < bps; b++) {
-            if (value & (1<<b))
-                dst[ (x+i)*bps + 7 - b] |= 1<<plane;
-       }
+    GetBitContext gb;
+    unsigned int i;
+    const unsigned b = (buf_size * 8) + bps - 1;
+    init_get_bits(&gb, buf, buf_size * 8);
+    for(i = 0; i < b; i++) {
+        dst[i] |= get_bits1(&gb) << plane;
     }
 }
 
 /**
- * Interleaved memset
+ * Decode interleaved plane buffer up to 24bpp
+ * @param dst Destination buffer
+ * @param buf Source buffer
+ * @param buf_size
+ * @param bps bits_per_coded_sample
+ * @param plane plane number to decode as
  */
-static void imemset(uint8_t *dst, int value, int x, int bps, int plane, int length)
+static void decodeplane32(uint32_t *dst, const uint8_t *const buf, int buf_size, int bps, int plane)
 {
-    int i, b;
-    for(i = 0; i < length; i++) {
-        for (b = 0; b < bps; b++) {
-            if (value & (1<<b))
-                dst[ (x+i)*bps + 7 - b] |= 1<<plane;
-       }
+    GetBitContext gb;
+    unsigned i;
+    const unsigned b = (buf_size * 8) + bps - 1;
+    init_get_bits(&gb, buf, buf_size * 8);
+    for(i = 0; i < b; i++) {
+        dst[i] |= get_bits1(&gb) << plane;
     }
 }
 
@@ -97,33 +129,32 @@ static int decode_frame_ilbm(AVCodecContext *avctx,
                             void *data, int *data_size,
                             AVPacket *avpkt)
 {
-    AVFrame *frame = avctx->priv_data;
+    IffContext *s = avctx->priv_data;
     const uint8_t *buf = avpkt->data;
-    int buf_size = avpkt->size;
-    int planewidth = avctx->width / avctx->bits_per_coded_sample;
-    int y, plane;
+    unsigned buf_size = avpkt->size;
+    const uint8_t *buf_end = buf+buf_size;
+    unsigned y, plane;
 
-    if (avctx->reget_buffer(avctx, frame) < 0){
+    if (avctx->reget_buffer(avctx, &s->frame) < 0){
         av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
         return -1;
     }
 
-    if (buf_size < avctx->width * avctx->height) {
-        av_log(avctx, AV_LOG_ERROR, "buffer underflow\n");
-        return -1;
-    }
-
     for(y = 0; y < avctx->height; y++ ) {
-        uint8_t *row = &frame->data[0][ y*frame->linesize[0] ];
-        memset(row, 0, avctx->width);
-        for (plane = 0; plane < avctx->bits_per_coded_sample; plane++) {
-            imemcpy(row, buf, 0, avctx->bits_per_coded_sample, plane, planewidth);
-            buf += planewidth;
+        uint8_t *row = &s->frame.data[0][ y*s->frame.linesize[0] ];
+        memset(row, 0, avctx->pix_fmt == PIX_FMT_PAL8 ? avctx->width : (avctx->width * 4));
+        for (plane = 0; plane < avctx->bits_per_coded_sample && buf < buf_end; plane++) {
+            if (avctx->pix_fmt == PIX_FMT_PAL8) {
+                decodeplane8(row, buf, FFMIN(s->planesize, buf_end - buf), avctx->bits_per_coded_sample, plane);
+            } else { // PIX_FMT_BGR32
+                decodeplane32((uint32_t *) row, buf, FFMIN(s->planesize, buf_end - buf), avctx->bits_per_coded_sample, plane);
+            }
+            buf += s->planesize;
         }
     }
 
     *data_size = sizeof(AVFrame);
-    *(AVFrame*)data = *frame;
+    *(AVFrame*)data = s->frame;
     return buf_size;
 }
 
@@ -131,43 +162,47 @@ static int decode_frame_byterun1(AVCodecContext *avctx,
                             void *data, int *data_size,
                             AVPacket *avpkt)
 {
-    AVFrame *frame = avctx->priv_data;
+    IffContext *s = avctx->priv_data;
     const uint8_t *buf = avpkt->data;
-    int buf_size = avpkt->size;
+    unsigned buf_size = avpkt->size;
     const uint8_t *buf_end = buf+buf_size;
-    int planewidth = avctx->width / avctx->bits_per_coded_sample;
-    int y, plane, x;
+    unsigned y, plane, x;
 
-    if (avctx->reget_buffer(avctx, frame) < 0){
+    if (avctx->reget_buffer(avctx, &s->frame) < 0){
         av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
         return -1;
     }
 
     for(y = 0; y < avctx->height ; y++ ) {
-        uint8_t *row = &frame->data[0][ y*frame->linesize[0] ];
+        uint8_t *row = &s->frame.data[0][ y*s->frame.linesize[0] ];
         if (avctx->codec_tag == MKTAG('I','L','B','M')) { //interleaved
-            memset(row, 0, avctx->width);
+            memset(row, 0, avctx->pix_fmt == PIX_FMT_PAL8 ? avctx->width : (avctx->width * 4));
             for (plane = 0; plane < avctx->bits_per_coded_sample; plane++) {
-                for(x = 0; x < planewidth && buf < buf_end; ) {
+                for(x = 0; x < s->planesize && buf < buf_end; ) {
                     int8_t value = *buf++;
-                    int length;
+                    unsigned length;
                     if (value >= 0) {
                         length = value + 1;
-                        imemcpy(row, buf, x, avctx->bits_per_coded_sample, plane, FFMIN3(length, buf_end - buf, planewidth - x));
+                        memcpy(s->planebuf + x, buf, FFMIN3(length, s->planesize - x, buf_end - buf));
                         buf += length;
                     } else if (value > -128) {
                         length = -value + 1;
-                        imemset(row, *buf++, x, avctx->bits_per_coded_sample, plane, FFMIN(length, planewidth - x));
+                        memset(s->planebuf + x, *buf++, FFMIN(length, s->planesize - x));
                     } else { //noop
                         continue;
                     }
                     x += length;
                 }
+                if (avctx->pix_fmt == PIX_FMT_PAL8) {
+                    decodeplane8(row, s->planebuf, s->planesize, avctx->bits_per_coded_sample, plane);
+                } else { //PIX_FMT_BGR32
+                    decodeplane32((uint32_t *) row, s->planebuf, s->planesize, avctx->bits_per_coded_sample, plane);
+                }
             }
         } else {
             for(x = 0; x < avctx->width && buf < buf_end; ) {
                 int8_t value = *buf++;
-                int length;
+                unsigned length;
                 if (value >= 0) {
                     length = value + 1;
                     memcpy(row + x, buf, FFMIN3(length, buf_end - buf, avctx->width - x));
@@ -184,23 +219,24 @@ static int decode_frame_byterun1(AVCodecContext *avctx,
     }
 
     *data_size = sizeof(AVFrame);
-    *(AVFrame*)data = *frame;
+    *(AVFrame*)data = s->frame;
     return buf_size;
 }
 
 static av_cold int decode_end(AVCodecContext *avctx)
 {
-    AVFrame *frame = avctx->priv_data;
-    if (frame->data[0])
-        avctx->release_buffer(avctx, frame);
+    IffContext *s = avctx->priv_data;
+    if (s->frame.data[0])
+        avctx->release_buffer(avctx, &s->frame);
+    av_freep(&s->planebuf);
     return 0;
 }
 
 AVCodec iff_ilbm_decoder = {
     "iff_ilbm",
-    CODEC_TYPE_VIDEO,
+    AVMEDIA_TYPE_VIDEO,
     CODEC_ID_IFF_ILBM,
-    sizeof(AVFrame),
+    sizeof(IffContext),
     decode_init,
     NULL,
     decode_end,
@@ -211,9 +247,9 @@ AVCodec iff_ilbm_decoder = {
 
 AVCodec iff_byterun1_decoder = {
     "iff_byterun1",
-    CODEC_TYPE_VIDEO,
+    AVMEDIA_TYPE_VIDEO,
     CODEC_ID_IFF_BYTERUN1,
-    sizeof(AVFrame),
+    sizeof(IffContext),
     decode_init,
     NULL,
     decode_end,