[FFmpeg-cvslog] avcodec/mpegvideo_enc: Add check for av_packet_new_side_data()

[Jiasheng Jiang]

ffmpeg | branch: master | Jiasheng Jiang <jiashengjiangcool at outlook.com> | Mon Jun 10 14:18:11 2024 +0000| [4065ff69a2ed49872f8694a03d0642b18c9d977c] | committer: Anton Khirnov

avcodec/mpegvideo_enc: Add check for av_packet_new_side_data()

Add check for av_packet_new_side_data() to avoid null pointer
dereference if allocation fails.

Fixes: bdc1220eeb ("h263enc: Add an option for outputting info about MBs as side data")
Signed-off-by: Jiasheng Jiang <jiashengjiangcool at outlook.com>
Signed-off-by: Anton Khirnov <anton at khirnov.net>

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

 libavcodec/mpegvideo_enc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 620ca08869..d33754d115 100644
--- a/libavcodec/mpegvideo_enc.c
+++ b/libavcodec/mpegvideo_enc.c
@@ -1825,6 +1825,8 @@ int ff_mpv_encode_picture(AVCodecContext *avctx, AVPacket *pkt,
             s->mb_info_ptr = av_packet_new_side_data(pkt,
                                  AV_PKT_DATA_H263_MB_INFO,
                                  s->mb_width*s->mb_height*12);
+            if (!s->mb_info_ptr)
+                return AVERROR(ENOMEM);
             s->prev_mb_info = s->last_mb_info = s->mb_info_size = 0;
         }