[PATCH] Bug 13653 - dsdb encrypted_secrets: Allow "ldb:// and "mdb://" in key file path

[Andrew Bartlett]

On Tue, 2018-10-16 at 15:38 +1300, Gary Lockyer via samba-technical
wrote:
> Correctly handle "ldb://" and "mdb://" schemes in the file path when
> determining the path for the encrypted secrets key file.
> 
> Fix for https://bugzilla.samba.org/show_bug.cgi?id=13653
> 
> Review appreciated

Except for 
+if "HAVE_LMDB" in config_hash:

(which we always have in selftest and which is a structure we try to
avoid) this looks good!

Just remove that line.

Finally, can you get the random_password into BlackboxTestCase and make
it use a 'proper' RNG (just so we don't have to fix it later for a
static security analysis tool). 

Other than that:
Reviewed-by: Andrew Bartlett <abartlet at samba.org>

Andrew Bartlett
-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba