[PATCH] Improve code in ctdb lock helper

Volker Lendecke vl at samba.org
Thu Dec 1 09:48:38 UTC 2016


On Thu, Dec 01, 2016 at 04:42:02PM +1100, Martin Schwenke wrote:
> On Thu, 1 Dec 2016 15:33:09 +1100, Amitay Isaacs <amitay at gmail.com>
> wrote:
> 
> > Here are two patches that do the following:
> > 
> > 1. Drop the support for locking multiple databases since it's not required
> > any more.
> > 2. Add explicit unlocking of record/db when lock helper is exiting
> > 
> > Please review and push.
> 
> Reviewed-by: Martin Schwenke <martin at meltin.net>

NACK on the second one at least without confirmation that the
async signal-safety aspect has been validated :-)

Volker



More information about the samba-technical mailing list