[PATCH] common session_info structure

Andreas Schneider asn at samba.org
Tue Jul 19 04:29:28 MDT 2011


On Tuesday 19 July 2011 10:59:51 Stefan  Metzmacher wrote:
> > I'd like to propose that this set of patches go in soon. I doubt
> > anyone is going to have the patience to do this again for a long time
> > if the patch set bit rots due to tree changes. So I'd suggest that
> > Andrew pushes this in a few days time based on my detailed review, but
> > I wanted to give you both a chance to look at this if you have time.
> 
> I took a brief look and it looks very good (but I haven't explored it in
> all tiny details)
> 
> Do we need
> http://git.samba.org/?p=abartlet/samba.git/.git;a=commitdiff;h=8bb4871f451ec
> 077126206a464142ee5a5d091ec in 3.6?
> 
> I guess some of this could fix (Andreas just confirmed it:-)
> https://bugzilla.samba.org/show_bug.cgi?id=8215

I tested and reviewed the patchset. I didn't go into details just looked over 
everything. The patchset looks really good and it fixes an important issue 
which is blocking my work. I only found small things and typos.

You have a +1, good job!


	-- andreas

-- 
Andreas Schneider                   GPG-ID: F33E3FC6
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list