[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