From c6e54a5eaacab21a56c41c4b138e800f6aa15faf Mon Sep 17 00:00:00 2001
From: Matej Habrnal <mhabrnal@redhat.com>
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 <mhabrnal@redhat.com>
---
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