From 1a743a4123c104a10c694f7ee9d2f0a1e7182513 Mon Sep 17 00:00:00 2001 From: Jan Engelhardt Date: Wed, 16 Oct 2024 09:55:50 +0200 Subject: [PATCH] sssd: always print path when config object is rejected References: https://github.com/SSSD/sssd/pull/7649 Observed: ``` Oct 16 09:44:04 a4 sssd[28717]: [sssd] [sss_ini_read_sssd_conf] (0x0020): Permission check on config file failed. Oct 16 09:44:04 a4 sssd[28717]: Can't read config: 'File ownership and permissions check failed' Oct 16 09:44:04 a4 sssd[28717]: Failed to read configuration: 'File ownership and permissions check failed' ``` Expected: _Well yes, but **which one**_!? Reviewed-by: Alexey Tikhonov Reviewed-by: Justin Stephenson (cherry picked from commit 2b7915dd84a6b8c3ee26e45357283677fe22f2cb) --- src/util/sss_ini.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/util/sss_ini.c b/src/util/sss_ini.c index 7f9824d88..2a611eb8c 100644 --- a/src/util/sss_ini.c +++ b/src/util/sss_ini.c @@ -888,7 +888,7 @@ int sss_ini_read_sssd_conf(struct sss_ini *self, ret = sss_ini_open(self, config_file, "[sssd]\n"); if (ret != EOK) { DEBUG(SSSDBG_CRIT_FAILURE, - "The sss_ini_open failed %s: %d\n", + "sss_ini_open on %s failed: %d\n", config_file, ret); return ERR_INI_OPEN_FAILED; @@ -898,26 +898,28 @@ int sss_ini_read_sssd_conf(struct sss_ini *self, ret = sss_ini_access_check(self); if (ret != EOK) { DEBUG(SSSDBG_CRIT_FAILURE, - "Permission check on config file failed.\n"); + "Permission check on config file %s failed: %d\n", + config_file, ret); return ERR_INI_INVALID_PERMISSION; } } else { DEBUG(SSSDBG_CONF_SETTINGS, - "File %1$s does not exist.\n", - (config_file ? config_file : "NULL")); + "File %s does not exist.\n", config_file); } ret = sss_ini_parse(self); if (ret != EOK) { sss_ini_config_print_errors(self->error_list); - DEBUG(SSSDBG_FATAL_FAILURE, "Failed to parse configuration.\n"); + DEBUG(SSSDBG_FATAL_FAILURE, "Failed to parse configuration file %s: %d\n", + config_file, ret); return ERR_INI_PARSE_FAILED; } ret = sss_ini_add_snippets(self, config_dir); if (ret != EOK) { DEBUG(SSSDBG_FATAL_FAILURE, - "Error while reading configuration directory.\n"); + "Error while reading configuration directory %s: %d\n", + config_dir, ret); return ERR_INI_ADD_SNIPPETS_FAILED; } -- 2.47.0