Pavel Shilovsky <piastryyy@xxxxxxxxx> writes: > + Aurelien > > Ok, before negotiate tcpStatus is not CifsGood, so, the allocation > won't be skipped. I think this function should be no-op for all > protocols except SMB 3.1.1 to reflect the meaning. Other protocols > don't use preauth hash anyway. > > @Aurelien, @everybody, what would be your thoughts about moving > protocol version check from IF block to the top of the function thus > skipping to allocate preauth hash for protocols that don't need it? In > this case Long's patch will require a change to keep > smb2_crypto_shash_allocate() and its invocation. Sounds good. I think we could go even further and do preauth crypto alloc/dealloc all in the preauth hash compute function too and remove the pointers from the server struct (except for the byte array with the hash result). It only runs during negprot+session establishement which is not a hot path at all and would keep the memory model simpler. Cheers, -- Aurélien Aptel / SUSE Labs Samba Team GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3 SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg, DE GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 247165 (AG München)
