[PATCH] s3: Fix chained sesssetupAndX/tconn messages

[Volker Lendecke]

On Wed, May 06, 2009 at 03:27:35PM -0700, Tim Prouty wrote:
> In master a sesssetupAndX chained with a tconn will not correctly set
> the TID in the response header.  I'm seeing an XP client send this
> chained sesssetup/tconn when samba has security = share.  Samba's
> current behavior is to return a TID of 0 in the smb header rather than
> the actual TID.  This patch also updates the UID in the header as
> well.  I wanted to have someone who knows the chaining code well
> review this before I push it.

Your patch looks absolutely right.
 
> I spent a bit of time working on a torture test to make it easier to
> reproduce, but the infrastructure isn't quite there to easily chain
> sesssetupAndX messages. 

Wrong :-)

Look at the chain2 test in master. Okay, okay, I've just
added it...

Can you extend that one or add another test that actually
verifies we don't regress on this? And, make sure it works
against Windows and activate it in the build farm?

Thanks,

Volker
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20090507/e7089209/attachment.bin