dpward / rpms / sssd

Forked from rpms/sssd 3 years ago
Clone
Blob Blame History Raw
From 3af4a383964cf391e0277bd8a272156a1805e357 Mon Sep 17 00:00:00 2001
From: Pavel Reichl <preichl@redhat.com>
Date: Wed, 5 Feb 2014 17:32:20 +0000
Subject: [PATCH 76/80] MONITOR: Incorrect permissions on sssd.conf

Print user friendly warning when permissions on sssd.conf are incorrect and
provide hint.

Resolves:
https://fedorahosted.org/sssd/ticket/2208

Reviewed-by: Stephen Gallagher <sgallagh@redhat.com>
(cherry picked from commit b3cc9b98966fa2d90172348c334b3b70c5261ab3)
---
 src/monitor/monitor.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c
index 09f530d2643b45fc31fb4dbe3cb69f2abc5510af..72a8b7c4a376dd9b68349b959cb869b9da059d38 100644
--- a/src/monitor/monitor.c
+++ b/src/monitor/monitor.c
@@ -84,6 +84,11 @@
  */
 #define KRB5_RCACHE_DIR_DISABLE "__LIBKRB5_DEFAULTS__"
 
+/* Warning messages */
+#define CONF_FILE_PERM_ERROR_MSG "Cannot read config file %s. Please check "\
+                                 "if permissions are 0600 and the file is "\
+                                 "owned by root.root."
+
 int cmdline_debug_level;
 int cmdline_debug_timestamps;
 int cmdline_debug_microseconds;
@@ -2798,7 +2803,8 @@ int main(int argc, const char *argv[])
         case EPERM:
         case EACCES:
             DEBUG(SSSDBG_CRIT_FAILURE,
-                 ("Insufficient permissions to read configuration file.\n"));
+                  (CONF_FILE_PERM_ERROR_MSG, config_file));
+            sss_log(SSS_LOG_ALERT, CONF_FILE_PERM_ERROR_MSG, config_file);
             break;
         default:
             DEBUG(SSSDBG_CRIT_FAILURE,
-- 
1.8.5.3