[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