[linux-cifs-client] [PATCH 1/5] [CIFS] add ver= prefix to upcall
format version
Q (Igor Mammedov)
qwerty0987654321 at mail.ru
Fri Nov 16 14:32:56 GMT 2007
Jeff Layton wrote:
> On Fri, 16 Nov 2007 17:17:45 +0300
> "Q (Igor Mammedov)" <qwerty0987654321 at mail.ru> wrote:
>
>> Jeff Layton wrote:
>>> Ok, we can keep it in hex. How's this?
>> Thanks, It's ok, except of a minor glitch. (see inline comment)
>>
>>> From 0ddf684c07942c1d447b5bd86efec375385a528a Mon Sep 17 00:00:00
>>> 2001 From: Jeff Layton <jlayton at redhat.com>
>>> Date: Fri, 16 Nov 2007 08:31:38 -0500
>>> Subject: [PATCH 1/5] [CIFS] add ver= prefix to upcall format version
>>>
>>> ....and change outdated comment.
>>>
>>> Signed-off-by: Jeff Layton <jlayton at redhat.com>
>>> ---
>>> fs/cifs/cifs_spnego.c | 10 ++++------
>>> 1 files changed, 4 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/fs/cifs/cifs_spnego.c b/fs/cifs/cifs_spnego.c
>>> index ad54a3a..198065c 100644
>>> --- a/fs/cifs/cifs_spnego.c
>>> +++ b/fs/cifs/cifs_spnego.c
>>> @@ -75,11 +75,9 @@ cifs_get_spnego_key(struct cifsSesInfo *sesInfo,
>>> const char *hostname) size_t desc_len;
>>> struct key *spnego_key;
>>>
>>> -
>>> - /* version + ;ip{4|6}= + address + ;host=hostname +
>>> - ;sec= + ;uid= + NULL */
>>> - desc_len = 4 + 5 + 32 + 1 + 5 + strlen(hostname) +
>>> - strlen(";sec=krb5") + 7 + sizeof(uid_t)*2 + 1;
>>> + /* BB: come up with better scheme for determining length */
>>> + desc_len = 9 + 5 + 32 + 1 + 5 + strlen(hostname) +
>>> + strlen(";sec=ntlmssp") + 7 + sizeof(uid_t)*2 +
>>> 1;
>> ^^^^^^^
>> Shouldn't here be "krb5" ? :)
>>
>
> I figured that at some point, we may want to call out for ntlmssp, and
> that string is longer than krb5. So we might as well allow for it now.
>
Yep, It wouldn't hurt :)
--
Best regards,
-------------------------
Igor Mammedov,
niallain "at" gmail.com
More information about the linux-cifs-client
mailing list