Re: [PATCH 1/3] cifs: md5 cleanup - functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Francois Romieu wrote:

Btw nobody cared about fs/cifs/connect.c::CIFSNTLMSSPNegotiateSessSetup
(indentation from Mars + unchecked allocations before dereferences).

--
Ueimor
That routine is disabled by default (as with the SPNEGO one) so it has not gotten much attention, it will probably go away or be significantly changed when someone goes through and collapses the four SessionSetup cases (currently each a distinct large function with only the default NTLM SessionSetup enabled by default) down to smaller functions with invoke some common functions. A good time to do this would be when a SessionSetupOldStyle routine is added to handle pre-Windows NT4 SessionSetup (OS/2, LAN Server, LAN Manager etc.). Another possibility for a good time to update these routines is when SPNEGO support is finished - the SPNEGO SessionSetup (which is also too big a function) will change a lot (and get simpler) if Andrew Bartlett's idea of an upcall to the ntlm_auth utility (the man page is somewhat out of date from the better Samba 4 version of this see http://www.samba.org/samba/docs/man/ntlm_auth.1.html but it gives the general idea) is done for that case - so that might be a good time to redo the session setup routines.

NTLMSSP authentication protocol is interesting (and the Davenport guys did a great job updating the documentation for it - see http://davenport.sourceforge.net/ntlm.html) and I would like to implement some of the cool optional features as I get time.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux