SMB3 Multi-Channel (Re: What I'm working on)

Stefan (metze) Metzmacher metze at
Mon Jul 21 12:36:37 MDT 2014

Am 08.07.2014 02:37, schrieb Jeremy Allison:
> On Mon, Jul 07, 2014 at 11:26:12AM +0200, Stefan (metze) Metzmacher wrote:
>> Hi Volker,
>>>>> I've cleanup up the preparation patches (82) ready for review in my
>>>>> branch, also attached.
>>>> And the attachment...
>>> Attached find the same patches with most of them r-b me,
>>> very few nit-picks and questions.
>> Thanks for the review!
>> I'll fix the renames from 'conn' into 'xconn' in the next days.
>> I'm not exactly sure what to do with the
>> tsocket_address_string() NULL checks, this is only used for DEBUG()
>> statements and they should handle NULL fine.
>> Should I add something like this?
>> if (addr == NULL) {
>>     addr = "<no memory>";
>> }
>> I'd prefer to avoid this and just pass NULL to DEBUG() which will result
>> in something like:
>> "<null>" or "(null)".
> I'm also concerned about the addition of a talloc_reference()
> in smb2_session_channel().
> I know it's the same way as it's being done in smb2_session_init(),
> but can we get a commitment to move those to a more maintainable
> method once this goes in (i.e. I don't want to block it, but
> maybe adding a comment that this should eventually be removed
> wouldn't be remiss :-).

I just removed it as it wasn't needed at all.

I also introduced a 'const char 'smbXsrv_connection_dbg(const struct
smbXsrv_connection *xconn)'
similar to fsp_fnum_dbg() and fsp_str_dbg().

The commits with TODO/REVIEW are the chance which where added or modified.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: tmp.diff.gz
Type: application/gzip
Size: 70496 bytes
Desc: not available
URL: <>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 246 bytes
Desc: OpenPGP digital signature
URL: <>

More information about the samba-technical mailing list