From 87257a49a1821d67870aa9760c71b6791583709c Mon Sep 17 00:00:00 2001 From: ikerexxe Date: Fri, 2 Oct 2020 16:09:42 +0200 Subject: [PATCH] lib/sssd: redirect warning message to file Instead of printing warning in stderr print it to file. This way the user is not spammed with unnecessary messages when updating packages. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1749001 --- lib/sssd.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/lib/sssd.c b/lib/sssd.c index 80e49e55..f864ce68 100644 --- a/lib/sssd.c +++ b/lib/sssd.c @@ -11,7 +11,7 @@ #include "prototypes.h" #include "sssd.h" -#define MSG_SSSD_FLUSH_CACHE_FAILED "%s: Failed to flush the sssd cache.\n" +#define MSG_SSSD_FLUSH_CACHE_FAILED "%s: Failed to flush the sssd cache." int sssd_flush_cache (int dbflags) { @@ -46,24 +46,22 @@ int sssd_flush_cache (int dbflags) free(sss_cache_args); if (rv != 0) { /* run_command writes its own more detailed message. */ - (void) fprintf (stderr, _(MSG_SSSD_FLUSH_CACHE_FAILED), Prog); + SYSLOG ((LOG_WARN, MSG_SSSD_FLUSH_CACHE_FAILED, Prog)); return -1; } code = WEXITSTATUS (status); if (!WIFEXITED (status)) { - (void) fprintf (stderr, - _("%s: sss_cache did not terminate normally (signal %d)\n"), - Prog, WTERMSIG (status)); + SYSLOG ((LOG_WARN, "%s: sss_cache did not terminate normally (signal %d)", + Prog, WTERMSIG (status))); return -1; } else if (code == E_CMD_NOTFOUND) { /* sss_cache is not installed, or it is installed but uses an interpreter that is missing. Probably the former. */ return 0; } else if (code != 0) { - (void) fprintf (stderr, _("%s: sss_cache exited with status %d\n"), - Prog, code); - (void) fprintf (stderr, _(MSG_SSSD_FLUSH_CACHE_FAILED), Prog); + SYSLOG ((LOG_WARN, "%s: sss_cache exited with status %d", Prog, code)); + SYSLOG ((LOG_WARN, MSG_SSSD_FLUSH_CACHE_FAILED, Prog)); return -1; } -- 2.26.2