From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: Re: [PATCH 2/2] cifs: Correct comment about domainname length Date: Mon, 29 Jul 2013 08:27:52 +0800 Message-ID: <51F5B708.4090404@asianux.com> References: <1374777285-25639-1-git-send-email-scott.lovenberg@gmail.com> <1374777285-25639-2-git-send-email-scott.lovenberg@gmail.com> <51F1C564.5070006@asianux.com> Mime-Version: 1.0 Content-Type: text/plain; charset=GB2312 Content-Transfer-Encoding: 7bit Cc: "sfrench-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org" , Jeff Layton , linux-cifs , samba-technical-w/Ol4Ecudpl8XjKLYN78aQ@public.gmane.org To: Scott Lovenberg Return-path: In-Reply-To: Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On 07/27/2013 02:12 AM, Scott Lovenberg wrote: > On Thu, Jul 25, 2013 at 8:40 PM, Chen Gang wrote: >> On 07/26/2013 02:34 AM, scott.lovenberg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote: >>> From: Scott Lovenberg >>> >>> CIFS_MAX_DOMAINNAME_LEN is the max length for a domain name, not a fully qualified domain name. >>> >>> Signed-off-by: Scott Lovenberg >>> --- >>> 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 > > Sorry, I misunderstood; I figured that it had to be the domain name > without the host since we were allowing a host name that's 1024 > characters long. That documentation should probably be added to the > cifs_mount.h as well then. > > Still, how can we have a FQDN that's 256 characters long when the host > name length can be 1024 characters long? > Excuse me, I am not quite familiar about cifs, so can not provide additional more information (I found it only by reading code). But I feel, it really need additional discussion and check by the related experts (related members who are familiar with cifs). Welcome any members' suggestions and completions. Thanks. -- Chen Gang