auxdisplay: img-ascii-lcd: Fix lock-up when displaying empty string
authorGeert Uytterhoeven <geert@linux-m68k.org>
Tue, 19 Oct 2021 14:45:02 +0000 (16:45 +0200)
committerMiguel Ojeda <ojeda@kernel.org>
Thu, 21 Oct 2021 21:36:28 +0000 (23:36 +0200)
commitafcb5a811ff3ab3969f09666535eb6018a160358
tree2a6d3c09e4975adfa8ab343171a500f5eb9ff40b
parentae53c6963f5a3dc89dd04fa1372910e442fabc14
auxdisplay: img-ascii-lcd: Fix lock-up when displaying empty string

While writing an empty string to a device attribute is a no-op, and thus
does not need explicit safeguards, the user can still write a single
newline to an attribute file:

    echo > .../message

If that happens, img_ascii_lcd_display() trims the newline, yielding an
empty string, and causing an infinite loop in img_ascii_lcd_scroll().

Fix this by adding a check for empty strings.  Clear the display in case
one is encountered.

Fixes: 0cad855fbd083ee5 ("auxdisplay: img-ascii-lcd: driver for simple ASCII LCD displays")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
drivers/auxdisplay/img-ascii-lcd.c