diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-10-18 05:01:43 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-18 07:38:37 -0400 |
commit | 5e2be4e0edff4a1021b6743ca6859129cd8e7067 (patch) | |
tree | 7d8716746c2d39e7274a8412629decd58b5c2749 /drivers/acpi/osl.c | |
parent | 9208e3110bf0a6ba4ded56e374857d4b6609c911 (diff) |
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.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c index b1629b571cb2..4923dd4232b3 100644 --- a/drivers/acpi/osl.c +++ b/drivers/acpi/osl.c | |||
@@ -1335,7 +1335,7 @@ static int __init acpi_os_name_setup(char *str) | |||
1335 | if (!str || !*str) | 1335 | if (!str || !*str) |
1336 | return 0; | 1336 | return 0; |
1337 | 1337 | ||
1338 | for (; count-- && str && *str; str++) { | 1338 | for (; count-- && *str; str++) { |
1339 | if (isalnum(*str) || *str == ' ' || *str == ':') | 1339 | if (isalnum(*str) || *str == ' ' || *str == ':') |
1340 | *p++ = *str; | 1340 | *p++ = *str; |
1341 | else if (*str == '\'' || *str == '"') | 1341 | else if (*str == '\'' || *str == '"') |