[FFmpeg-devel] [PATCH] avcodec/packet: initialize payload size to zero

mypopy at gmail.com mypopy at gmail.com
Wed Apr 3 09:09:33 EEST 2019


On Wed, Apr 3, 2019 at 11:57 AM Andriy Gelman <andriy.gelman at gmail.com> wrote:
>
> From: Andriy Gelman <andriy.gelman at gmail.com>
>
> ---
>  libavcodec/avpacket.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c
> index 8f0603df78..6ae092ed31 100644
> --- a/libavcodec/avpacket.c
> +++ b/libavcodec/avpacket.c
> @@ -44,6 +44,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>      pkt->flags                = 0;
>      pkt->stream_index         = 0;
>      pkt->buf                  = NULL;
> +    pkt->size                 = 0;
>      pkt->side_data            = NULL;
>      pkt->side_data_elems      = 0;
>  }

I think you can refer to the link about this question:
https://www.mail-archive.com/ffmpeg-devel@ffmpeg.org/msg64312.html

In this list, As Hendrik Leppkes likes to point out, " Code can
currently rely on size/data to not be modified - and I'm sure there
are also places where it does."

So I think we can't change as this patch if can't find a way to fight
the API break. Thanks.


More information about the ffmpeg-devel mailing list