[FFmpeg-devel] [PATCH 4/4] wavpackenc: reset trailer info on block encoding

Paul B Mahol onemda at gmail.com
Fri Aug 22 14:28:38 CEST 2014


On 8/22/14, Christophe Gisquet <christophe.gisquet at gmail.com> wrote:
> In some cases, in particular if several blocks are needed because of
> the channel layout (e.g. 2.1), the information used to write the
> trailing bits terminating the sample data was not reset.
>
> This would cause potential desync on the decoder, although decoded
> samples were actually mostly fine.
>
> Fixes ticket #3879.
> ---
>  libavcodec/wavpackenc.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/libavcodec/wavpackenc.c b/libavcodec/wavpackenc.c
> index 299a035..63971c6 100644
> --- a/libavcodec/wavpackenc.c
> +++ b/libavcodec/wavpackenc.c
> @@ -2487,6 +2487,9 @@ static int wavpack_encode_block(WavPackEncodeContext
> *s,
>      struct Decorr *dpp;
>      PutByteContext pb;
>
> +    if (s->flags & WV_MONO_DATA) {
> +        CLEAR(s->w);
> +    }
>      if (!(s->flags & WV_MONO) && s->optimize_mono) {
>          int32_t lor = 0, diff = 0;
>
> --
> 1.9.2.msysgit.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

lgtm


More information about the ffmpeg-devel mailing list