|
 |
483b06 |
From 131fbeff0397aa4e98bab8a22f0a1d366f223f05 Mon Sep 17 00:00:00 2001
|
|
|
483b06 |
From: Alexander Bokovoy <abokovoy@redhat.com>
|
|
|
483b06 |
Date: Mon, 22 May 2017 22:36:18 +0300
|
|
|
483b06 |
Subject: [PATCH] krb5: make sure KDC certificate is readable
|
|
|
483b06 |
|
|
|
483b06 |
When requesting certificate for KDC profile, make sure its public part
|
|
|
483b06 |
is actually readable to others.
|
|
|
483b06 |
|
|
|
483b06 |
Fixes https://pagure.io/freeipa/issue/6973
|
|
|
483b06 |
|
|
|
483b06 |
Reviewed-By: Simo Sorce <ssorce@redhat.com>
|
|
|
483b06 |
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
|
|
|
483b06 |
---
|
|
|
483b06 |
install/restart_scripts/renew_kdc_cert | 4 ----
|
|
|
483b06 |
ipalib/install/certmonger.py | 12 +++++++++---
|
|
|
483b06 |
ipaserver/install/krbinstance.py | 3 ++-
|
|
|
483b06 |
3 files changed, 11 insertions(+), 8 deletions(-)
|
|
|
483b06 |
|
|
|
483b06 |
diff --git a/install/restart_scripts/renew_kdc_cert b/install/restart_scripts/renew_kdc_cert
|
|
|
483b06 |
index 9247920874fc9540ac3421dd59fd902cc195243f..14902893f0e61e31f798fa39737a6ed9d31de111 100755
|
|
|
483b06 |
--- a/install/restart_scripts/renew_kdc_cert
|
|
|
483b06 |
+++ b/install/restart_scripts/renew_kdc_cert
|
|
|
483b06 |
@@ -3,19 +3,15 @@
|
|
|
483b06 |
# Copyright (C) 2017 FreeIPA Contributors see COPYING for license
|
|
|
483b06 |
#
|
|
|
483b06 |
|
|
|
483b06 |
-import os
|
|
|
483b06 |
import syslog
|
|
|
483b06 |
import traceback
|
|
|
483b06 |
|
|
|
483b06 |
from ipaplatform import services
|
|
|
483b06 |
-from ipaplatform.paths import paths
|
|
|
483b06 |
from ipaserver.install import certs
|
|
|
483b06 |
|
|
|
483b06 |
|
|
|
483b06 |
def main():
|
|
|
483b06 |
with certs.renewal_lock:
|
|
|
483b06 |
- os.chmod(paths.KDC_CERT, 0o644)
|
|
|
483b06 |
-
|
|
|
483b06 |
try:
|
|
|
483b06 |
if services.knownservices.krb5kdc.is_running():
|
|
|
483b06 |
syslog.syslog(syslog.LOG_NOTICE, 'restarting krb5kdc')
|
|
|
483b06 |
diff --git a/ipalib/install/certmonger.py b/ipalib/install/certmonger.py
|
|
|
483b06 |
index 5709853ffebdbf58929b9a935e906ae67341bea8..ad031a738f4397d230ed131bde6ac7ddb7ef6fdb 100644
|
|
|
483b06 |
--- a/ipalib/install/certmonger.py
|
|
|
483b06 |
+++ b/ipalib/install/certmonger.py
|
|
|
483b06 |
@@ -302,7 +302,7 @@ def add_subject(request_id, subject):
|
|
|
483b06 |
def request_and_wait_for_cert(
|
|
|
483b06 |
certpath, subject, principal, nickname=None, passwd_fname=None,
|
|
|
483b06 |
dns=None, ca='IPA', profile=None,
|
|
|
483b06 |
- pre_command=None, post_command=None, storage='NSSDB'):
|
|
|
483b06 |
+ pre_command=None, post_command=None, storage='NSSDB', perms=None):
|
|
|
483b06 |
"""
|
|
|
483b06 |
Execute certmonger to request a server certificate.
|
|
|
483b06 |
|
|
|
483b06 |
@@ -310,7 +310,7 @@ def request_and_wait_for_cert(
|
|
|
483b06 |
"""
|
|
|
483b06 |
reqId = request_cert(certpath, subject, principal, nickname,
|
|
|
483b06 |
passwd_fname, dns, ca, profile,
|
|
|
483b06 |
- pre_command, post_command, storage)
|
|
|
483b06 |
+ pre_command, post_command, storage, perms)
|
|
|
483b06 |
state = wait_for_request(reqId, api.env.startup_timeout)
|
|
|
483b06 |
ca_error = get_request_value(reqId, 'ca-error')
|
|
|
483b06 |
if state != 'MONITORING' or ca_error:
|
|
|
483b06 |
@@ -321,12 +321,14 @@ def request_and_wait_for_cert(
|
|
|
483b06 |
def request_cert(
|
|
|
483b06 |
certpath, subject, principal, nickname=None, passwd_fname=None,
|
|
|
483b06 |
dns=None, ca='IPA', profile=None,
|
|
|
483b06 |
- pre_command=None, post_command=None, storage='NSSDB'):
|
|
|
483b06 |
+ pre_command=None, post_command=None, storage='NSSDB', perms=None):
|
|
|
483b06 |
"""
|
|
|
483b06 |
Execute certmonger to request a server certificate.
|
|
|
483b06 |
|
|
|
483b06 |
``dns``
|
|
|
483b06 |
A sequence of DNS names to appear in SAN request extension.
|
|
|
483b06 |
+ ``perms``
|
|
|
483b06 |
+ A tuple of (cert, key) permissions in e.g., (0644,0660)
|
|
|
483b06 |
"""
|
|
|
483b06 |
if storage == 'FILE':
|
|
|
483b06 |
certfile, keyfile = certpath
|
|
|
483b06 |
@@ -367,6 +369,10 @@ def request_cert(
|
|
|
483b06 |
post_command = certmonger_cmd_template % (post_command)
|
|
|
483b06 |
request_parameters['cert-postsave-command'] = post_command
|
|
|
483b06 |
|
|
|
483b06 |
+ if perms:
|
|
|
483b06 |
+ request_parameters['key-perms'] = perms[0]
|
|
|
483b06 |
+ request_parameters['cert-perms'] = perms[1]
|
|
|
483b06 |
+
|
|
|
483b06 |
result = cm.obj_if.add_request(request_parameters)
|
|
|
483b06 |
try:
|
|
|
483b06 |
if result[0]:
|
|
|
483b06 |
diff --git a/ipaserver/install/krbinstance.py b/ipaserver/install/krbinstance.py
|
|
|
483b06 |
index 1692e0b2badb23c18386346a552c83881018cf60..a1053d55ccaae17bef93547c036fb9d08d296f0b 100644
|
|
|
483b06 |
--- a/ipaserver/install/krbinstance.py
|
|
|
483b06 |
+++ b/ipaserver/install/krbinstance.py
|
|
|
483b06 |
@@ -432,7 +432,8 @@ class KrbInstance(service.Service):
|
|
|
483b06 |
dns=self.fqdn,
|
|
|
483b06 |
storage='FILE',
|
|
|
483b06 |
profile=KDC_PROFILE,
|
|
|
483b06 |
- post_command='renew_kdc_cert')
|
|
|
483b06 |
+ post_command='renew_kdc_cert',
|
|
|
483b06 |
+ perms=(0o644, 0o600))
|
|
|
483b06 |
except dbus.DBusException as e:
|
|
|
483b06 |
# if the certificate is already tracked, ignore the error
|
|
|
483b06 |
name = e.get_dbus_name()
|
|
|
483b06 |
--
|
|
|
483b06 |
2.9.4
|
|
|
483b06 |
|