vpxdec: Don't memcpy in frame map failed
authorNicolas Dufresne <nicolas.dufresne@collabora.co.uk>
Sun, 1 Mar 2015 18:52:50 +0000 (13:52 -0500)
committerNicolas Dufresne <nicolas.dufresne@collabora.co.uk>
Sun, 1 Mar 2015 19:03:04 +0000 (14:03 -0500)
This avoid a crash if mapping the frame failed.

ext/vpx/gstvp8dec.c
ext/vpx/gstvp9dec.c

index fc022c7..fb6e928 100644 (file)
@@ -378,6 +378,7 @@ gst_vp8_dec_image_to_buffer (GstVP8Dec * dec, const vpx_image_t * img,
 
   if (!gst_video_frame_map (&frame, info, buffer, GST_MAP_WRITE)) {
     GST_ERROR_OBJECT (dec, "Could not map video buffer");
+    return;
   }
 
   for (comp = 0; comp < 3; comp++) {
index ff075bb..880c191 100644 (file)
@@ -375,6 +375,7 @@ gst_vp9_dec_image_to_buffer (GstVP9Dec * dec, const vpx_image_t * img,
 
   if (!gst_video_frame_map (&frame, info, buffer, GST_MAP_WRITE)) {
     GST_ERROR_OBJECT (dec, "Could not map video buffer");
+    return;
   }
 
   for (comp = 0; comp < 3; comp++) {