From c6e54a5eaacab21a56c41c4b138e800f6aa15faf Mon Sep 17 00:00:00 2001 From: Matej Habrnal Date: Tue, 8 Dec 2015 16:33:31 +0100 Subject: [PATCH] lib: check_recent_crash_file do not produce error_msg Regarding to the desire to unite log messages in abrt-hook-ccpp, we need to erase loging from check_recent_crash_file() function and let the loging on caller of the function. Related to rhbz#1337186 Signed-off-by: Matej Habrnal --- src/daemon/abrt-server.c | 3 +++ src/lib/check_recent_crash_file.c | 1 - 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/daemon/abrt-server.c b/src/daemon/abrt-server.c index a0faef6..9a1361b 100644 --- a/src/daemon/abrt-server.c +++ b/src/daemon/abrt-server.c @@ -885,7 +885,10 @@ static int perform_http_xact(void) int repeating_crash = check_recent_crash_file(last_file, executable); free(last_file); if (repeating_crash) /* Only pretend that we saved it */ + { + error_msg("Not saving repeating crash in '%s'", executable); goto out; /* ret is 0: "success" */ + } } #if 0 diff --git a/src/lib/check_recent_crash_file.c b/src/lib/check_recent_crash_file.c index 63db260..2df6aff 100644 --- a/src/lib/check_recent_crash_file.c +++ b/src/lib/check_recent_crash_file.c @@ -44,7 +44,6 @@ int check_recent_crash_file(const char *filename, const char *executable) buf[sz] = '\0'; if (strcmp(executable, buf) == 0) { - error_msg("Not saving repeating crash in '%s'", executable); close(fd); return 1; } -- 1.8.3.1