PATCHES: Password sync as active directory domain controller

[Stefan Metzmacher]

Hi Andrew,

> We can't have a grep of files in /proc in Samba.  Sorry.
> 
> If you want mutual exclusion between the scripts, can you use a
> transaction lock over the ldb, or fcntl locks on another file?

I'm using fcntl locks now.

> Otherwise, the main thing I see is that autobuild doesn't run with --
> with-gpgme.  

It does it's autodetected if libgpgme11-dev is installed.

> Finally, after the https://evil32.com/ episode, we should not accept 8-
> char key ids.  Please update the docs to only suggest 16-char IDs, and
> make the code refuse to accept < 16-char IDs.

Fixed.

> Otherwise, I'm finally OK with all this.  There is still a lot of code
> here, and some of it is pretty dense, but these are the only objections
> I have so far.  I really appreciate the efforts you have gone to to
> address my concerns.

https://git.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master4-gpgme
just needs review markers.

metze

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160722/5ca58868/signature.sig>