[FFmpeg-cvslog] avcodec/avpacket: fix leak on realloc in av_packet_add_side_data()

James Almer git at videolan.org
Sun Nov 20 01:28:08 EET 2016


ffmpeg | branch: release/3.0 | James Almer <jamrial at gmail.com> | Sat Nov 19 12:38:44 2016 -0300| [492bbe58902ddf7f9085f13345fd01098c44c6de] | committer: James Almer

avcodec/avpacket: fix leak on realloc in av_packet_add_side_data()

If realloc fails, the pointer is overwritten and the previously allocated
buffer is leaked, which goes against the expected behavior of keeping the
packet unchanged in case of error.

Michael Niedermayer <michael at niedermayer.cc>
Signed-off-by: James Almer <jamrial at gmail.com>

(cherry picked from commit 574929d8b6de32ae712fcca7ab09f01a3e4616be)

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=492bbe58902ddf7f9085f13345fd01098c44c6de
---

 libavcodec/avpacket.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c
index 6de85de..b3acc88 100644
--- a/libavcodec/avpacket.c
+++ b/libavcodec/avpacket.c
@@ -280,16 +280,17 @@ FF_ENABLE_DEPRECATION_WARNINGS
 int av_packet_add_side_data(AVPacket *pkt, enum AVPacketSideDataType type,
                             uint8_t *data, size_t size)
 {
+    AVPacketSideData *tmp;
     int elems = pkt->side_data_elems;
 
     if ((unsigned)elems + 1 > INT_MAX / sizeof(*pkt->side_data))
         return AVERROR(ERANGE);
 
-    pkt->side_data = av_realloc(pkt->side_data,
-                                (elems + 1) * sizeof(*pkt->side_data));
-    if (!pkt->side_data)
+    tmp = av_realloc(pkt->side_data, (elems + 1) * sizeof(*tmp));
+    if (!tmp)
         return AVERROR(ENOMEM);
 
+    pkt->side_data = tmp;
     pkt->side_data[elems].data = data;
     pkt->side_data[elems].size = size;
     pkt->side_data[elems].type = type;



More information about the ffmpeg-cvslog mailing list