video: Fix crash when scroll screen
authoreric.gao@rock-chips.com <eric.gao@rock-chips.com>
Mon, 10 Apr 2017 02:02:20 +0000 (10:02 +0800)
committerAnatolij Gustschin <agust@denx.de>
Fri, 14 Apr 2017 14:11:38 +0000 (16:11 +0200)
commit7682736c891d2b6f0cce167f4ecd55d2df0f562f
treeea6b534252caef6844c6087530993bcb13260e48
parente6a419c5f772f3379fdf834352190a90358022ab
video: Fix crash when scroll screen

After enabling log printing to lcd, when the screen starts
scrolling, system crashes. Log is shown as bellow:

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

Checking the source code, we found that the variable "pixels"
gets 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 bytes for a character. So the variable "pixels" is 4
times bigger than it's exact value, which will cause the memory
overflow when the cpu runs the following code:

    for (i = 0; i < pixels; i++)
        *dst++ = clr; <<----

Signed-off-by: Eric Gao <eric.gao@rock-chips.com>
drivers/video/console_normal.c