Merge remote-tracking branch 'qatar/master'
authorMichael Niedermayer <michaelni@gmx.at>
Sun, 7 Oct 2012 09:23:29 +0000 (11:23 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Sun, 7 Oct 2012 09:28:38 +0000 (11:28 +0200)
* qatar/master:
  wmaenc: use float planar sample format
  (e)ac3enc: use planar sample format
  aacenc: use planar sample format
  adpcmenc: use planar sample format for adpcm_ima_wav and adpcm_ima_qt
  adpcmenc: move 'ch' variable to higher scope
  adpcmenc: fix 3 instances of variable shadowing
  adpcm_ima_wav: simplify encoding
  libvorbis: use planar sample format
  libmp3lame: use planar sample formats
  vorbisenc: use float planar sample format
  ffm: do not write or read the audio sample format
  parseutils: fix parsing of invalid alpha values
  doc/RELEASE_NOTES: update for the 9 release.
  smoothstreamingenc: Add a more verbose error message
  smoothstreamingenc: Ignore the return value from mkdir
  smoothstreamingenc: Try writing a manifest when opening the muxer
  smoothstreamingenc: Move the output_chunk_list and write_manifest functions up
  smoothstreamingenc: Properly return errors from ism_flush to the caller
  smoothstreamingenc: Check the output UrlContext before accessing it

Conflicts:
doc/RELEASE_NOTES
libavcodec/aacenc.c
libavcodec/ac3enc_template.c
libavcodec/wmaenc.c
tests/ref/lavf/ffm

Merged-by: Michael Niedermayer <michaelni@gmx.at>
17 files changed:
1  2 
libavcodec/aacenc.c
libavcodec/ac3enc_fixed.c
libavcodec/ac3enc_float.c
libavcodec/ac3enc_template.c
libavcodec/adpcmenc.c
libavcodec/eac3enc.c
libavcodec/libmp3lame.c
libavcodec/libvorbisenc.c
libavcodec/vorbisenc.c
libavcodec/wma.c
libavcodec/wma.h
libavcodec/wmaenc.c
libavformat/ffmdec.c
libavformat/ffmenc.c
libavformat/smoothstreamingenc.c
libavutil/parseutils.c
tests/ref/lavf/ffm

@@@ -479,17 -479,17 +479,17 @@@ static void put_bitstream_info(AVCodecC
  }
  
  /*
-  * Deinterleave input samples.
+  * Copy input samples.
 - * Channels are reordered from Libav's default order to AAC order.
 + * Channels are reordered from libavcodec's default order to AAC order.
   */
- static void deinterleave_input_samples(AACEncContext *s, const AVFrame *frame)
+ static void copy_input_samples(AACEncContext *s, const AVFrame *frame)
  {
-     int ch, i;
-     const int sinc = s->channels;
-     const uint8_t *channel_map = aac_chan_maps[sinc - 1];
+     int ch;
+     int end = 2048 + (frame ? frame->nb_samples : 0);
+     const uint8_t *channel_map = aac_chan_maps[s->channels - 1];
  
-     /* deinterleave and remap input samples */
-     for (ch = 0; ch < sinc; ch++) {
+     /* copy and remap input samples */
+     for (ch = 0; ch < s->channels; ch++) {
          /* copy last 1024 samples of previous frame to the start of the current frame */
          memcpy(&s->planar_samples[ch][1024], &s->planar_samples[ch][2048], 1024 * sizeof(s->planar_samples[0][0]));
  
@@@ -824,10 -821,9 +821,10 @@@ AVCodec ff_aac_encoder = 
      .init           = aac_encode_init,
      .encode2        = aac_encode_frame,
      .close          = aac_encode_end,
 +    .supported_samplerates = avpriv_mpeg4audio_sample_rates,
      .capabilities   = CODEC_CAP_SMALL_LAST_FRAME | CODEC_CAP_DELAY |
                        CODEC_CAP_EXPERIMENTAL,
-     .sample_fmts    = (const enum AVSampleFormat[]){ AV_SAMPLE_FMT_FLT,
+     .sample_fmts    = (const enum AVSampleFormat[]){ AV_SAMPLE_FMT_FLTP,
                                                       AV_SAMPLE_FMT_NONE },
      .long_name      = NULL_IF_CONFIG_SMALL("AAC (Advanced Audio Coding)"),
      .priv_class     = &aacenc_class,
Simple merge
Simple merge
@@@ -4,20 -4,20 +4,20 @@@
   * Copyright (c) 2006-2011 Justin Ruggles <justin.ruggles@gmail.com>
   * Copyright (c) 2006-2010 Prakash Punnoor <prakash@punnoor.de>
   *
-- * 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
   */
  
@@@ -68,19 -65,15 +68,15 @@@ alloc_fail
  
  
  /*
-  * Deinterleave input samples.
+  * Copy input samples.
 - * Channels are reordered from Libav's default order to AC-3 order.
 + * Channels are reordered from FFmpeg's default order to AC-3 order.
   */
- static void deinterleave_input_samples(AC3EncodeContext *s,
-                                        const SampleType *samples)
+ static void copy_input_samples(AC3EncodeContext *s, SampleType **samples)
  {
-     int ch, i;
+     int ch;
  
-     /* deinterleave and remap input samples */
+     /* copy and remap input samples */
      for (ch = 0; ch < s->channels; ch++) {
-         const SampleType *sptr;
-         int sinc;
          /* copy last 256 samples of previous frame to the start of the current frame */
          memcpy(&s->planar_samples[ch][0], &s->planar_samples[ch][AC3_BLOCK_SIZE * s->num_blocks],
                 AC3_BLOCK_SIZE * sizeof(s->planar_samples[0][0]));
Simple merge
Simple merge
Simple merge
index b042e0c,0000000..6064dd5
mode 100644,000000..100644
--- /dev/null
@@@ -1,391 -1,0 +1,389 @@@
-         const float *audio = (const float *)frame->data[0];
 +/*
 + * Copyright (c) 2002 Mark Hills <mark@pogo.org.uk>
 + *
 + * This file is part of FFmpeg.
 + *
 + * 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.
 + *
 + * 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 FFmpeg; if not, write to the Free Software
 + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 + */
 +
 +#include <vorbis/vorbisenc.h>
 +
 +#include "libavutil/avassert.h"
 +#include "libavutil/fifo.h"
 +#include "libavutil/opt.h"
 +#include "avcodec.h"
 +#include "audio_frame_queue.h"
 +#include "internal.h"
 +#include "vorbis.h"
 +#include "vorbis_parser.h"
 +
 +
 +/* Number of samples the user should send in each call.
 + * This value is used because it is the LCD of all possible frame sizes, so
 + * an output packet will always start at the same point as one of the input
 + * packets.
 + */
 +#define OGGVORBIS_FRAME_SIZE 64
 +
 +#define BUFFER_SIZE (1024 * 64)
 +
 +typedef struct OggVorbisEncContext {
 +    AVClass *av_class;                  /**< class for AVOptions            */
 +    AVFrame frame;
 +    vorbis_info vi;                     /**< vorbis_info used during init   */
 +    vorbis_dsp_state vd;                /**< DSP state used for analysis    */
 +    vorbis_block vb;                    /**< vorbis_block used for analysis */
 +    AVFifoBuffer *pkt_fifo;             /**< output packet buffer           */
 +    int eof;                            /**< end-of-file flag               */
 +    int dsp_initialized;                /**< vd has been initialized        */
 +    vorbis_comment vc;                  /**< VorbisComment info             */
 +    double iblock;                      /**< impulse block bias option      */
 +    VorbisParseContext vp;              /**< parse context to get durations */
 +    AudioFrameQueue afq;                /**< frame queue for timestamps     */
 +} OggVorbisEncContext;
 +
 +static const AVOption options[] = {
 +    { "iblock", "Sets the impulse block bias", offsetof(OggVorbisEncContext, iblock), AV_OPT_TYPE_DOUBLE, { .dbl = 0 }, -15, 0, AV_OPT_FLAG_AUDIO_PARAM | AV_OPT_FLAG_ENCODING_PARAM },
 +    { NULL }
 +};
 +
 +static const AVCodecDefault defaults[] = {
 +    { "b",  "0" },
 +    { NULL },
 +};
 +
 +static const AVClass class = {
 +    .class_name = "libvorbis",
 +    .item_name  = av_default_item_name,
 +    .option     = options,
 +    .version    = LIBAVUTIL_VERSION_INT,
 +};
 +
 +static int vorbis_error_to_averror(int ov_err)
 +{
 +    switch (ov_err) {
 +    case OV_EFAULT: return AVERROR_BUG;
 +    case OV_EINVAL: return AVERROR(EINVAL);
 +    case OV_EIMPL:  return AVERROR(EINVAL);
 +    default:        return AVERROR_UNKNOWN;
 +    }
 +}
 +
 +static av_cold int oggvorbis_init_encoder(vorbis_info *vi,
 +                                          AVCodecContext *avctx)
 +{
 +    OggVorbisEncContext *s = avctx->priv_data;
 +    double cfreq;
 +    int ret;
 +
 +    if (avctx->flags & CODEC_FLAG_QSCALE || !avctx->bit_rate) {
 +        /* variable bitrate
 +         * NOTE: we use the oggenc range of -1 to 10 for global_quality for
 +         *       user convenience, but libvorbis uses -0.1 to 1.0.
 +         */
 +        float q = avctx->global_quality / (float)FF_QP2LAMBDA;
 +        /* default to 3 if the user did not set quality or bitrate */
 +        if (!(avctx->flags & CODEC_FLAG_QSCALE))
 +            q = 3.0;
 +        if ((ret = vorbis_encode_setup_vbr(vi, avctx->channels,
 +                                           avctx->sample_rate,
 +                                           q / 10.0)))
 +            goto error;
 +    } else {
 +        int minrate = avctx->rc_min_rate > 0 ? avctx->rc_min_rate : -1;
 +        int maxrate = avctx->rc_max_rate > 0 ? avctx->rc_max_rate : -1;
 +
 +        /* average bitrate */
 +        if ((ret = vorbis_encode_setup_managed(vi, avctx->channels,
 +                                               avctx->sample_rate, maxrate,
 +                                               avctx->bit_rate, minrate)))
 +            goto error;
 +
 +        /* variable bitrate by estimate, disable slow rate management */
 +        if (minrate == -1 && maxrate == -1)
 +            if ((ret = vorbis_encode_ctl(vi, OV_ECTL_RATEMANAGE2_SET, NULL)))
 +                goto error; /* should not happen */
 +    }
 +
 +    /* cutoff frequency */
 +    if (avctx->cutoff > 0) {
 +        cfreq = avctx->cutoff / 1000.0;
 +        if ((ret = vorbis_encode_ctl(vi, OV_ECTL_LOWPASS_SET, &cfreq)))
 +            goto error; /* should not happen */
 +    }
 +
 +    /* impulse block bias */
 +    if (s->iblock) {
 +        if ((ret = vorbis_encode_ctl(vi, OV_ECTL_IBLOCK_SET, &s->iblock)))
 +            goto error;
 +    }
 +
 +    if (avctx->channels == 3 &&
 +            avctx->channel_layout != (AV_CH_LAYOUT_STEREO|AV_CH_FRONT_CENTER) ||
 +        avctx->channels == 4 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_2_2 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_QUAD ||
 +        avctx->channels == 5 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_5POINT0 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_5POINT0_BACK ||
 +        avctx->channels == 6 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_5POINT1 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_5POINT1_BACK ||
 +        avctx->channels == 7 &&
 +            avctx->channel_layout != (AV_CH_LAYOUT_5POINT1|AV_CH_BACK_CENTER) ||
 +        avctx->channels == 8 &&
 +            avctx->channel_layout != AV_CH_LAYOUT_7POINT1) {
 +        if (avctx->channel_layout) {
 +            char name[32];
 +            av_get_channel_layout_string(name, sizeof(name), avctx->channels,
 +                                         avctx->channel_layout);
 +            av_log(avctx, AV_LOG_ERROR, "%s not supported by Vorbis: "
 +                                             "output stream will have incorrect "
 +                                             "channel layout.\n", name);
 +        } else {
 +            av_log(avctx, AV_LOG_WARNING, "No channel layout specified. The encoder "
 +                                               "will use Vorbis channel layout for "
 +                                               "%d channels.\n", avctx->channels);
 +        }
 +    }
 +
 +    if ((ret = vorbis_encode_setup_init(vi)))
 +        goto error;
 +
 +    return 0;
 +error:
 +    return vorbis_error_to_averror(ret);
 +}
 +
 +/* How many bytes are needed for a buffer of length 'l' */
 +static int xiph_len(int l)
 +{
 +    return 1 + l / 255 + l;
 +}
 +
 +static av_cold int oggvorbis_encode_close(AVCodecContext *avctx)
 +{
 +    OggVorbisEncContext *s = avctx->priv_data;
 +
 +    /* notify vorbisenc this is EOF */
 +    if (s->dsp_initialized)
 +        vorbis_analysis_wrote(&s->vd, 0);
 +
 +    vorbis_block_clear(&s->vb);
 +    vorbis_dsp_clear(&s->vd);
 +    vorbis_info_clear(&s->vi);
 +
 +    av_fifo_free(s->pkt_fifo);
 +    ff_af_queue_close(&s->afq);
 +#if FF_API_OLD_ENCODE_AUDIO
 +    av_freep(&avctx->coded_frame);
 +#endif
 +    av_freep(&avctx->extradata);
 +
 +    return 0;
 +}
 +
 +static av_cold int oggvorbis_encode_init(AVCodecContext *avctx)
 +{
 +    OggVorbisEncContext *s = avctx->priv_data;
 +    ogg_packet header, header_comm, header_code;
 +    uint8_t *p;
 +    unsigned int offset;
 +    int ret;
 +
 +    vorbis_info_init(&s->vi);
 +    if ((ret = oggvorbis_init_encoder(&s->vi, avctx))) {
 +        av_log(avctx, AV_LOG_ERROR, "encoder setup failed\n");
 +        goto error;
 +    }
 +    if ((ret = vorbis_analysis_init(&s->vd, &s->vi))) {
 +        av_log(avctx, AV_LOG_ERROR, "analysis init failed\n");
 +        ret = vorbis_error_to_averror(ret);
 +        goto error;
 +    }
 +    s->dsp_initialized = 1;
 +    if ((ret = vorbis_block_init(&s->vd, &s->vb))) {
 +        av_log(avctx, AV_LOG_ERROR, "dsp init failed\n");
 +        ret = vorbis_error_to_averror(ret);
 +        goto error;
 +    }
 +
 +    vorbis_comment_init(&s->vc);
 +    if (!(avctx->flags & CODEC_FLAG_BITEXACT))
 +        vorbis_comment_add_tag(&s->vc, "encoder", LIBAVCODEC_IDENT);
 +
 +    if ((ret = vorbis_analysis_headerout(&s->vd, &s->vc, &header, &header_comm,
 +                                         &header_code))) {
 +        ret = vorbis_error_to_averror(ret);
 +        goto error;
 +    }
 +
 +    avctx->extradata_size = 1 + xiph_len(header.bytes)      +
 +                                xiph_len(header_comm.bytes) +
 +                                header_code.bytes;
 +    p = avctx->extradata = av_malloc(avctx->extradata_size +
 +                                     FF_INPUT_BUFFER_PADDING_SIZE);
 +    if (!p) {
 +        ret = AVERROR(ENOMEM);
 +        goto error;
 +    }
 +    p[0]    = 2;
 +    offset  = 1;
 +    offset += av_xiphlacing(&p[offset], header.bytes);
 +    offset += av_xiphlacing(&p[offset], header_comm.bytes);
 +    memcpy(&p[offset], header.packet, header.bytes);
 +    offset += header.bytes;
 +    memcpy(&p[offset], header_comm.packet, header_comm.bytes);
 +    offset += header_comm.bytes;
 +    memcpy(&p[offset], header_code.packet, header_code.bytes);
 +    offset += header_code.bytes;
 +    av_assert0(offset == avctx->extradata_size);
 +
 +    if ((ret = avpriv_vorbis_parse_extradata(avctx, &s->vp)) < 0) {
 +        av_log(avctx, AV_LOG_ERROR, "invalid extradata\n");
 +        return ret;
 +    }
 +
 +    vorbis_comment_clear(&s->vc);
 +
 +    avctx->frame_size = OGGVORBIS_FRAME_SIZE;
 +    ff_af_queue_init(avctx, &s->afq);
 +
 +    s->pkt_fifo = av_fifo_alloc(BUFFER_SIZE);
 +    if (!s->pkt_fifo) {
 +        ret = AVERROR(ENOMEM);
 +        goto error;
 +    }
 +
 +#if FF_API_OLD_ENCODE_AUDIO
 +    avctx->coded_frame = avcodec_alloc_frame();
 +    if (!avctx->coded_frame) {
 +        ret = AVERROR(ENOMEM);
 +        goto error;
 +    }
 +#endif
 +
 +    return 0;
 +error:
 +    oggvorbis_encode_close(avctx);
 +    return ret;
 +}
 +
 +static int oggvorbis_encode_frame(AVCodecContext *avctx, AVPacket *avpkt,
 +                                  const AVFrame *frame, int *got_packet_ptr)
 +{
 +    OggVorbisEncContext *s = avctx->priv_data;
 +    ogg_packet op;
 +    int ret, duration;
 +
 +    /* send samples to libvorbis */
 +    if (frame) {
-             int i;
 +        const int samples = frame->nb_samples;
 +        float **buffer;
 +        int c, channels = s->vi.channels;
 +
 +        buffer = vorbis_analysis_buffer(&s->vd, samples);
 +        for (c = 0; c < channels; c++) {
-             for (i = 0; i < samples; i++)
-                 buffer[c][i] = audio[i * channels + co];
 +            int co = (channels > 8) ? c :
 +                     ff_vorbis_encoding_channel_layout_offsets[channels - 1][c];
-     .sample_fmts    = (const enum AVSampleFormat[]) { AV_SAMPLE_FMT_FLT,
++            memcpy(buffer[c], frame->extended_data[co],
++                   samples * sizeof(*buffer[c]));
 +        }
 +        if ((ret = vorbis_analysis_wrote(&s->vd, samples)) < 0) {
 +            av_log(avctx, AV_LOG_ERROR, "error in vorbis_analysis_wrote()\n");
 +            return vorbis_error_to_averror(ret);
 +        }
 +        if ((ret = ff_af_queue_add(&s->afq, frame) < 0))
 +            return ret;
 +    } else {
 +        if (!s->eof)
 +            if ((ret = vorbis_analysis_wrote(&s->vd, 0)) < 0) {
 +                av_log(avctx, AV_LOG_ERROR, "error in vorbis_analysis_wrote()\n");
 +                return vorbis_error_to_averror(ret);
 +            }
 +        s->eof = 1;
 +    }
 +
 +    /* retrieve available packets from libvorbis */
 +    while ((ret = vorbis_analysis_blockout(&s->vd, &s->vb)) == 1) {
 +        if ((ret = vorbis_analysis(&s->vb, NULL)) < 0)
 +            break;
 +        if ((ret = vorbis_bitrate_addblock(&s->vb)) < 0)
 +            break;
 +
 +        /* add any available packets to the output packet buffer */
 +        while ((ret = vorbis_bitrate_flushpacket(&s->vd, &op)) == 1) {
 +            if (av_fifo_space(s->pkt_fifo) < sizeof(ogg_packet) + op.bytes) {
 +                av_log(avctx, AV_LOG_ERROR, "packet buffer is too small\n");
 +                return AVERROR_BUG;
 +            }
 +            av_fifo_generic_write(s->pkt_fifo, &op, sizeof(ogg_packet), NULL);
 +            av_fifo_generic_write(s->pkt_fifo, op.packet, op.bytes, NULL);
 +        }
 +        if (ret < 0) {
 +            av_log(avctx, AV_LOG_ERROR, "error getting available packets\n");
 +            break;
 +        }
 +    }
 +    if (ret < 0) {
 +        av_log(avctx, AV_LOG_ERROR, "error getting available packets\n");
 +        return vorbis_error_to_averror(ret);
 +    }
 +
 +    /* check for available packets */
 +    if (av_fifo_size(s->pkt_fifo) < sizeof(ogg_packet))
 +        return 0;
 +
 +    av_fifo_generic_read(s->pkt_fifo, &op, sizeof(ogg_packet), NULL);
 +
 +    if ((ret = ff_alloc_packet2(avctx, avpkt, op.bytes)))
 +        return ret;
 +    av_fifo_generic_read(s->pkt_fifo, avpkt->data, op.bytes, NULL);
 +
 +    avpkt->pts = ff_samples_to_time_base(avctx, op.granulepos);
 +
 +    duration = avpriv_vorbis_parse_frame(&s->vp, avpkt->data, avpkt->size);
 +    if (duration > 0) {
 +        /* we do not know encoder delay until we get the first packet from
 +         * libvorbis, so we have to update the AudioFrameQueue counts */
 +        if (!avctx->delay) {
 +            avctx->delay              = duration;
 +            av_assert0(!s->afq.remaining_delay);
 +            s->afq.frames->duration  += duration;
 +            s->afq.frames->pts       -= duration;
 +            s->afq.remaining_samples += duration;
 +        }
 +        ff_af_queue_remove(&s->afq, duration, &avpkt->pts, &avpkt->duration);
 +    }
 +
 +    *got_packet_ptr = 1;
 +    return 0;
 +}
 +
 +AVCodec ff_libvorbis_encoder = {
 +    .name           = "libvorbis",
 +    .type           = AVMEDIA_TYPE_AUDIO,
 +    .id             = AV_CODEC_ID_VORBIS,
 +    .priv_data_size = sizeof(OggVorbisEncContext),
 +    .init           = oggvorbis_encode_init,
 +    .encode2        = oggvorbis_encode_frame,
 +    .close          = oggvorbis_encode_close,
 +    .capabilities   = CODEC_CAP_DELAY,
++    .sample_fmts    = (const enum AVSampleFormat[]) { AV_SAMPLE_FMT_FLTP,
 +                                                      AV_SAMPLE_FMT_NONE },
 +    .long_name      = NULL_IF_CONFIG_SMALL("libvorbis"),
 +    .priv_class     = &class,
 +    .defaults       = defaults,
 +};
Simple merge
Simple merge
Simple merge
@@@ -345,8 -350,7 +346,7 @@@ static int encode_superframe(AVCodecCon
                               const AVFrame *frame, int *got_packet_ptr)
  {
      WMACodecContext *s = avctx->priv_data;
-     const int16_t *samples = (const int16_t *)frame->data[0];
 -    int i, total_gain, ret;
 +    int i, total_gain, ret, error;
  
      s->block_len_bits= s->frame_len_bits; //required by non variable block len
      s->block_len = 1 << s->block_len_bits;
Simple merge
Simple merge
Simple merge
@@@ -394,14 -360,10 +394,14 @@@ int av_parse_color(uint8_t *rgba_color
          if (!strncmp(alpha_string, "0x", 2)) {
              alpha = strtoul(alpha_string, &tail, 16);
          } else {
 -            alpha = 255 * strtod(alpha_string, &tail);
 +            double norm_alpha = strtod(alpha_string, &tail);
 +            if (norm_alpha < 0.0 || norm_alpha > 1.0)
 +                alpha = 256;
 +            else
 +                alpha = 255 * norm_alpha;
          }
  
-         if (tail == alpha_string || *tail || alpha > 255) {
+         if (tail == alpha_string || *tail || alpha > 255 || alpha < 0) {
              av_log(log_ctx, AV_LOG_ERROR, "Invalid alpha value specifier '%s' in '%s'\n",
                     alpha_string, color_string);
              return AVERROR(EINVAL);
@@@ -1,3 -1,3 +1,3 @@@
- e6d8254af2b1ad1f58d60da4d80c6e96 *./tests/data/lavf/lavf.ffm
 -c5dcf5950031020864db57bbde0064df *./tests/data/lavf/lavf.ffm
++c76e8f9a9bcd04379dfa3239e272d049 *./tests/data/lavf/lavf.ffm
  376832 ./tests/data/lavf/lavf.ffm
 -./tests/data/lavf/lavf.ffm CRC=0x38388ba1
 +./tests/data/lavf/lavf.ffm CRC=0x5b136bb1