From 1b1bace75095cf7a26a56156ff537479ef4b9619 Mon Sep 17 00:00:00 2001 From: Tomas Krizek Date: May 17 2017 10:50:42 +0000 Subject: ca, kra install: validate DM password Before proceeding with installation, validate DM password. If the provided DM password is invalid, abort the installation. Fixes https://pagure.io/freeipa/issue/6892 Signed-off-by: Tomas Krizek Reviewed-By: Martin Basti Reviewed-By: Christian Heimes Reviewed-By: Stanislav Laznicka --- diff --git a/install/tools/ipa-ca-install b/install/tools/ipa-ca-install index da6e5c3..36ae7d2 100755 --- a/install/tools/ipa-ca-install +++ b/install/tools/ipa-ca-install @@ -117,17 +117,19 @@ def parse_options(): def _get_dirman_password(password=None, unattended=False): + # sys.exit() is used on purpose, because otherwise user is advised to + # uninstall the component, even though it is not needed if not password: if unattended: sys.exit('Directory Manager password required') - try: - password = installutils.read_password( - "Directory Manager (existing master)", confirm=False, - validate=False) - except KeyboardInterrupt: - sys.exit(0) - if password is None: - sys.exit("Directory Manager password required") + password = installutils.read_password( + "Directory Manager (existing master)", confirm=False, + validate=False) + try: + installutils.validate_dm_password_ldap(password) + except ValueError: + sys.exit("Directory Manager password is invalid") + return password diff --git a/ipaserver/install/ipa_kra_install.py b/ipaserver/install/ipa_kra_install.py index 65f7461..2b59ba2 100644 --- a/ipaserver/install/ipa_kra_install.py +++ b/ipaserver/install/ipa_kra_install.py @@ -137,6 +137,14 @@ class KRAInstaller(KRAInstall): def run(self): super(KRAInstaller, self).run() + # Verify DM password. This has to be called after ask_for_options(), + # so it can't be placed in validate_options(). + try: + installutils.validate_dm_password_ldap(self.options.password) + except ValueError: + raise admintool.ScriptError( + "Directory Manager password is invalid") + if not cainstance.is_ca_installed_locally(): raise RuntimeError("Dogtag CA is not installed. " "Please install the CA first")