On Mon, Mar 17, 2014 at 02:29:08AM +0100, Vittorio Giovara wrote:
> From: Paul B Mahol <[email protected]>
> 
> Signed-off-by: Vittorio Giovara <[email protected]>
> ---
>  Changelog                |   1 +
>  doc/general.texi         |   4 +
>  libavcodec/Makefile      |   2 +
>  libavcodec/allcodecs.c   |   2 +
>  libavcodec/avcodec.h     |   2 +
>  libavcodec/codec_desc.c  |  14 ++
>  libavcodec/paf.c         | 453 
> +++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/version.h     |   4 +-
>  libavformat/Makefile     |   1 +
>  libavformat/allformats.c |   1 +
>  libavformat/paf.c        | 262 +++++++++++++++++++++++++++
>  libavformat/version.h    |   2 +-
>  12 files changed, 745 insertions(+), 3 deletions(-)
>  create mode 100644 libavcodec/paf.c
>  create mode 100644 libavformat/paf.c
> 
> diff --git a/libavcodec/paf.c b/libavcodec/paf.c
> new file mode 100644
> index 0000000..ae85e19
> --- /dev/null
> +++ b/libavcodec/paf.c
> @@ -0,0 +1,453 @@
> +/*
> + * Packed Animation File video and audio decoder
> + * Copyright (c) 2012 Paul B Mahol
> + *
> + * This file is part of Libav.
> + *
> + * Libav 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,
> + * 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
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
> USA
> + */
> +
> +#include "libavutil/intreadwrite.h"
> +
> +#include "avcodec.h"
> +#include "bytestream.h"
> +#include "copy_block.h"
> +#include "internal.h"
> +
> +#define PAF_SOUND_SAMPLES     2205
> +#define PAF_SOUND_FRAME_SIZE  ((256 + PAF_SOUND_SAMPLES) * 2)
> +
> +static const uint8_t block_sequences[16][8] = {
> +    { 0, 0, 0, 0, 0, 0, 0, 0 },
> +    { 2, 0, 0, 0, 0, 0, 0, 0 },
> +    { 5, 7, 0, 0, 0, 0, 0, 0 },
> +    { 5, 0, 0, 0, 0, 0, 0, 0 },
> +    { 6, 0, 0, 0, 0, 0, 0, 0 },
> +    { 5, 7, 5, 7, 0, 0, 0, 0 },
> +    { 5, 7, 5, 0, 0, 0, 0, 0 },
> +    { 5, 7, 6, 0, 0, 0, 0, 0 },
> +    { 5, 5, 0, 0, 0, 0, 0, 0 },
> +    { 3, 0, 0, 0, 0, 0, 0, 0 },
> +    { 6, 6, 0, 0, 0, 0, 0, 0 },
> +    { 2, 4, 0, 0, 0, 0, 0, 0 },
> +    { 2, 4, 5, 7, 0, 0, 0, 0 },
> +    { 2, 4, 5, 0, 0, 0, 0, 0 },
> +    { 2, 4, 6, 0, 0, 0, 0, 0 },
> +    { 2, 4, 5, 7, 5, 7, 0, 0 }
> +};
> +
> +typedef struct PAFVideoDecContext {
> +    AVFrame  *pic;
> +    GetByteContext gb;
> +
> +    int current_frame;
> +    uint8_t *frame[4];
> +    int frame_size;
> +    int video_size;
> +
> +    uint8_t *opcodes;
> +} PAFVideoDecContext;
> +
> +static av_cold int paf_vid_close(AVCodecContext *avctx)
> +{
> +    PAFVideoDecContext *c = avctx->priv_data;
> +    int i;
> +
> +    av_frame_free(&c->pic);
> +
> +    for (i = 0; i < 4; i++)
> +        av_freep(&c->frame[i]);
> +
> +    return 0;
> +}
> +
> +static av_cold int paf_vid_init(AVCodecContext *avctx)
> +{
> +    PAFVideoDecContext *c = avctx->priv_data;
> +    int i;
> +
> +    if (avctx->height & 3 || avctx->width & 3) {
> +        av_log(avctx, AV_LOG_ERROR, "width and height must be multiplies of 
> 4\n");

"must be multiple of 4"

> +        return AVERROR_INVALIDDATA;
> +    }
> +
> +    avctx->pix_fmt = AV_PIX_FMT_PAL8;
> +
> +    c->pic = av_frame_alloc();
> +    if (!c->pic)
> +        return AVERROR(ENOMEM);
> +
> +    c->frame_size = FFALIGN(avctx->height, 256) * avctx->width;

wow

> +    c->video_size = avctx->height * avctx->width;
> +    for (i = 0; i < 4; i++) {
> +        c->frame[i] = av_mallocz(c->frame_size);
> +        if (!c->frame[i]) {
> +            paf_vid_close(avctx);
> +            return AVERROR(ENOMEM);
> +        }
> +    }
> +
> +    return 0;
> +}
> +
> +static int get_video_page_offset(AVCodecContext *avctx, uint8_t a, uint8_t b)
> +{
> +    int x, y;
> +
> +    x = b & 0x7F;
> +    y = ((a & 0x3F) << 1) | (b >> 7 & 1);
> +
> +    return y * 2 * avctx->width + x * 2;
> +}
> +
> +static void copy4h(AVCodecContext *avctx, uint8_t *dst)
> +{
> +    PAFVideoDecContext *c = avctx->priv_data;
> +    int i;
> +
> +    for (i = 0; i < 4; i++) {
> +        bytestream2_get_buffer(&c->gb, dst, 4);
> +        dst += avctx->width;
> +    }
> +}

this is stupid - just pass needed variables (like width) instead of
AVCodecContext*

> +
> +static void copy_color_mask(AVCodecContext *avctx, uint8_t mask, uint8_t 
> *dst, uint8_t color)
> +{
> +    int i;
> +
> +    for (i = 0; i < 4; i++) {
> +        if ((mask >> 4) & (1 << (3 - i)))
> +            dst[i] = color;
> +        if ((mask & 15) & (1 << (3 - i)))
> +            dst[avctx->width + i] = color;
> +    }
> +}
> +
> +static void copy_src_mask(AVCodecContext *avctx, uint8_t mask, uint8_t *dst, 
> const uint8_t *src)
> +{
> +    int i;
> +
> +    for (i = 0; i < 4; i++) {
> +        if ((mask >> 4) & (1 << (3 - i)))
> +            dst[i] = src[i];
> +        if ((mask & 15) & (1 << (3 - i)))
> +            dst[avctx->width + i] = src[avctx->width + i];
> +    }
> +}

ditto plus stupid condition
if (mask & (1 << 7 - i))
    dst[i] = src[i];
if (mask & (1 << 3 - i))
    dst[stride + i] = ...

is clearer

> +static int decode_0(AVCodecContext *avctx, uint8_t code, uint8_t *pkt)
> +{
> +    PAFVideoDecContext *c = avctx->priv_data;
> +    uint32_t opcode_size, offset;
> +    uint8_t *dst, *dend, mask = 0, color = 0, a, b, p;
> +    const uint8_t *src, *send, *opcodes;
> +    int i, j, x = 0;
> +
> +    i = bytestream2_get_byte(&c->gb);
> +    if (i) {
> +        if (code & 0x10) {
> +            int align;
> +
> +            align = bytestream2_tell(&c->gb) & 3;
> +            if (align)
> +                bytestream2_skip(&c->gb, 4 - align);

bad variable name, "off" or "pos" or even "unalign" would be better

> +        }
> +        do {
> +            a      = bytestream2_get_byte(&c->gb);
> +            b      = bytestream2_get_byte(&c->gb);
> +            p      = (a & 0xC0) >> 6;
> +            dst    = c->frame[p] + get_video_page_offset(avctx, a, b);
> +            dend   = c->frame[p] + c->frame_size;
> +            offset = (b & 0x7F) * 2;
> +            j      = bytestream2_get_le16(&c->gb) + offset;

from the code it looks like you deal with packed 16-bit variable, so probably
this will be clearer:

val = bytestream2_get_be16(&c->gb);
page = val >> 14;
x    = (val & 0x7F) * 2;
y    = ((val >> 7) & 0x7F) * 2;

dst = c->frame[page] + x + y * avctx->width;

> +
> +            do {
> +                offset++;
> +                if (dst + 3 * avctx->width + 4 > dend)
> +                    return AVERROR_INVALIDDATA;
> +                copy4h(avctx, dst);

despite the name it reads 4x4 block

> +                if ((offset & 0x3F) == 0)
> +                    dst += avctx->width * 3;
> +                dst += 4;
> +            } while (offset < j);
> +        } while (--i);
> +    }
> +
> +    dst  = c->frame[c->current_frame];
> +    dend = c->frame[c->current_frame] + c->frame_size;
> +    do {
> +        a    = bytestream2_get_byte(&c->gb);
> +        b    = bytestream2_get_byte(&c->gb);
> +        p    = (a & 0xC0) >> 6;
> +        src  = c->frame[p] + get_video_page_offset(avctx, a, b);
> +        send = c->frame[p] + c->frame_size;
> +        if ((src + 3 * avctx->width + 4 > send) ||
> +            (dst + 3 * avctx->width + 4 > dend))
> +            return AVERROR_INVALIDDATA;
> +        copy_block4(dst, src, avctx->width, avctx->width, 4);
> +        i++;
> +        if ((i & 0x3F) == 0)
> +            dst += avctx->width * 3;
> +        dst += 4;
> +    } while (i < c->video_size / 16);
> +
> +    opcode_size = bytestream2_get_le16(&c->gb);
> +    bytestream2_skip(&c->gb, 2);
> +
> +    if (bytestream2_get_bytes_left(&c->gb) < opcode_size)
> +        return AVERROR_INVALIDDATA;
> +
> +    opcodes = pkt + bytestream2_tell(&c->gb);
> +    bytestream2_skipu(&c->gb, opcode_size);
> +
> +    dst = c->frame[c->current_frame];
> +
> +    for (i = 0; i < avctx->height; i += 4, dst += avctx->width * 3)
> +        for (j = 0; j < avctx->width; j += 4, dst += 4) {
> +            int opcode, k = 0;
> +
> +            if (x > opcode_size)
> +                return AVERROR_INVALIDDATA;
> +            if (j & 4) {
> +                opcode = opcodes[x] & 15;
> +                x++;
> +            } else {
> +                opcode = opcodes[x] >> 4;
> +            }
> +
> +            while (block_sequences[opcode][k]) {
> +                offset = avctx->width * 2;
> +                code   = block_sequences[opcode][k++];
> +
> +                switch (code) {
> +                case 2:
> +                    offset = 0;
> +                case 3:
> +                    color = bytestream2_get_byte(&c->gb);
> +                case 4:
> +                    mask = bytestream2_get_byte(&c->gb);
> +                    copy_color_mask(avctx, mask, dst + offset, color);
> +                    break;
> +                case 5:
> +                    offset = 0;
> +                case 6:
> +                    a    = bytestream2_get_byte(&c->gb);
> +                    b    = bytestream2_get_byte(&c->gb);
> +                    p    = (a & 0xC0) >> 6;
> +                    src  = c->frame[p] + get_video_page_offset(avctx, a, b);
> +                    send = c->frame[p] + c->frame_size;
> +                case 7:
> +                    if (src + offset + avctx->width + 4 > send)
> +                        return AVERROR_INVALIDDATA;
> +                    mask = bytestream2_get_byte(&c->gb);
> +                    copy_src_mask(avctx, mask, dst + offset, src + offset);
> +                    break;
> +                }
> +            }
> +        }
> +
> +    return 0;
> +}
> +
> +static int paf_vid_decode(AVCodecContext *avctx, void *data,
> +                          int *got_frame, AVPacket *pkt)
> +{
> +    PAFVideoDecContext *c = avctx->priv_data;
> +    uint8_t code, *dst, *src, *end;
> +    int i, frame, ret;
> +
> +    if ((ret = ff_reget_buffer(avctx, c->pic)) < 0)
> +        return ret;
> +
> +    bytestream2_init(&c->gb, pkt->data, pkt->size);
> +
> +    code = bytestream2_get_byte(&c->gb);
> +    if (code & 0x20) {

those flags could get some names

> +        for (i = 0; i < 4; i++)
> +            memset(c->frame[i], 0, c->frame_size);
> +
> +        memset(c->pic->data[1], 0, AVPALETTE_SIZE);
> +        c->current_frame  = 0;
> +        c->pic->key_frame = 1;
> +        c->pic->pict_type = AV_PICTURE_TYPE_I;
> +    } else {
> +        c->pic->key_frame = 0;
> +        c->pic->pict_type = AV_PICTURE_TYPE_P;
> +    }
> +
> +    if (code & 0x40) {
> +        uint32_t *out = (uint32_t *)c->pic->data[1];
> +        int index, count;
> +
> +        index = bytestream2_get_byte(&c->gb);
> +        count = bytestream2_get_byte(&c->gb) + 1;
> +
> +        if (index + count > 256)
> +            return AVERROR_INVALIDDATA;
> +        if (bytestream2_get_bytes_left(&c->gb) < 3 * count)
> +            return AVERROR_INVALIDDATA;
> +
> +        out += index;
> +        for (i = 0; i < count; i++) {
> +            unsigned r, g, b;
> +
> +            r = bytestream2_get_byteu(&c->gb);
> +            r = r << 2 | r >> 4;
> +            g = bytestream2_get_byteu(&c->gb);
> +            g = g << 2 | g >> 4;
> +            b = bytestream2_get_byteu(&c->gb);
> +            b = b << 2 | b >> 4;
> +            *out++ = 0xFFU << 24 | r << 16 | g << 8 | b;

whatever Diego says - parentheses please

> +        }
> +        c->pic->palette_has_changed = 1;
> +    }
> +
> +    switch (code & 0x0F) {
> +    case 0:
> +        if ((ret = decode_0(avctx, code, pkt->data)) < 0)
> +            return ret;
> +        break;
> +    case 1:
> +        dst = c->frame[c->current_frame];
> +        bytestream2_skip(&c->gb, 2);
> +        if (bytestream2_get_bytes_left(&c->gb) < c->video_size)
> +            return AVERROR_INVALIDDATA;
> +        bytestream2_get_bufferu(&c->gb, dst, c->video_size);
> +        break;
> +    case 2:
> +        frame = bytestream2_get_byte(&c->gb);
> +        if (frame > 3)
> +            return AVERROR_INVALIDDATA;
> +        if (frame != c->current_frame)
> +            memcpy(c->frame[c->current_frame], c->frame[frame], 
> c->frame_size);
> +        break;
> +    case 4:
> +        dst = c->frame[c->current_frame];
> +        end = dst + c->video_size;
> +
> +        bytestream2_skip(&c->gb, 2);
> +
> +        while (dst < end) {
> +            int8_t code;
> +            int count;
> +
> +            if (bytestream2_get_bytes_left(&c->gb) < 2)
> +                return AVERROR_INVALIDDATA;
> +
> +            code  = bytestream2_get_byteu(&c->gb);
> +            count = FFABS(code) + 1;
> +
> +            if (dst + count > end)
> +                return AVERROR_INVALIDDATA;
> +            if (code < 0)
> +                memset(dst, bytestream2_get_byteu(&c->gb), count);
> +            else
> +                bytestream2_get_buffer(&c->gb, dst, count);
> +            dst += count;
> +        }
> +        break;
> +    default:
> +        avpriv_request_sample(avctx, "unknown/invalid code");
> +        return AVERROR_INVALIDDATA;
> +    }
> +
> +    dst = c->pic->data[0];
> +    src = c->frame[c->current_frame];
> +    for (i = 0; i < avctx->height; i++) {
> +        memcpy(dst, src, avctx->width);
> +        dst += c->pic->linesize[0];
> +        src += avctx->width;
> +    }

'twould be much better to use aligned stride instead of avctx->width for
internal buffers and being able to use av_copy_plane() here

> +
> +    c->current_frame = (c->current_frame + 1) & 3;
> +    if ((ret = av_frame_ref(data, c->pic)) < 0)
> +        return ret;
> +
> +    *got_frame = 1;
> +
> +    return pkt->size;
> +}
> +
> +static av_cold int paf_aud_init(AVCodecContext *avctx)
> +{
> +    if (avctx->channels != 2) {
> +        av_log(avctx, AV_LOG_ERROR, "invalid number of channels\n");
> +        return AVERROR_INVALIDDATA;
> +    }
> +
> +    avctx->channel_layout = AV_CH_LAYOUT_STEREO;
> +    avctx->sample_fmt     = AV_SAMPLE_FMT_S16;
> +
> +    return 0;
> +}
> +
> +static int paf_aud_decode(AVCodecContext *avctx, void *data,
> +                          int *got_frame_ptr, AVPacket *pkt)
> +{
> +    AVFrame *frame = data;
> +    uint8_t *buf   = pkt->data;
> +    int16_t *output_samples;
> +    const uint8_t *t;
> +    int frames, ret, i, j, k;
> +
> +    frames = pkt->size / PAF_SOUND_FRAME_SIZE;
> +    if (frames < 1)
> +        return AVERROR_INVALIDDATA;
> +
> +    frame->nb_samples = PAF_SOUND_SAMPLES * frames;
> +    if ((ret = ff_get_buffer(avctx, frame, 0)) < 0)
> +        return ret;
> +
> +    output_samples = (int16_t *)frame->data[0];
> +    for (i = 0; i < frames; i++) {
> +        t = buf + 256 * sizeof(uint16_t);
> +        for (j = 0; j < PAF_SOUND_SAMPLES; j++)
> +            for (k = 0; k < 2; k++) {
> +                *output_samples++ = AV_RL16(buf + *t * 2);
> +                t++;
> +            }

would be less WTFy if someone gave proper variable names here
Or read codebook samples into local array once and used it instead.

> +        buf += PAF_SOUND_FRAME_SIZE;
> +    }
> +
> +    *got_frame_ptr = 1;
> +
> +    return pkt->size;
> +}
> +
> +#if CONFIG_PAF_VIDEO_DECODER
> +AVCodec ff_paf_video_decoder = {
> +    .name           = "paf_video",
> +    .long_name      = NULL_IF_CONFIG_SMALL("Amazing Studio Packed Animation 
> File Video"),
> +    .type           = AVMEDIA_TYPE_VIDEO,
> +    .id             = AV_CODEC_ID_PAF_VIDEO,
> +    .priv_data_size = sizeof(PAFVideoDecContext),
> +    .init           = paf_vid_init,
> +    .close          = paf_vid_close,
> +    .decode         = paf_vid_decode,
> +    .capabilities   = CODEC_CAP_DR1,
> +};
> +#endif
> +
> +#if CONFIG_PAF_AUDIO_DECODER
> +AVCodec ff_paf_audio_decoder = {
> +    .name         = "paf_audio",
> +    .long_name    = NULL_IF_CONFIG_SMALL("Amazing Studio Packed Animation 
> File Audio"),
> +    .type         = AVMEDIA_TYPE_AUDIO,
> +    .id           = AV_CODEC_ID_PAF_AUDIO,
> +    .init         = paf_aud_init,
> +    .decode       = paf_aud_decode,
> +    .capabilities = CODEC_CAP_DR1,
> +};
> +#endif
> diff --git a/libavcodec/version.h b/libavcodec/version.h
> index 5ab49d5..d2f80ad 100644
> --- a/libavcodec/version.h
> +++ b/libavcodec/version.h
> @@ -29,8 +29,8 @@
>  #include "libavutil/version.h"
>  
>  #define LIBAVCODEC_VERSION_MAJOR 55
> -#define LIBAVCODEC_VERSION_MINOR 34
> -#define LIBAVCODEC_VERSION_MICRO  1
> +#define LIBAVCODEC_VERSION_MINOR 35
> +#define LIBAVCODEC_VERSION_MICRO  0
>  
>  #define LIBAVCODEC_VERSION_INT  AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
>                                                 LIBAVCODEC_VERSION_MINOR, \
> diff --git a/libavformat/Makefile b/libavformat/Makefile
> index 5694314..6b7def5 100644
> --- a/libavformat/Makefile
> +++ b/libavformat/Makefile
> @@ -226,6 +226,7 @@ OBJS-$(CONFIG_OGG_MUXER)                 += oggenc.o \
>                                              vorbiscomment.o
>  OBJS-$(CONFIG_OMA_DEMUXER)               += omadec.o pcm.o oma.o
>  OBJS-$(CONFIG_OMA_MUXER)                 += omaenc.o rawenc.o oma.o 
> id3v2enc.o
> +OBJS-$(CONFIG_PAF_DEMUXER)               += paf.o
>  OBJS-$(CONFIG_PCM_ALAW_DEMUXER)          += pcmdec.o pcm.o
>  OBJS-$(CONFIG_PCM_ALAW_MUXER)            += pcmenc.o rawenc.o
>  OBJS-$(CONFIG_PCM_F32BE_DEMUXER)         += pcmdec.o pcm.o
> diff --git a/libavformat/allformats.c b/libavformat/allformats.c
> index 52a121e..f9f7c68 100644
> --- a/libavformat/allformats.c
> +++ b/libavformat/allformats.c
> @@ -176,6 +176,7 @@ void av_register_all(void)
>      REGISTER_DEMUXER (NUV,              nuv);
>      REGISTER_MUXDEMUX(OGG,              ogg);
>      REGISTER_MUXDEMUX(OMA,              oma);
> +    REGISTER_DEMUXER (PAF,              paf);
>      REGISTER_MUXDEMUX(PCM_ALAW,         pcm_alaw);
>      REGISTER_MUXDEMUX(PCM_MULAW,        pcm_mulaw);
>      REGISTER_MUXDEMUX(PCM_F64BE,        pcm_f64be);
> diff --git a/libavformat/paf.c b/libavformat/paf.c
> new file mode 100644
> index 0000000..96da8db
> --- /dev/null
> +++ b/libavformat/paf.c
> @@ -0,0 +1,262 @@
> +/*
> + * Packed Animation File demuxer
> + * Copyright (c) 2012 Paul B Mahol
> + *
> + * This file is part of Libav.
> + *
> + * Libav 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,
> + * 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
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
> USA
> + */
> +
> +#include "libavutil/channel_layout.h"
> +
> +#include "avformat.h"
> +#include "internal.h"
> +
> +#define MAGIC "Packed Animation File V1.0\n(c) 1992-96 Amazing 
> Studio\x0a\x1a"
> +#define PAF_SOUND_SAMPLES     2205
> +#define PAF_SOUND_FRAME_SIZE  ((256 + PAF_SOUND_SAMPLES) * 2)
> +
> +typedef struct {
> +    uint32_t buffer_size;
> +    uint32_t frame_blks;
> +    uint32_t nb_frames;
> +    uint32_t start_offset;
> +    uint32_t preload_count;
> +    uint32_t max_video_blks;
> +    uint32_t max_audio_blks;
> +
> +    uint32_t current_frame;
> +    uint32_t current_frame_count;
> +    uint32_t current_frame_block;
> +
> +    uint32_t *blocks_count_table;
> +    uint32_t *frames_offset_table;
> +    uint32_t *blocks_offset_table;
> +
> +    uint8_t  *video_frame;
> +    int video_size;
> +
> +    uint8_t  *audio_frame;
> +    uint8_t  *temp_audio_frame;
> +    int audio_size;
> +
> +    int got_audio;
> +} PAFDemuxContext;
> +
> +static int read_probe(AVProbeData *p)
> +{
> +    if ((p->buf_size >= strlen(MAGIC)) &&
> +        !memcmp(p->buf, MAGIC, strlen(MAGIC)))
> +        return AVPROBE_SCORE_MAX;
> +    return 0;
> +}
> +
> +static int read_close(AVFormatContext *s)
> +{
> +    PAFDemuxContext *p = s->priv_data;
> +
> +    av_freep(&p->blocks_count_table);
> +    av_freep(&p->frames_offset_table);
> +    av_freep(&p->blocks_offset_table);
> +    av_freep(&p->video_frame);
> +    av_freep(&p->audio_frame);
> +    av_freep(&p->temp_audio_frame);
> +
> +    return 0;
> +}
> +
> +static void read_table(AVFormatContext *s, uint32_t *table, uint32_t count)
> +{
> +    int i;
> +
> +    for (i = 0; i < count; i++)
> +        table[i] = avio_rl32(s->pb);
> +
> +    avio_skip(s->pb, 4 * (FFALIGN(count, 512) - count));
> +}
> +
> +static int read_header(AVFormatContext *s)
> +{
> +    PAFDemuxContext *p  = s->priv_data;
> +    AVIOContext     *pb = s->pb;
> +    AVStream        *ast, *vst;
> +    int ret = 0;
> +
> +    avio_skip(pb, 132);
> +
> +    vst = avformat_new_stream(s, 0);
> +    if (!vst)
> +        return AVERROR(ENOMEM);
> +
> +    vst->start_time = 0;
> +    vst->nb_frames  =
> +    vst->duration   =
> +    p->nb_frames = avio_rl32(pb);

valign

> +    avio_skip(pb, 4);
> +    vst->codec->width  = avio_rl32(pb);
> +    vst->codec->height = avio_rl32(pb);
> +    avio_skip(pb, 4);
> +    vst->codec->codec_type = AVMEDIA_TYPE_VIDEO;
> +    vst->codec->codec_tag  = 0;
> +    vst->codec->codec_id   = AV_CODEC_ID_PAF_VIDEO;
> +    avpriv_set_pts_info(vst, 64, 1, 10);
> +
> +    ast = avformat_new_stream(s, 0);
> +    if (!ast)
> +        return AVERROR(ENOMEM);
> +
> +    ast->start_time            = 0;
> +    ast->codec->codec_type     = AVMEDIA_TYPE_AUDIO;
> +    ast->codec->codec_tag      = 0;
> +    ast->codec->codec_id       = AV_CODEC_ID_PAF_AUDIO;
> +    ast->codec->channels       = 2;
> +    ast->codec->channel_layout = AV_CH_LAYOUT_STEREO;
> +    ast->codec->sample_rate    = 22050;
> +    avpriv_set_pts_info(ast, 64, 1, 22050);
> +
> +    p->buffer_size    = avio_rl32(pb);
> +    p->preload_count  = avio_rl32(pb);
> +    p->frame_blks     = avio_rl32(pb);
> +    p->start_offset   = avio_rl32(pb);
> +    p->max_video_blks = avio_rl32(pb);
> +    p->max_audio_blks = avio_rl32(pb);
> +    if (p->buffer_size    < 175  ||
> +        p->max_audio_blks < 2    ||
> +        p->max_video_blks < 1    ||
> +        p->frame_blks     < 1    ||
> +        p->nb_frames      < 1    ||
> +        p->preload_count  < 1    ||
> +        p->buffer_size    > 2048 ||
> +        p->max_video_blks > 2048 ||
> +        p->max_audio_blks > 2048 ||
> +        p->nb_frames      > INT_MAX / sizeof(uint32_t) ||
> +        p->frame_blks     > INT_MAX / sizeof(uint32_t))
> +        return AVERROR_INVALIDDATA;
> +
> +    p->blocks_count_table  = av_mallocz(p->nb_frames * sizeof(uint32_t));
> +    p->frames_offset_table = av_mallocz(p->nb_frames * sizeof(uint32_t));
> +    p->blocks_offset_table = av_mallocz(p->frame_blks * sizeof(uint32_t));

sizeof(*p->member)

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to