Skip to content

Commit

Permalink
cpuidle: Replace strnicmp with strncasecmp
Browse files Browse the repository at this point in the history
The kernel used to contain two functions for length-delimited,
case-insensitive string comparison, strnicmp with correct semantics
and a slightly buggy strncasecmp. The latter is the POSIX name, so
strnicmp was renamed to strncasecmp, and strnicmp made into a wrapper
for the new strncasecmp to avoid breaking existing users.

To allow the compat wrapper strnicmp to be removed at some point in
the future, and to avoid the extra indirection cost, do
s/strnicmp/strncasecmp/g.

Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
  • Loading branch information
Rasmus Villemoes authored and Rafael J. Wysocki committed Sep 24, 2014
1 parent 0f33be0 commit 9133664
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/cpuidle/governor.c
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ static struct cpuidle_governor * __cpuidle_find_governor(const char *str)
struct cpuidle_governor *gov;

list_for_each_entry(gov, &cpuidle_governors, governor_list)
if (!strnicmp(str, gov->name, CPUIDLE_NAME_LEN))
if (!strncasecmp(str, gov->name, CPUIDLE_NAME_LEN))
return gov;

return NULL;
Expand Down

0 comments on commit 9133664

Please sign in to comment.