SMB2/3 Credits (Multi-Credit) support

Jeremy Allison jra at samba.org
Wed Jun 27 17:23:18 MDT 2012


On Thu, Jun 28, 2012 at 01:16:38AM +0200, Stefan (metze) Metzmacher wrote:
> 
> DEFAULT_SMB2_MAX_CREDIT_BITMAP_FACTOR was just there to hide real bugs.
> 
> 1. If the client starts with a SMB1 negprot, we never removed seqnum 0.
> 
> 2. when moving the seqnum_low, we didn't calculate the bitmap_offset
> correct.
> 
> 3. we moved seqnum_low + 1 (as lowest value) instead of id == seqnum_low.
> 
> 4. we didn't stop granting credits if the client didn't use seqnum_low.
> 
> I've tested my patches a lot, using a modified smbtorture
> to check how windows behaves and in what situation it stops granting credits
> and disconnect the connection if a use an invalid seqnum.

Fair enough. We'll need to get these bugs fixed in 3.6.next
though.

Can you log separate bugs for each of these so we can get
the patches into the v3-6-test tree ?

Cheers,

	Jeremy.


More information about the samba-technical mailing list