[FFmpeg-devel] [PATCH] avcodec/decode: fix warning when decoding pseudo paletted formats

Michael Niedermayer michael at niedermayer.cc
Wed Apr 4 22:42:31 EEST 2018


On Wed, Apr 04, 2018 at 06:07:20PM +0200, wm4 wrote:
> The pseudo palette allocation is optional now. But if it's still
> allocated (like the internal get_buffer2 implementation does, for
> compatibility), it shouldn't print a warning.
> ---
>  libavcodec/decode.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/libavcodec/decode.c b/libavcodec/decode.c
> index d883a5f9fc..421a8f1a35 100644
> --- a/libavcodec/decode.c
> +++ b/libavcodec/decode.c
> @@ -1779,6 +1779,8 @@ static void validate_avframe_allocation(AVCodecContext *avctx, AVFrame *frame)
>          int flags = desc ? desc->flags : 0;
>          if (num_planes == 1 && (flags & AV_PIX_FMT_FLAG_PAL))
>              num_planes = 2;
> +        if ((flags & FF_PSEUDOPAL) && frame->data[1])
> +            num_planes = 2;
>          for (i = 0; i < num_planes; i++) {
>              av_assert0(frame->data[i]);
>          }

LGTM

thx


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180404/dcb61979/attachment.sig>


More information about the ffmpeg-devel mailing list