From: Bin Meng <bmeng.cn@gmail.com>
Date: Thu, 30 Jul 2015 10:49:13 +0000 (-0700)
Subject: x86: Sanity test on vesa parameters before setting up kernel screen_info
X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=1e7a04730426a96110989ed9c2bafb0d66ec2428;p=u-boot.git

x86: Sanity test on vesa parameters before setting up kernel screen_info

We should not set up kernel screen_info when the vesa parameters are
insane, otherwise kernel will panic.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>
---

diff --git a/drivers/pci/pci_rom.c b/drivers/pci/pci_rom.c
index a33efae263..26db3ca0a1 100644
--- a/drivers/pci/pci_rom.c
+++ b/drivers/pci/pci_rom.c
@@ -235,6 +235,10 @@ void setup_video(struct screen_info *screen_info)
 #ifdef CONFIG_FRAMEBUFFER_SET_VESA_MODE
 	struct vesa_mode_info *vesa = &mode_info.vesa;
 
+	/* Sanity test on VESA parameters */
+	if (!vesa->x_resolution || !vesa->y_resolution)
+		return;
+
 	screen_info->orig_video_isVGA = VIDEO_TYPE_VLFB;
 
 	screen_info->lfb_width = vesa->x_resolution;