[Bug 754429] New: basetextoverlay: crash in basetextoverlay for padding greater than video size

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 754429
           Summary: basetextoverlay: crash in basetextoverlay for padding
                    greater than video size
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: ps.gotarne at samsung.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

gst-launch-1.0 videotestsrc num-buffers=20 ! clockoverlay ypad=300 !
videoconvert ! autovideosink

The above pipeline crash because basetextoverlay try to allocate very large
amount of memory.
The amount of memory to be allocted is calculated from the text image width and
height.
If the xpad or ypad value is greater than the video size, text image
height/width value became -ve.
Because of the -ve value used in the memory size, it try to allocate very large
block of memory and crash.

basetextoverly should avoid the drawing if padding is larger than the video
size. 
basetextoverly is already handling this logic but it couldn't proceed because
of the crash while allocating memory for textimage.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.