From e45423d76f1c229b1a30ddde205a72774cdb12c6 Mon Sep 17 00:00:00 2001 From: Dan Carpenter Date: Sat, 20 May 2017 00:27:18 +0300 Subject: [PATCH] staging: speakup: signedness bug in spk_ttyio_in_nowait() On most of the common arches char is signed so it can't ever == 0xff. Let's fix this by making it a u8. Fixes: 6b9ad1c742bf ("staging: speakup: add send_xchar, tiocmset and input functionality for tty") Signed-off-by: Dan Carpenter Reviewed-by: Samuel Thibault Signed-off-by: Greg Kroah-Hartman --- drivers/staging/speakup/spk_ttyio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/spk_ttyio.c b/drivers/staging/speakup/spk_ttyio.c index 6e0f042f6a44..8ed7c71c712f 100644 --- a/drivers/staging/speakup/spk_ttyio.c +++ b/drivers/staging/speakup/spk_ttyio.c @@ -220,7 +220,7 @@ static unsigned char spk_ttyio_in(void) static unsigned char spk_ttyio_in_nowait(void) { - char rv = ttyio_in(0); + u8 rv = ttyio_in(0); return (rv == 0xff) ? 0 : rv; }