Merge commit '51dc4de1218a81ee8e5b3f941839c5e3125a6d4b'
authorDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sun, 8 May 2016 22:31:01 +0000 (23:31 +0100)
committerDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sun, 8 May 2016 22:33:08 +0000 (23:33 +0100)
* commit '51dc4de1218a81ee8e5b3f941839c5e3125a6d4b':
  rscc: Add extended pixel format support

Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
1  2 
libavcodec/rscc.c

diff --combined libavcodec/rscc.c
@@@ -2,20 -2,20 +2,20 @@@
   * innoHeim/Rsupport Screen Capture Codec
   * Copyright (C) 2015 Vittorio Giovara <vittorio.giovara@gmail.com>
   *
 - * 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
   */
  
@@@ -31,7 -31,7 +31,7 @@@
   * and it can be deflated or not. Similarly, pixel data comes after the header
   * and a variable size value, and it can be deflated or just raw.
   *
-  * Supports: BGRA
+  * Supports: BGRA, BGR24, RGB555, RGB8
   */
  
  #include <stdint.h>
@@@ -57,6 -57,7 +57,7 @@@ typedef struct RsccContext 
      AVFrame *reference;
      Tile *tiles;
      unsigned int tiles_size;
+     int component_size;
  
      /* zlib interaction */
      uint8_t *inflated_buf;
@@@ -80,14 -81,37 +81,38 @@@ static av_cold int rscc_init(AVCodecCon
      if (!ctx->reference)
          return AVERROR(ENOMEM);
  
-     if (avctx->codec_tag == MKTAG('I','S','C','C')) {
+     /* Get pixel format and the size of the pixel */
+     if (avctx->codec_tag == MKTAG('I', 'S', 'C', 'C')) {
          avctx->pix_fmt = AV_PIX_FMT_BGRA;
+         ctx->component_size = 4;
+     } else if (avctx->codec_tag == MKTAG('R', 'S', 'C', 'C')) {
+         ctx->component_size = avctx->bits_per_coded_sample / 8;
+         switch (avctx->bits_per_coded_sample) {
+         case 8:
+             avpriv_report_missing_feature(avctx, "8 bits per pixel");
+             return AVERROR_PATCHWELCOME;
+         case 16:
+             avctx->pix_fmt = AV_PIX_FMT_RGB555LE;
+             break;
+         case 24:
+             avctx->pix_fmt = AV_PIX_FMT_BGR24;
+             break;
+         case 32:
+             avctx->pix_fmt = AV_PIX_FMT_BGRA;
+             break;
+         default:
+             av_log(avctx, AV_LOG_ERROR, "Invalid bits per pixel value (%d)\n",
+                    avctx->bits_per_coded_sample);
+             return AVERROR_INVALIDDATA;
+         }
      } else {
 -        av_log(avctx, AV_LOG_ERROR, "Invalid codec tag\n");
 -        return AVERROR_INVALIDDATA;
 +        avctx->pix_fmt = AV_PIX_FMT_BGR0;
++        ctx->component_size = 4;
++        av_log(avctx, AV_LOG_WARNING, "Invalid codec tag\n");
      }
  
      /* Store the value to check for keyframes */
-     ctx->inflated_size = avctx->width * avctx->height * 4;
+     ctx->inflated_size = avctx->width * avctx->height * ctx->component_size;
  
      /* Allocate maximum size possible, a full frame */
      ctx->inflated_buf = av_malloc(ctx->inflated_size);
@@@ -184,7 -208,7 +209,7 @@@ static int rscc_decode_frame(AVCodecCon
          ctx->tiles[i].y = bytestream2_get_le16(gbc);
          ctx->tiles[i].h = bytestream2_get_le16(gbc);
  
-         pixel_size += ctx->tiles[i].w * ctx->tiles[i].h * 4;
+         pixel_size += ctx->tiles[i].w * ctx->tiles[i].h * ctx->component_size;
  
          ff_dlog(avctx, "tile %d orig(%d,%d) %dx%d.\n", i,
                  ctx->tiles[i].x, ctx->tiles[i].y,
  
      /* Get pixels buffer, it may be deflated or just raw */
      if (pixel_size == packed_size) {
 +        if (bytestream2_get_bytes_left(gbc) < pixel_size) {
 +            av_log(avctx, AV_LOG_ERROR, "Insufficient input for %d\n", pixel_size);
 +            ret = AVERROR_INVALIDDATA;
 +            goto end;
 +        }
          pixels = gbc->buffer;
      } else {
          uLongf len = ctx->inflated_size;
      for (i = 0; i < tiles_nb; i++) {
          uint8_t *dst = ctx->reference->data[0] + ctx->reference->linesize[0] *
                         (avctx->height - ctx->tiles[i].y - 1) +
-                        ctx->tiles[i].x * 4;
+                        ctx->tiles[i].x * ctx->component_size;
          av_image_copy_plane(dst, -1 * ctx->reference->linesize[0],
-                             raw, ctx->tiles[i].w * 4,
-                             ctx->tiles[i].w * 4, ctx->tiles[i].h);
-         raw += ctx->tiles[i].w * 4 * ctx->tiles[i].h;
+                             raw, ctx->tiles[i].w * ctx->component_size,
+                             ctx->tiles[i].w * ctx->component_size,
+                             ctx->tiles[i].h);
+         raw += ctx->tiles[i].w * ctx->component_size * ctx->tiles[i].h;
      }
  
      /* Frame is ready to be output */