[Bug 787442] kms: set buffer to NULL if allocation failed

[GStreamer (GNOME Bugzilla)]

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

--- Comment #4 from Guillaume Desmottes <gdesmott at gnome.org> ---
Created attachment 359400
  --> https://bugzilla.gnome.org/attachment.cgi?id=359400&action=edit
kms: fix crash if bo allocation failed when copying to dumb buffer

If bo allocation failed we destroy the buffer and return GST_FLOW_ERROR,
but the @buffer pointer was still pointing to the address of the
destroyed buffer. gst_kms_sink_copy_to_dumb_buffer() was then trying to
unref it when bailing out causing a crash.

Leave @buffer untouched if allocation failed to fix the crash.
Also remove the check on *buffer being not NULL as gst_buffer_new()
will abort if it failed.

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