[patch 1/3] powerpc: Sky CPU: redundant or incorrect tests on unsigned
From: akpm@linux-foundation.org
Date: 2009-11-17 23:05:13
1 review trailer.
From: akpm@linux-foundation.org
Date: 2009-11-17 23:05:13
From: Roel Kluin <redacted> count is unsigned and cannot be less than 0. Signed-off-by: Roel Kluin <redacted> Acked-by: Cyrill Gorcunov <redacted> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Kumar Gala <redacted> Cc: Brian Waite <redacted> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> --- drivers/misc/hdpuftrs/hdpu_cpustate.c | 5 ----- 1 file changed, 5 deletions(-) diff -puN drivers/misc/hdpuftrs/hdpu_cpustate.c~powerpc-sky-cpu-redundant-or-incorrect-tests-on-unsigned drivers/misc/hdpuftrs/hdpu_cpustate.c
--- a/drivers/misc/hdpuftrs/hdpu_cpustate.c~powerpc-sky-cpu-redundant-or-incorrect-tests-on-unsigned
+++ a/drivers/misc/hdpuftrs/hdpu_cpustate.c@@ -121,8 +121,6 @@ static ssize_t cpustate_read(struct file { unsigned char data; - if (count < 0) - return -EFAULT; if (count == 0) return 0;
@@ -137,9 +135,6 @@ static ssize_t cpustate_write(struct fil { unsigned char data; - if (count < 0) - return -EFAULT; - if (count == 0) return 0;
_