On 07/26/2013 02:34 AM, scott.lovenberg@xxxxxxxxx wrote: > From: Scott Lovenberg <scott.lovenberg@xxxxxxxxx> > > CIFS_MAX_DOMAINNAME_LEN is the max length for a domain name, not a fully qualified domain name. > > Signed-off-by: Scott Lovenberg <scott.lovenberg@xxxxxxxxx> > --- > include/uapi/linux/cifs/cifs_mount.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/uapi/linux/cifs/cifs_mount.h b/include/uapi/linux/cifs/cifs_mount.h > index a58fc6e..d2408ec 100644 > --- a/include/uapi/linux/cifs/cifs_mount.h > +++ b/include/uapi/linux/cifs/cifs_mount.h > @@ -17,7 +17,7 @@ > #define _CIFS_MOUNT_H > > /* Max string lengths for cifs mounting options. */ > -#define CIFS_MAX_DOMAINNAME_LEN 256 /* max fully qualified domain name */ > +#define CIFS_MAX_DOMAINNAME_LEN 256 /* max domain name length */ > #define CIFS_MAX_USERNAME_LEN 256 /* reasonable max for current servers */ > #define CIFS_MAX_PASSWORD_LEN 512 /* Windows max seems to be 256 wide chars */ > #define CIFS_MAX_SHARE_LEN 256 /* reasonable max share name length */ > For our kernel implementation: for 'CIFS_MAX_DOMAINNAME_LEN', it counts 255 + '\0', for 'CIFS_MAX_USERNAME_LEN', it counts 256 (may add additional '\0'), And original mail said about 'CIFS_MAX_DOMAINNAME_LEN': "The maximum length of ... the fully qualified domain name (FQDN) is 63 octets per label and 255 bytes per FQDN. This maximum includes 254 bytes for the FQDN and one byte for the ending dot." And excuse me, I do not know what 'CIFS_MAX_USERNAME_LEN' should be. Do we also need consider about it in our definition comments ? Thanks. -- Chen Gang -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html