diff options
author | Ondrej Hamada <ohamada@redhat.com> | 2011-11-18 13:55:16 +0100 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2011-11-21 09:39:50 +0100 |
commit | 9eb703f408b01ece30e57e844bc533e3e459fb2c (patch) | |
tree | 123ab6f7c57b8f1a7a5e3631408d3cb1eae7f89d /ipa-client | |
parent | 57841aad2939a8558b29e2cdde0ab651229deaf0 (diff) | |
download | freeipa.git-9eb703f408b01ece30e57e844bc533e3e459fb2c.tar.gz freeipa.git-9eb703f408b01ece30e57e844bc533e3e459fb2c.tar.xz freeipa.git-9eb703f408b01ece30e57e844bc533e3e459fb2c.zip |
Client install root privileges check
ipa-client-install was failing and returning traceback when
wasn't run by root. It was caused by logging initialization that
was taking part before the root privileges check. To correct it,
the check was moved before the logging initialization.
https://fedorahosted.org/freeipa/ticket/2123
Diffstat (limited to 'ipa-client')
-rwxr-xr-x | ipa-client/ipa-install/ipa-client-install | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install index 2f7f1ff5..77418f72 100755 --- a/ipa-client/ipa-install/ipa-client-install +++ b/ipa-client/ipa-install/ipa-client-install @@ -1250,11 +1250,11 @@ def install(options, env, fstore, statestore): def main(): safe_options, options = parse_options() + if not os.getegid() == 0: + sys.exit("\nYou must be root to run ipa-client-install.\n") logging_setup(options) logging.debug('%s was invoked with options: %s' % (sys.argv[0], safe_options)) logging.debug("missing options might be asked for interactively later\n") - if not os.getegid() == 0: - sys.exit("\nYou must be root to run ipa-client-install.\n") env={"PATH":"/bin:/sbin:/usr/kerberos/bin:/usr/kerberos/sbin:/usr/bin:/usr/sbin"} |