The code in question is slightly wrong, but it really doesn't matter.
If you look at the code usage in the file, it only cares if there is
any difference between the names. When the code gets to the line in
question above, it has already determined that a case insensitive
difference exists, and now it is trying to return the proper {1,0,-1}
value as defined by str*cmp. In this it fails, but none of its callers
care that it returned 1 instead of -1 or vice versa.
Yes, the code should be fixed, and the same problem exists in
fs/vfat/namei.c. Given that this is a bug in appearance only, there
really is no urgency to it.
> Well, if you're going to do that, you also need to change the equality
> line above to be:
> if (tolower(*s1) == tolower(*s2)) return 0;
No, there is no need. Look at the code again. A 0 return is properly
handled.
- Gordon
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.altern.org/andrebalsa/doc/lkml-faq.html