From 669c302f2d78ab4bbd35982373ae079246c8979d Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Tue, 15 Jan 2013 21:53:30 +1100 Subject: samba-tool domain classicupgrade: Print a better error when the ldap backend PW was not found Reviewed-by: Stefan Metzmacher --- python/samba/upgrade.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'python') diff --git a/python/samba/upgrade.py b/python/samba/upgrade.py index d680a7ca23..e013d2c831 100644 --- a/python/samba/upgrade.py +++ b/python/samba/upgrade.py @@ -598,7 +598,10 @@ def upgrade_from_samba3(samba3, logger, targetdir, session_info=None, if samba3.lp.get("passdb backend").split(":")[0].strip() == "ldapsam": base_dn = samba3.lp.get("ldap suffix") ldapuser = samba3.lp.get("ldap admin dn") - ldappass = (secrets_db.get_ldap_bind_pw(ldapuser)).strip('\x00') + ldappass = secrets_db.get_ldap_bind_pw(ldapuser) + if ldappass is None: + raise ProvisioningError("ldapsam passdb backend detected but no LDAP Bind PW found in secrets.tdb for user %s. Please point this tool at the secrets.tdb that was used by the previous installation.") + ldappass = ldappass.strip('\x00') ldap = True else: ldapuser = None -- cgit