libexport.a: fix a long-standing typo in name_cmp()
authorChuck Lever <chuck.lever@oracle.com>
Wed, 23 Dec 2009 16:18:17 +0000 (11:18 -0500)
committerSteve Dickson <steved@redhat.com>
Wed, 23 Dec 2009 16:18:17 +0000 (11:18 -0500)
Not sure what "(!*a || !a == ',')" means... but just a few lines later
is
"(!*a || *a == ',')".  I think "a is '\0' or ','" is what was intended.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
support/export/client.c

index 5fcf355..6236561 100644 (file)
@@ -297,7 +297,7 @@ name_cmp(char *a, char *b)
        /* compare strings a and b, but only upto ',' in a */
        while (*a && *b && *a != ',' && *a == *b)
                a++, b++;
-       if (!*b && (!*a || !a == ',') )
+       if (!*b && (!*a || *a == ','))
                return 0;
        if (!*b) return 1;
        if (!*a || *a == ',') return -1;