From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Lovenberg Subject: Re: [PATCH] cifs: extend the buffer length enought for sprintf() using Date: Wed, 17 Jul 2013 14:27:23 -0400 Message-ID: References: <51E5E9DA.8020603@asianux.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Chen Gang , Jeff Layton , "sfrench-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org" , linux-cifs , samba-technical-w/Ol4Ecudpl8XjKLYN78aQ@public.gmane.org To: Steve French Return-path: In-Reply-To: Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Tue, Jul 16, 2013 at 10:06 PM, Steve French wrote: >> If this is correct for the domain size, MAX_DOMAIN_SIZE in the >> cifs-utils mount helper (mount.cifs.c) has to be changed. It is >> currently 64 + null terminator. I can open a bug and patch it if this >> is correct. >> >> CC'ing Jeff Layton since he maintains the cifs-utils package. >> >> -- >> Peace and Blessings, >> -Scott. > > http://support.microsoft.com/kb/909264 indicates that (at least for > Microsoft) domain names can be considerably longer than 64 bytes, so > this patch seems reasonable. Merged into cifs-2.6.git - if anyone > objects or wants to add ack/reviewed let me know. > > > "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." > > > -- > Thanks, > > Steve Thanks, Steve. I'll patch this tonight. -- Peace and Blessings, -Scott.