We should not prematurely increment the pointer, to avoid inadvertently
skipping the NUL terminator.
Fixes: 6d9199bd0f22 ("x86-64: enable hypervisor output on VESA frame buffer")
Reported-by: Kamil Frankowicz <kamil.frankowicz@cert.pl>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
--- a/xen/drivers/video/vesa.c
+++ b/xen/drivers/video/vesa.c
@@ -31,8 +31,8 @@ static unsigned int __initdata vram_rema
static unsigned int __initdata font_height;
static int __init cf_check parse_font_height(const char *s)
{
- if ( simple_strtoul(s, &s, 10) == 8 && (*s++ == 'x') )
- font_height = simple_strtoul(s, &s, 10);
+ if ( simple_strtoul(s, &s, 10) == 8 && (*s == 'x') )
+ font_height = simple_strtoul(s + 1, &s, 10);
if ( *s != '\0' )
font_height = 0;