diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-10-18 12:01:43 +0300 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-18 13:38:37 +0200 |
| commit | 5e2be4e0edff4a1021b6743ca6859129cd8e7067 (patch) | |
| tree | 7d8716746c2d39e7274a8412629decd58b5c2749 /drivers | |
| parent | 9208e3110bf0a6ba4ded56e374857d4b6609c911 (diff) | |
| download | op-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')
| -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 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 == '"') |
