From abed8af00c4aa7f7367b5b3b2b093329b75c8693 Mon Sep 17 00:00:00 2001 From: Vineeth TM Date: Fri, 5 Jun 2015 08:53:30 +0900 Subject: [PATCH] simplevideomark: Add Error logs When the pattern offset is outside the video, the print error message https://bugzilla.gnome.org/show_bug.cgi?id=743908 --- gst/videosignal/gstsimplevideomark.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gst/videosignal/gstsimplevideomark.c b/gst/videosignal/gstsimplevideomark.c index a0fb1fe320..9d57b5220e 100644 --- a/gst/videosignal/gstsimplevideomark.c +++ b/gst/videosignal/gstsimplevideomark.c @@ -371,8 +371,11 @@ gst_video_mark_yuv (GstSimpleVideoMark * simplevideomark, GstVideoFrame * frame) simplevideomark->pattern_count + simplevideomark->pattern_data_count; /* If x and y offset values are outside the video, no need to draw */ if ((x + (pw * total_pattern)) < 0 || x > width || (y + height) < 0 - || y > height) + || y > height) { + GST_ERROR_OBJECT (simplevideomark, + "simplevideomark pattern is outside the video. Not drawing."); return GST_FLOW_OK; + } /* Offset calculation less than 0, then reset to 0 */ if (offset_calc < 0)