[PATCH] Add thread safety to socket_wrapper

[Andreas Schneider]

On Friday, 2 March 2018 06:52:09 CET Anoop C S via samba-technical wrote:
> On Thu, 2018-03-01 at 09:14 -0800, Jeremy Allison wrote:
> > On Thu, Mar 01, 2018 at 10:35:26AM +0100, Volker Lendecke wrote:
> > > On Thu, Mar 01, 2018 at 01:51:39PM +0530, Anoop C S via samba-technical 
wrote:
> > > > I have made changes accordingly. See the attached patch set.
> > > 
> > > Thanks for that change!
> > > 
> > > Small nit-pick: A boolean indicating +1 or -1 swrap_alter_refcount
> > > obscures the callers. Can we either make that two functions like
> > > swrap_inc/dec_refcount or make the parameter an integer like +1/-1.
> > > This would make the callers much more obvious.
> > 
> > My preference would be for swrap_inc()/dec_refcount(). It makes much
> > more sense to me and makes the action obvious.
> > 
> > swrap_alter_refcount() taking a bool is really opaque to follow,
> > sorry.
> 
> That's OK. I understand. I was also little bit skeptical while making that
> change. Please see the attached patch set.

Any objections pushing this?

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