Skip to contentSkip to conversation

video: Fix crash when scroll screen

PATCH New Eric Gao 2 participants 2 messages Apr 10, 2017 – Apr 14, 2017 Framebuffer layer

lore
Share & tools
lore

Eric Gao

author
#0 lore raw

PATCH 1/1 video: Fix crash when scroll screen

After enable log printing to lcd,when the screen start scroll,system crash, log is shown as bellow.

"Synchronous Abort" handler, esr 0x96000045 "Synchronous Abort" handler, esr 0x96000045

Checking the source code, we found that the variate "pixels" get a wrong value.

int pixels = VIDEO_FONT_HEIGHT * vid_priv->line_length;

"pixels" here means the value of pixels for a character,rather than the byte for a character. so the variate "pixels" is 4 times bigger than it's exact value. which will cause the memory overflow when the cpu run the following code.

for (i = 0; i < pixels; i++)

    *dst++ = clr; <<----

---

 drivers/video/console_normal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/video/console_normal.c +1 -1
index 89a55dd..b627d48 100644
@@ -18,7 +18,7 @@ static int console_normal_set_row(struct udevice *dev, uint row, int clr)
 {
 	struct video_priv *vid_priv = dev_get_uclass_priv(dev->parent);
 	void *line;
-	int pixels = VIDEO_FONT_HEIGHT * vid_priv->line_length;
+	int pixels = VIDEO_FONT_HEIGHT * vid_priv->xsize;
 	int i;
 
 	line = vid_priv->fb + row * VIDEO_FONT_HEIGHT * vid_priv->line_length;
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 video: Fix crash when scroll screen

On Mon, 10 Apr 2017 10:02:20 +0800
Eric Gao eric.gao at rock-chips.com wrote:
...

Quoted 2 lines
 drivers/video/console_normal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

applied to u-boot-video/master. Thanks!

--
Anatolij