[FFmpeg-devel] [PATCH 02/13] avformat/mux: Don't unnecessarily zero-initialize AVPacketList

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Tue Aug 13 05:47:15 EEST 2019


If no error occurs and this AVPacketList is used at all, its packet
substructure will be overwritten and its next pointer explicitly set, so
every field will still be initialized even when using av_malloc.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
 libavformat/mux.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/mux.c b/libavformat/mux.c
index ac370fb24d..870e716950 100644
--- a/libavformat/mux.c
+++ b/libavformat/mux.c
@@ -924,7 +924,7 @@ int ff_interleave_add_packet(AVFormatContext *s, AVPacket *pkt,
     AVStream *st   = s->streams[pkt->stream_index];
     int chunked    = s->max_chunk_size || s->max_chunk_duration;
 
-    this_pktl      = av_mallocz(sizeof(AVPacketList));
+    this_pktl      = av_malloc(sizeof(AVPacketList));
     if (!this_pktl)
         return AVERROR(ENOMEM);
     if ((pkt->flags & AV_PKT_FLAG_UNCODED_FRAME)) {
-- 
2.21.0



More information about the ffmpeg-devel mailing list