From: Sebastian Dröge <sebastian@centricular.com>
Date: Tue, 12 Apr 2016 12:00:31 +0000 (+0300)
Subject: deinterlace: Instead of confusing crashes later, just error out immediately if mappin... 
X-Git-Tag: 1.10.4~464
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0c84b1b10430406975ec401f88a40edfb942f854;p=platform%2Fupstream%2Fgst-plugins-good.git

deinterlace: Instead of confusing crashes later, just error out immediately if mapping a video frame fails

This probably still crashes but at least we get some hint about what goes
wrong instead of random behaviour later.
---

diff --git a/gst/deinterlace/gstdeinterlace.c b/gst/deinterlace/gstdeinterlace.c
index 987c4ed..e4e1654 100644
--- a/gst/deinterlace/gstdeinterlace.c
+++ b/gst/deinterlace/gstdeinterlace.c
@@ -744,7 +744,11 @@ gst_video_frame_new_and_map (GstVideoInfo * vinfo, GstBuffer * buffer,
     GstMapFlags flags)
 {
   GstVideoFrame *frame = g_malloc0 (sizeof (GstVideoFrame));
-  gst_video_frame_map (frame, vinfo, buffer, flags);
+  if (!gst_video_frame_map (frame, vinfo, buffer, flags)) {
+    g_free (frame);
+    g_return_val_if_reached (NULL);
+    return NULL;
+  }
   return frame;
 }