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
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
