video: console: Fix buffer overflow in cmd 'font list'

jannau · trini · commit 24adbee344b2 · 2024-01-29T14:53:11.000-05:00
vidconsole_ops.get_font is documented to return -ENOENT after the last
video_fontdata entry.

Signed-off-by: Janne Grunau &lt;j@jannau.net&gt;
diff --git a/drivers/video/console_core.c b/drivers/video/console_core.c
@@ -225,7 +225,7 @@ int console_simple_get_font(struct udevice *dev, int seq, struct vidfont_info *i
 {
 	info->name = fonts[seq].name;
 
-	return 0;
+	return info->name ? 0 : -ENOENT;
 }
 
 int console_simple_select_font(struct udevice *dev, const char *name, uint size)

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ int console_simple_get_font(struct udevice dev, int seq, struct vidfont_info i`
`225`	`225`	`{`
`226`	`226`	`info->name = fonts[seq].name;`
`227`	`227`
`228`		`- return 0;`
	`228`	`+ return info->name ? 0 : -ENOENT;`
`229`	`229`	`}`
`230`	`230`
`231`	`231`	`int console_simple_select_font(struct udevice dev, const char name, uint size)`