[Bug 767712] xvimage: Missing sanity checking for allocation sizes for various video formats

[GStreamer (GNOME Bugzilla)]

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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #331685|none                        |reviewed
             status|                            |

--- Comment #34 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 331685
  --> https://bugzilla.gnome.org/attachment.cgi?id=331685
Error out on sanity check failure, add checking for RGB pixel formats

Thanks!

>+    if (expected_size != 0 && mem->xvimage->data_size != expected_size)
>+      goto unexpected_size;

Just wondering if this shouldn't be

  if (data_size < expected_size)

instead?

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