]> arthur.barton.de Git - netatalk.git/commitdiff
Fix brokenness with case insensitive username matching with an empty gecos field.
authorFrank Lahm <franklahm@googlemail.com>
Mon, 1 Aug 2011 14:32:06 +0000 (16:32 +0200)
committerFrank Lahm <franklahm@googlemail.com>
Mon, 1 Aug 2011 14:32:06 +0000 (16:32 +0200)
etc/afpd/uam.c

index 37ef9dd2d0a8a6b901e2cd8417e51f0140692cb0..5001bfc51d2eb7420cbd8817554a2948eaab5a3b 100644 (file)
@@ -263,11 +263,11 @@ struct passwd *uam_getname(void *private, char *name, const int len)
         if ((p = strchr(pwent->pw_gecos, ',')))
             *p = '\0';
 
-       if ((size_t)-1 == ( gecoslen = convert_string(obj->options.unixcharset, CH_UCS2, 
-                               pwent->pw_gecos, -1, user, sizeof(username))) )
-               continue;
-       if ((size_t)-1 == ( pwnamelen = convert_string(obj->options.unixcharset, CH_UCS2, 
-                               pwent->pw_name, -1, pwname, sizeof(username))) )
+       gecoslen = convert_string(obj->options.unixcharset, CH_UCS2, 
+                               pwent->pw_gecos, -1, user, sizeof(username));
+       pwnamelen = convert_string(obj->options.unixcharset, CH_UCS2, 
+                               pwent->pw_name, -1, pwname, sizeof(username));
+       if ((size_t)-1 == gecoslen && (size_t)-1 == pwnamelen)
                continue;