[PATCH] Next round of netlogon_cli_creds refactoring

Andrew Bartlett abartlet at samba.org
Mon Sep 25 06:08:41 UTC 2017

On Mon, 2017-09-25 at 06:04 +0200, Volker Lendecke wrote:
> On Mon, Sep 25, 2017 at 01:33:15PM +1300, Andrew Bartlett wrote:
> > The final patch looks wrong, was it brought in from a larger branch you
> > were working on?  A compiler with --picky-developer shows that with
> > netlogon_creds_cli_lock() static, it is un-used.
> No, this was deliberate. I want to get rid of the cli_lock function. I
> just had not run a full autobuild with this, functionally making a
> function static should not make any difference. Without that, it had
> survived an autobuild. It's only the sync wrapper that is unused, the
> async ones are still used but should not be used outside this
> function.
> > Otherwise it seems OK to me.
> Is that a formal RB+?

With that clarified:

Reviewed-by: Andrew Bartlett <abartlet at samba.org>
for everything bar this last patch.


Andrew Bartlett
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba

More information about the samba-technical mailing list