vmncdec: Sanity-check width/height before using it
authorSebastian Dröge <sebastian@centricular.com>
Wed, 16 Nov 2016 18:41:39 +0000 (20:41 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 16 Nov 2016 18:41:39 +0000 (20:41 +0200)
commit4cb1bcf1422bbcd79c0f683edb7ee85e3f7a31fe
tree30ba305eb7f43a7d5a6c0ee8195db8d360d9873c
parentc7192f045009ec4a7cf6d1026f226bb3426f1fd0
vmncdec: Sanity-check width/height before using it

We will allocate a screen area of width*height*bpp bytes, however this
calculation can easily overflow if too high width or height are given
inside the stream. Nonetheless we would just assume that enough memory
was allocated, try to fill it and overwrite as much memory as wanted.

Also allocate the screen area filled with zeroes to ensure that we start
with full-black and not any random (or not so random) data.

https://scarybeastsecurity.blogspot.gr/2016/11/0day-poc-risky-design-decisions-in.html

Ideally we should just remove this plugin in favour of the one in
gst-libav, which generally seems to be of better code quality.

https://bugzilla.gnome.org/show_bug.cgi?id=774533
gst/vmnc/vmncdec.c