summaryrefslogtreecommitdiffstats
path: root/drivers/acpi/osl.c
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2013-10-18 12:01:43 +0300
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-10-18 13:38:37 +0200
commit5e2be4e0edff4a1021b6743ca6859129cd8e7067 (patch)
tree7d8716746c2d39e7274a8412629decd58b5c2749 /drivers/acpi/osl.c
parent9208e3110bf0a6ba4ded56e374857d4b6609c911 (diff)
downloadop-kernel-dev-5e2be4e0edff4a1021b6743ca6859129cd8e7067.zip
op-kernel-dev-5e2be4e0edff4a1021b6743ca6859129cd8e7067.tar.gz
ACPI / osl: remove an unneeded NULL check
"str" is never NULL here so I have removed the check. There are static checkers which complain about superfluous NULL checks because it may indicate confusion or a bug. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/osl.c')
-rw-r--r--drivers/acpi/osl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c
index b1629b5..4923dd4 100644
--- a/drivers/acpi/osl.c
+++ b/drivers/acpi/osl.c
@@ -1335,7 +1335,7 @@ static int __init acpi_os_name_setup(char *str)
if (!str || !*str)
return 0;
- for (; count-- && str && *str; str++) {
+ for (; count-- && *str; str++) {
if (isalnum(*str) || *str == ' ' || *str == ':')
*p++ = *str;
else if (*str == '\'' || *str == '"')
OpenPOWER on IntegriCloud