From d0f608f60ba9b14fbec52eca892015061419e922 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Mon, 28 Nov 2016 09:30:25 +0200 Subject: [PATCH] vrawpay: Error out cleanly if mapping the video frame fails Instead of later dereferencing NULL and crashing. --- gst/rtp/gstrtpvrawpay.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gst/rtp/gstrtpvrawpay.c b/gst/rtp/gstrtpvrawpay.c index 19367434bb..9368936b14 100644 --- a/gst/rtp/gstrtpvrawpay.c +++ b/gst/rtp/gstrtpvrawpay.c @@ -284,7 +284,10 @@ gst_rtp_vraw_pay_handle_buffer (GstRTPBasePayload * payload, GstBuffer * buffer) rtpvrawpay = GST_RTP_VRAW_PAY (payload); - gst_video_frame_map (&frame, &rtpvrawpay->vinfo, buffer, GST_MAP_READ); + if (!gst_video_frame_map (&frame, &rtpvrawpay->vinfo, buffer, GST_MAP_READ)) { + gst_buffer_unref (buffer); + return GST_FLOW_ERROR; + } GST_LOG_OBJECT (rtpvrawpay, "new frame of %" G_GSIZE_FORMAT " bytes", gst_buffer_get_size (buffer));