diff mbox series

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

Message ID BYAPR03MB41680A89C7A1B81790741E45ADC52@BYAPR03MB4168.namprd03.prod.outlook.com
State New
Headers show
Series [FFmpeg-devel] avcodec/mpegvideo_enc: Add check for av_packet_new_side_data() | expand


Context Check Description
andriy/commit_msg_x86 warning Please wrap lines in the body of the commit message between 60 and 72 characters.
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Jiasheng Jiang June 9, 2024, 6:49 p.m. UTC
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@outlook.com>
 libavcodec/mpegvideo_enc.c | 2 ++
 1 file changed, 2 insertions(+)
diff mbox series


diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 82bab43e14..9b4a516bc0 100644
--- a/libavcodec/mpegvideo_enc.c
+++ b/libavcodec/mpegvideo_enc.c
@@ -1786,6 +1786,8 @@  int ff_mpv_encode_picture(AVCodecContext *avctx, AVPacket *pkt,
             s->mb_info_ptr = av_packet_new_side_data(pkt,
+            if (!s->mb_info_ptr)
+                return AVERROR(ENOMEM);
             s->prev_mb_info = s->last_mb_info = s->mb_info_size = 0;