video: Safe guard info->finfo when mapping a frame
authorThibault Saunier <tsaunier@igalia.com>
Sat, 21 Apr 2018 04:00:28 +0000 (01:00 -0300)
committerThibault Saunier <tsaunier@igalia.com>
Sat, 21 Apr 2018 23:47:31 +0000 (20:47 -0300)
Not sure how that slept through but that case is possible and leads
to segfaults in any code path of the function right now.

https://bugzilla.gnome.org/show_bug.cgi?id=795436

gst-libs/gst/video/video-frame.c

index 30a2d85..cb385ce 100644 (file)
@@ -75,6 +75,7 @@ gst_video_frame_map_id (GstVideoFrame * frame, GstVideoInfo * info,
 
   g_return_val_if_fail (frame != NULL, FALSE);
   g_return_val_if_fail (info != NULL, FALSE);
+  g_return_val_if_fail (info->finfo != NULL, FALSE);
   g_return_val_if_fail (GST_IS_BUFFER (buffer), FALSE);
 
   if (id == -1)