Messages in this thread | | | Date | Wed, 20 Aug 2014 15:33:17 +0200 | Subject | Re: [PATCH] platform/x86/toshiba-apci.c possible bad if test? | From | David Herrmann <> |
| |
Hi
On Fri, Jun 13, 2014 at 5:04 AM, Nick <xerofoify@gmail.com> wrote: > diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c > index 76441dc..dfd2243 100644 > --- a/drivers/platform/x86/toshiba_acpi.c > +++ b/drivers/platform/x86/toshiba_acpi.c > @@ -1238,7 +1238,7 @@ static ssize_t toshiba_kbd_bl_mode_store(struct device *dev, > int mode = -1; > int time = -1; > > - if (sscanf(buf, "%i", &mode) != 1 && (mode != 2 || mode != 1)) > + if (sscanf(buf, "%i", &mode) != 1 || (mode != 2 || mode != 1)) > return -EINVAL;
This patch is wrong. Matthew, please revert this or fix it.
(mode != 2 || mode != 1) is a tautology
therefore, this will always return -EINVAL. Furthermore, it lacks a sign-off.
Thanks David
> /* Set the Keyboard Backlight Mode where: > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
| |