diff options
author | Nico Huber <nico.huber@secunet.com> | 2017-05-12 17:10:58 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2017-05-13 15:33:46 +0200 |
commit | cfd433b96daa2d2f7f4f99fff7608e110b64dca4 (patch) | |
tree | 2136415ceb18234dc3a362e2026373508271f04c /src/include/pc80 | |
parent | dd2e35edc14b71b81e9600c7e46a5e934f03d51e (diff) | |
download | coreboot-cfd433b96daa2d2f7f4f99fff7608e110b64dca4.tar.xz |
nb/intel/x4x: Fix uninitialized variable issue
A left-over from 5e3cb72a71 (nb/x4x: Do not enable IGD when not
supported). Should fix coverity issue 1375009. Remove a redundant
line that uses the variable `gfxsize` out of its scope and move the
variable declaration. Make sure the variable is always initialized,
drop unneeded error-handling for `get_option()` and sanitize the
read value instead.
Change-Id: Iee2beda30d8c74df0f412622c3ff3357819e386b
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/19680
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/include/pc80')
0 files changed, 0 insertions, 0 deletions