src/share/utf8/charset.c : Fix typo in conditional.
authorErik de Castro Lopo <erikd@mega-nerd.com>
Mon, 3 Feb 2014 07:48:24 +0000 (18:48 +1100)
committerErik de Castro Lopo <erikd@mega-nerd.com>
Mon, 3 Feb 2014 07:48:27 +0000 (18:48 +1100)
Typo in the conditional to check the exit condition in ascii_strcasecmp.
It checks for (!*s1 || !*s1) instead of (!*s1 || !*s2). The typo did
not affect the result of the function as the loop  is exited before
changing s1 or s2 anyway.

The problem was found by cppcheck which is run automatically on the
Debian sources. Results here:
http://cppcheck.sourceforge.net/devinfo/daca2-cppcheck1.63/daca2.html

Patch-from: Robert Kausch <robert.kausch@freac.org>

src/share/utf8/charset.c

index cfde562..bd71f46 100644 (file)
@@ -56,7 +56,7 @@ static int ascii_strcasecmp(const char *s1, const char *s2)
   char c1, c2;
 
   for (;; s1++, s2++) {
-    if (!*s1 || !*s1)
+    if (!*s1 || !*s2)
       break;
     if (*s1 == *s2)
       continue;