[U-Boot] [PATCH 2/7] x86: Sanity test on vesa parameters before setting up kernel screen_info

Bin Meng bmeng.cn at gmail.com
Thu Jul 30 12:49:13 CEST 2015


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 at gmail.com>
---

 drivers/pci/pci_rom.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/pci/pci_rom.c b/drivers/pci/pci_rom.c
index a33efae..26db3ca 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;
-- 
1.8.2.1



More information about the U-Boot mailing list