|
Julius Milan |
c90eb8 |
From ab628c2b338c8ce317fa7cbabb8129f37b578f41 Mon Sep 17 00:00:00 2001
|
|
Julius Milan |
c90eb8 |
From: Matej Habrnal <mhabrnal@redhat.com>
|
|
Julius Milan |
c90eb8 |
Date: Thu, 24 Aug 2017 14:03:54 +0200
|
|
Julius Milan |
c90eb8 |
Subject: [PATCH 25/27] logging: rename log() to log_warning()
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
Renaming log() logging function to log_warning() because
|
|
Julius Milan |
c90eb8 |
of conflict with log() function from math.h.
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
Related to #1481205
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
Signed-off-by: Matej Habrnal <mhabrnal@redhat.com>
|
|
Julius Milan |
c90eb8 |
---
|
|
Julius Milan |
c90eb8 |
src/applet/applet.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/cli/process.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/cli/report.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/cli/rm.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/daemon/abrt-action-save-package-data.c | 16 ++++++++--------
|
|
Julius Milan |
c90eb8 |
src/daemon/abrt-handle-event.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/daemon/abrt-handle-upload.in | 2 +-
|
|
Julius Milan |
c90eb8 |
src/daemon/abrt-server.c | 12 ++++++------
|
|
Julius Milan |
c90eb8 |
src/daemon/abrt-upload-watch.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/daemon/abrtd.c | 6 +++---
|
|
Julius Milan |
c90eb8 |
src/dbus/abrt-configuration.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/dbus/abrt-dbus.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/dbus/abrt-polkit.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/dbus/abrt_problems2.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/dbus/abrt_problems2_service.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/hooks/abrt-hook-ccpp.c | 18 +++++++++---------
|
|
Julius Milan |
c90eb8 |
src/lib/hooklib.c | 12 ++++++------
|
|
Julius Milan |
c90eb8 |
src/lib/ignored_problems.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/lib/kernel.c | 6 +++---
|
|
Julius Milan |
c90eb8 |
src/lib/problem_api.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-analyze-backtrace.c | 8 ++++----
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-analyze-core.in | 10 +++++-----
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-analyze-oops.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-find-bodhi-update | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-generate-backtrace.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-generate-core-backtrace.c | 4 ++--
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-list-dsos | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-trim-files.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-action-ureport | 6 +++---
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-dump-journal-xorg.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-dump-oops.c | 2 +-
|
|
Julius Milan |
c90eb8 |
src/plugins/abrt-watch-log.c | 8 ++++----
|
|
Julius Milan |
c90eb8 |
src/plugins/bodhi.c | 8 ++++----
|
|
Julius Milan |
c90eb8 |
src/plugins/oops-utils.c | 10 +++++-----
|
|
Julius Milan |
c90eb8 |
tests/koops-parser.at | 18 +++++++++---------
|
|
Julius Milan |
c90eb8 |
tests/runtests/upload-ftp/ftpserver.py | 12 ++++++------
|
|
Julius Milan |
c90eb8 |
36 files changed, 103 insertions(+), 103 deletions(-)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/applet/applet.c b/src/applet/applet.c
|
|
Julius Milan |
c90eb8 |
index 18b911f..da55c04 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/applet/applet.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/applet/applet.c
|
|
Julius Milan |
c90eb8 |
@@ -102,7 +102,7 @@ static void migrate_auto_reporting_to_gsettings(void)
|
|
Julius Milan |
c90eb8 |
remove_map_string_item(settings, OPT_NAME);
|
|
Julius Milan |
c90eb8 |
save_app_conf_file(APP_NAME, settings);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("Successfully migrated "APP_NAME":"OPT_NAME" to "GS_SCHEMA_ID_PRIVACY":"GS_PRIVACY_OPT_AUTO_REPORTING);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Successfully migrated "APP_NAME":"OPT_NAME" to "GS_SCHEMA_ID_PRIVACY":"GS_PRIVACY_OPT_AUTO_REPORTING);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
#undef OPT_NAME
|
|
Julius Milan |
c90eb8 |
finito:
|
|
Julius Milan |
c90eb8 |
diff --git a/src/cli/process.c b/src/cli/process.c
|
|
Julius Milan |
c90eb8 |
index 9ccc271..a9d2c8d 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/cli/process.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/cli/process.c
|
|
Julius Milan |
c90eb8 |
@@ -67,7 +67,7 @@ static int process_one_crash(problem_data_t *problem_data, int report_flags)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if(strcmp(action, "rm") == 0 || strcmp(action, "remove") == 0 )
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("Deleting '%s'"), dir_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Deleting '%s'"), dir_name);
|
|
Julius Milan |
c90eb8 |
const char *dirs_strv[] = {dir_name, NULL};
|
|
Julius Milan |
c90eb8 |
_cmd_remove(dirs_strv);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -76,7 +76,7 @@ static int process_one_crash(problem_data_t *problem_data, int report_flags)
|
|
Julius Milan |
c90eb8 |
else if (((report_flags & CMD_REPORT_UNSAFE) || not_reportable == NULL)
|
|
Julius Milan |
c90eb8 |
&& (strcmp(action, "e") == 0 || strcmp(action, "report") == 0))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("Reporting '%s'"), dir_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Reporting '%s'"), dir_name);
|
|
Julius Milan |
c90eb8 |
const char *dirs_strv[] = {dir_name, NULL};
|
|
Julius Milan |
c90eb8 |
_cmd_report(dirs_strv, report_flags);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/cli/report.c b/src/cli/report.c
|
|
Julius Milan |
c90eb8 |
index a76923f..38f063f 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/cli/report.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/cli/report.c
|
|
Julius Milan |
c90eb8 |
@@ -45,7 +45,7 @@ int _cmd_report(const char **dirs_strv, int flags)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
char *reason = load_text_over_dbus(real_problem_id, FILENAME_NOT_REPORTABLE);
|
|
Julius Milan |
c90eb8 |
if (reason != NULL)
|
|
Julius Milan |
c90eb8 |
- log("%s\n", reason);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s\n", reason);
|
|
Julius Milan |
c90eb8 |
free(reason);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -75,7 +75,7 @@ int _cmd_report(const char **dirs_strv, int flags)
|
|
Julius Milan |
c90eb8 |
/* the problem was successfully reported and option is -d */
|
|
Julius Milan |
c90eb8 |
if((flags & CMD_REPORT_REMOVE) && (status == 0 || status == EXIT_STOP_EVENT_RUN))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("Deleting '%s'"), real_problem_id);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Deleting '%s'"), real_problem_id);
|
|
Julius Milan |
c90eb8 |
delete_dump_dir_possibly_using_abrtd(real_problem_id);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/cli/rm.c b/src/cli/rm.c
|
|
Julius Milan |
c90eb8 |
index 95ae097..8cc3145 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/cli/rm.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/cli/rm.c
|
|
Julius Milan |
c90eb8 |
@@ -45,7 +45,7 @@ static int remove_using_abrtd_or_fs(const char **dirs_strv)
|
|
Julius Milan |
c90eb8 |
const char *rm_dir = *dirs_strv++;
|
|
Julius Milan |
c90eb8 |
status = delete_dump_dir_possibly_using_abrtd(rm_dir);
|
|
Julius Milan |
c90eb8 |
if (!status)
|
|
Julius Milan |
c90eb8 |
- log("rm '%s'", rm_dir);
|
|
Julius Milan |
c90eb8 |
+ log_warning("rm '%s'", rm_dir);
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
errs++;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrt-action-save-package-data.c b/src/daemon/abrt-action-save-package-data.c
|
|
Julius Milan |
c90eb8 |
index 36d84b6..21b4c97 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrt-action-save-package-data.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrt-action-save-package-data.c
|
|
Julius Milan |
c90eb8 |
@@ -237,7 +237,7 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name, const ch
|
|
Julius Milan |
c90eb8 |
kernel = dd_load_text(dd, FILENAME_KERNEL);
|
|
Julius Milan |
c90eb8 |
if (!kernel)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("File 'kernel' containing kernel version not "
|
|
Julius Milan |
c90eb8 |
+ log_warning("File 'kernel' containing kernel version not "
|
|
Julius Milan |
c90eb8 |
"found in current directory");
|
|
Julius Milan |
c90eb8 |
goto ret;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -270,7 +270,7 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name, const ch
|
|
Julius Milan |
c90eb8 |
* the corresponding services. */
|
|
Julius Milan |
c90eb8 |
if (!kernel_oops && is_path_blacklisted(executable))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Blacklisted executable '%s'", executable);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Blacklisted executable '%s'", executable);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -284,9 +284,9 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name, const ch
|
|
Julius Milan |
c90eb8 |
goto ret0; /* no error */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
if (kernel_oops)
|
|
Julius Milan |
c90eb8 |
- log("Can't find kernel package corresponding to '%s'", kernel);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Can't find kernel package corresponding to '%s'", kernel);
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log("Executable '%s' doesn't belong to any package"
|
|
Julius Milan |
c90eb8 |
+ log_warning("Executable '%s' doesn't belong to any package"
|
|
Julius Milan |
c90eb8 |
" and ProcessUnpackaged is set to 'no'", executable);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -310,7 +310,7 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name, const ch
|
|
Julius Milan |
c90eb8 |
/* executable may have changed, check it again */
|
|
Julius Milan |
c90eb8 |
if (is_path_blacklisted(executable))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Blacklisted executable '%s'", executable);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Blacklisted executable '%s'", executable);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
if (!script_pkg)
|
|
Julius Milan |
c90eb8 |
@@ -320,7 +320,7 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name, const ch
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
if (!settings_bProcessUnpackaged)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Interpreter crashed, but no packaged script detected: '%s'", cmdline);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Interpreter crashed, but no packaged script detected: '%s'", cmdline);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -345,7 +345,7 @@ skip_interpreter:
|
|
Julius Milan |
c90eb8 |
* the corresponding services. */
|
|
Julius Milan |
c90eb8 |
if (!kernel_oops && g_list_find_custom(settings_setBlackListedPkgs, package_short_name, (GCompareFunc)g_strcmp0))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Blacklisted package '%s'", package_short_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Blacklisted package '%s'", package_short_name);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -353,7 +353,7 @@ skip_interpreter:
|
|
Julius Milan |
c90eb8 |
if (!(fingerprint != NULL && rpm_fingerprint_is_imported(fingerprint))
|
|
Julius Milan |
c90eb8 |
&& settings_bOpenGPGCheck)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Package '%s' isn't signed with proper key", package_short_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Package '%s' isn't signed with proper key", package_short_name);
|
|
Julius Milan |
c90eb8 |
goto ret; /* return 1 (failure) */
|
|
Julius Milan |
c90eb8 |
/* We used to also check the integrity of the executable here:
|
|
Julius Milan |
c90eb8 |
* if (!CheckHash(package_short_name.c_str(), executable)) BOOM();
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrt-handle-event.c b/src/daemon/abrt-handle-event.c
|
|
Julius Milan |
c90eb8 |
index a66f9ac..a34d6c9 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrt-handle-event.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrt-handle-event.c
|
|
Julius Milan |
c90eb8 |
@@ -330,7 +330,7 @@ static char *do_log(char *log_line, void *param)
|
|
Julius Milan |
c90eb8 |
* Otherwise, errors on post-create result in
|
|
Julius Milan |
c90eb8 |
* "Corrupted or bad dump DIR, deleting" without adequate explanation why.
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
- log("%s", log_line);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s", log_line);
|
|
Julius Milan |
c90eb8 |
return log_line;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrt-handle-upload.in b/src/daemon/abrt-handle-upload.in
|
|
Julius Milan |
c90eb8 |
index 3cf4452..f31d3a1 100755
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrt-handle-upload.in
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrt-handle-upload.in
|
|
Julius Milan |
c90eb8 |
@@ -229,7 +229,7 @@ if __name__ == "__main__":
|
|
Julius Milan |
c90eb8 |
if ex != 0:
|
|
Julius Milan |
c90eb8 |
print_clean_and_die(_("Verification error on '{0}'").format(archive))
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log(_("Unpacking '{0}'").format(archive))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Unpacking '{0}'").format(archive))
|
|
Julius Milan |
c90eb8 |
try:
|
|
Julius Milan |
c90eb8 |
os.mkdir(tempdir)
|
|
Julius Milan |
c90eb8 |
except OSError:
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrt-server.c b/src/daemon/abrt-server.c
|
|
Julius Milan |
c90eb8 |
index 26df51d..692ccee 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrt-server.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrt-server.c
|
|
Julius Milan |
c90eb8 |
@@ -387,7 +387,7 @@ static int run_post_create(const char *dirname, struct response *resp)
|
|
Julius Milan |
c90eb8 |
bool child_is_post_create = 1; /* else it is a notify child */
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
read_child_output:
|
|
Julius Milan |
c90eb8 |
- //log("Reading from event fd %d", child_stdout_fd);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("Reading from event fd %d", child_stdout_fd);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* Read streamed data and split lines */
|
|
Julius Milan |
c90eb8 |
for (;;)
|
|
Julius Milan |
c90eb8 |
@@ -415,7 +415,7 @@ static int run_post_create(const char *dirname, struct response *resp)
|
|
Julius Milan |
c90eb8 |
dup_of_dir = xstrdup(msg + strlen("DUP_OF_DIR: "));
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log("%s", msg);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s", msg);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
strbuf_clear(cmd_output);
|
|
Julius Milan |
c90eb8 |
/* jump to next line */
|
|
Julius Milan |
c90eb8 |
@@ -444,14 +444,14 @@ static int run_post_create(const char *dirname, struct response *resp)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
if (WIFSIGNALED(status))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("'post-create' on '%s' killed by signal %d",
|
|
Julius Milan |
c90eb8 |
+ log_warning("'post-create' on '%s' killed by signal %d",
|
|
Julius Milan |
c90eb8 |
dirname, WTERMSIG(status));
|
|
Julius Milan |
c90eb8 |
goto delete_bad_dir;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
/* else: it is WIFEXITED(status) */
|
|
Julius Milan |
c90eb8 |
if (!dup_of_dir)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("'post-create' on '%s' exited with %d",
|
|
Julius Milan |
c90eb8 |
+ log_warning("'post-create' on '%s' exited with %d",
|
|
Julius Milan |
c90eb8 |
dirname, WEXITSTATUS(status));
|
|
Julius Milan |
c90eb8 |
goto delete_bad_dir;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -504,7 +504,7 @@ static int run_post_create(const char *dirname, struct response *resp)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (!last_ocr)
|
|
Julius Milan |
c90eb8 |
{ /* the new dump directory may lie in the dump location for some time */
|
|
Julius Milan |
c90eb8 |
- log("Using current time for the last occurrence file which may be incorrect.");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Using current time for the last occurrence file which may be incorrect.");
|
|
Julius Milan |
c90eb8 |
time_t t = time(NULL);
|
|
Julius Milan |
c90eb8 |
last_ocr = xasprintf("%lu", (long)t);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -537,7 +537,7 @@ static int run_post_create(const char *dirname, struct response *resp)
|
|
Julius Milan |
c90eb8 |
(dup_of_dir ? "notify-dup" : "notify"),
|
|
Julius Milan |
c90eb8 |
&fd
|
|
Julius Milan |
c90eb8 |
);
|
|
Julius Milan |
c90eb8 |
- //log("Started notify, fd %d -> %d", fd, child_stdout_fd);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("Started notify, fd %d -> %d", fd, child_stdout_fd);
|
|
Julius Milan |
c90eb8 |
xmove_fd(fd, child_stdout_fd);
|
|
Julius Milan |
c90eb8 |
child_is_post_create = 0;
|
|
Julius Milan |
c90eb8 |
if (dup_of_dir)
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrt-upload-watch.c b/src/daemon/abrt-upload-watch.c
|
|
Julius Milan |
c90eb8 |
index 137135c..1a0f642 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrt-upload-watch.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrt-upload-watch.c
|
|
Julius Milan |
c90eb8 |
@@ -103,7 +103,7 @@ run_abrt_handle_upload(struct process *proc, const char *name)
|
|
Julius Milan |
c90eb8 |
static void
|
|
Julius Milan |
c90eb8 |
handle_new_path(struct process *proc, char *name)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Detected creation of file '%s' in upload directory '%s'", name, proc->upload_directory);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Detected creation of file '%s' in upload directory '%s'", name, proc->upload_directory);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (proc->children < proc->max_children)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
diff --git a/src/daemon/abrtd.c b/src/daemon/abrtd.c
|
|
Julius Milan |
c90eb8 |
index 445baf4..3275396 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/daemon/abrtd.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/daemon/abrtd.c
|
|
Julius Milan |
c90eb8 |
@@ -195,7 +195,7 @@ static void queue_post_craete_process(struct abrt_server_proc *proc)
|
|
Julius Milan |
c90eb8 |
stop_abrt_server(removed_proc);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("Size of '%s' >= %u MB (MaxCrashReportsSize), deleting %s directory '%s'",
|
|
Julius Milan |
c90eb8 |
+ log_warning("Size of '%s' >= %u MB (MaxCrashReportsSize), deleting %s directory '%s'",
|
|
Julius Milan |
c90eb8 |
g_settings_dump_location, g_settings_nMaxCrashReportsSize,
|
|
Julius Milan |
c90eb8 |
kind, worst_dir);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -283,7 +283,7 @@ static gboolean abrt_server_output_cb(GIOChannel *channel, GIOCondition conditio
|
|
Julius Milan |
c90eb8 |
queue_post_craete_process(proc);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log("abrt-server(%d): not recognized message: '%s'", proc->pid, line);
|
|
Julius Milan |
c90eb8 |
+ log_warning("abrt-server(%d): not recognized message: '%s'", proc->pid, line);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
g_free(line);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -754,7 +754,7 @@ int main(int argc, char** argv)
|
|
Julius Milan |
c90eb8 |
#endif
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
unsetenv("ABRT_SYSLOG");
|
|
Julius Milan |
c90eb8 |
- msg_prefix = g_progname; /* for log(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
+ msg_prefix = g_progname; /* for log_warning(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (getuid() != 0)
|
|
Julius Milan |
c90eb8 |
error_msg_and_die("Must be run as root");
|
|
Julius Milan |
c90eb8 |
diff --git a/src/dbus/abrt-configuration.c b/src/dbus/abrt-configuration.c
|
|
Julius Milan |
c90eb8 |
index a1cb235..3264275 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/dbus/abrt-configuration.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/dbus/abrt-configuration.c
|
|
Julius Milan |
c90eb8 |
@@ -1015,7 +1015,7 @@ int main(int argc, char *argv[])
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
export_abrt_envvars(0);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- msg_prefix = "abrt-configuration"; /* for log(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
+ msg_prefix = "abrt-configuration"; /* for log_warning(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (getuid() != 0)
|
|
Julius Milan |
c90eb8 |
error_msg_and_die(_("This program must be run as root."));
|
|
Julius Milan |
c90eb8 |
diff --git a/src/dbus/abrt-dbus.c b/src/dbus/abrt-dbus.c
|
|
Julius Milan |
c90eb8 |
index c624d66..e68bce8 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/dbus/abrt-dbus.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/dbus/abrt-dbus.c
|
|
Julius Milan |
c90eb8 |
@@ -203,7 +203,7 @@ static struct dump_dir *open_dump_directory(GDBusMethodInvocation *invocation,
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
if (!allowed_problem_dir(problem_dir))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("UID=%d attempted to access not allowed problem directory '%s'",
|
|
Julius Milan |
c90eb8 |
+ log_warning("UID=%d attempted to access not allowed problem directory '%s'",
|
|
Julius Milan |
c90eb8 |
caller_uid, problem_dir);
|
|
Julius Milan |
c90eb8 |
if (!(flags & OPEN_FAIL_NO_REPLY))
|
|
Julius Milan |
c90eb8 |
return_InvalidProblemDir_error(invocation, problem_dir);
|
|
Julius Milan |
c90eb8 |
@@ -1035,7 +1035,7 @@ int main(int argc, char *argv[])
|
|
Julius Milan |
c90eb8 |
if (!env_path || !env_path[0])
|
|
Julius Milan |
c90eb8 |
putenv((char*)"PATH=/usr/sbin:/usr/bin:/sbin:/bin");
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- msg_prefix = "abrt-dbus"; /* for log(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
+ msg_prefix = "abrt-dbus"; /* for log_warning(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (getuid() != 0)
|
|
Julius Milan |
c90eb8 |
error_msg_and_die(_("This program must be run as root."));
|
|
Julius Milan |
c90eb8 |
diff --git a/src/dbus/abrt-polkit.c b/src/dbus/abrt-polkit.c
|
|
Julius Milan |
c90eb8 |
index e376be1..994ff31 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/dbus/abrt-polkit.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/dbus/abrt-polkit.c
|
|
Julius Milan |
c90eb8 |
@@ -33,7 +33,7 @@
|
|
Julius Milan |
c90eb8 |
#ifdef HAVE_POLKIT
|
|
Julius Milan |
c90eb8 |
static gboolean do_cancel(GCancellable* cancellable)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Timer has expired; cancelling authorization check\n");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Timer has expired; cancelling authorization check\n");
|
|
Julius Milan |
c90eb8 |
g_cancellable_cancel(cancellable);
|
|
Julius Milan |
c90eb8 |
return FALSE;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/dbus/abrt_problems2.c b/src/dbus/abrt_problems2.c
|
|
Julius Milan |
c90eb8 |
index 2180fbf..1d947f2 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/dbus/abrt_problems2.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/dbus/abrt_problems2.c
|
|
Julius Milan |
c90eb8 |
@@ -91,7 +91,7 @@ int main(int argc, char *argv[])
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
export_abrt_envvars(0);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- msg_prefix = "abrt-problems2"; /* for log(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
+ msg_prefix = "abrt-problems2"; /* for log_warning(), error_msg() and such */
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (getuid() != 0)
|
|
Julius Milan |
c90eb8 |
error_msg_and_die("This program must be run as root.");
|
|
Julius Milan |
c90eb8 |
diff --git a/src/dbus/abrt_problems2_service.c b/src/dbus/abrt_problems2_service.c
|
|
Julius Milan |
c90eb8 |
index 380004a..7c813d1 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/dbus/abrt_problems2_service.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/dbus/abrt_problems2_service.c
|
|
Julius Milan |
c90eb8 |
@@ -93,7 +93,7 @@ static void problems2_object_type_print_all_objects(struct problems2_object_type
|
|
Julius Milan |
c90eb8 |
const char *p;
|
|
Julius Milan |
c90eb8 |
AbrtP2Object *obj;
|
|
Julius Milan |
c90eb8 |
while(g_hash_table_iter_next(&iter, (gpointer)&p, (gpointer)&obj))
|
|
Julius Milan |
c90eb8 |
- log("%s: '%s' : %p", prefix, p, obj);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s: '%s' : %p", prefix, p, obj);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
#endif
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/hooks/abrt-hook-ccpp.c b/src/hooks/abrt-hook-ccpp.c
|
|
Julius Milan |
c90eb8 |
index 1b86189..1c4e45e 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/hooks/abrt-hook-ccpp.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/hooks/abrt-hook-ccpp.c
|
|
Julius Milan |
c90eb8 |
@@ -197,16 +197,16 @@ static int open_user_core(uid_t uid, uid_t fsuid, gid_t fsgid, pid_t pid, char *
|
|
Julius Milan |
c90eb8 |
const char *val = "%";
|
|
Julius Milan |
c90eb8 |
if (specifier_num > 0) /* not %% */
|
|
Julius Milan |
c90eb8 |
val = percent_values[specifier_num - 1];
|
|
Julius Milan |
c90eb8 |
- //log("c:'%c'", c);
|
|
Julius Milan |
c90eb8 |
- //log("val:'%s'", val);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("c:'%c'", c);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("val:'%s'", val);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* Replace %c at core_basename[idx] by its value */
|
|
Julius Milan |
c90eb8 |
idx--;
|
|
Julius Milan |
c90eb8 |
char *old = core_basename;
|
|
Julius Milan |
c90eb8 |
core_basename = xasprintf("%.*s%s%s", idx, core_basename, val, core_basename + idx + 2);
|
|
Julius Milan |
c90eb8 |
- //log("pos:'%*s|'", idx, "");
|
|
Julius Milan |
c90eb8 |
- //log("new:'%s'", core_basename);
|
|
Julius Milan |
c90eb8 |
- //log("old:'%s'", old);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("pos:'%*s|'", idx, "");
|
|
Julius Milan |
c90eb8 |
+ //log_warning("new:'%s'", core_basename);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("old:'%s'", old);
|
|
Julius Milan |
c90eb8 |
free(old);
|
|
Julius Milan |
c90eb8 |
idx += strlen(val);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -785,18 +785,18 @@ create_core_backtrace(struct dump_dir *dd, uid_t uid, uid_t fsuid, gid_t gid,
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
if (WIFSIGNALED(status))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Core backtrace generator signaled with %d", WTERMSIG(status));
|
|
Julius Milan |
c90eb8 |
+ log_warning("Core backtrace generator signaled with %d", WTERMSIG(status));
|
|
Julius Milan |
c90eb8 |
goto core_backtrace_failed;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
if (!WIFEXITED(status))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Core backtrace generator did not properly exit");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Core backtrace generator did not properly exit");
|
|
Julius Milan |
c90eb8 |
goto core_backtrace_failed;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
const int r = WEXITSTATUS(status);
|
|
Julius Milan |
c90eb8 |
if (r != 0)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Core backtrace generator exited with error %d", r);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Core backtrace generator exited with error %d", r);
|
|
Julius Milan |
c90eb8 |
goto core_backtrace_failed;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -1097,7 +1097,7 @@ int main(int argc, char** argv)
|
|
Julius Milan |
c90eb8 |
signame, "abrtd is not running");
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* not an error, exit with exit code 0 */
|
|
Julius Milan |
c90eb8 |
- log("If abrtd crashed, "
|
|
Julius Milan |
c90eb8 |
+ log_warning("If abrtd crashed, "
|
|
Julius Milan |
c90eb8 |
"/proc/sys/kernel/core_pattern contains a stale value, "
|
|
Julius Milan |
c90eb8 |
"consider resetting it to 'core'"
|
|
Julius Milan |
c90eb8 |
);
|
|
Julius Milan |
c90eb8 |
diff --git a/src/lib/hooklib.c b/src/lib/hooklib.c
|
|
Julius Milan |
c90eb8 |
index f9337b7..135c7cd 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/lib/hooklib.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/lib/hooklib.c
|
|
Julius Milan |
c90eb8 |
@@ -75,7 +75,7 @@ void trim_problem_dirs(const char *dirname, double cap_size, const char *exclude
|
|
Julius Milan |
c90eb8 |
free(worst_basename);
|
|
Julius Milan |
c90eb8 |
break;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
- log("%s is %.0f bytes (more than %.0fMiB), deleting '%s'",
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s is %.0f bytes (more than %.0fMiB), deleting '%s'",
|
|
Julius Milan |
c90eb8 |
dirname, cur_size, cap_size / (1024*1024), worst_basename);
|
|
Julius Milan |
c90eb8 |
char *d = concat_path_file(dirname, worst_basename);
|
|
Julius Milan |
c90eb8 |
free(worst_basename);
|
|
Julius Milan |
c90eb8 |
@@ -256,7 +256,7 @@ char *get_backtrace(const char *dump_dir_name, unsigned timeout_sec, const char
|
|
Julius Milan |
c90eb8 |
dd_close(dd);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* Let user know what's going on */
|
|
Julius Milan |
c90eb8 |
- log(_("Generating backtrace"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Generating backtrace"));
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
unsigned i = 0;
|
|
Julius Milan |
c90eb8 |
char *args[25];
|
|
Julius Milan |
c90eb8 |
@@ -367,11 +367,11 @@ char *get_backtrace(const char *dump_dir_name, unsigned timeout_sec, const char
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
bt_depth /= 2;
|
|
Julius Milan |
c90eb8 |
if (bt)
|
|
Julius Milan |
c90eb8 |
- log("Backtrace is too big (%u bytes), reducing depth to %u",
|
|
Julius Milan |
c90eb8 |
+ log_warning("Backtrace is too big (%u bytes), reducing depth to %u",
|
|
Julius Milan |
c90eb8 |
(unsigned)strlen(bt), bt_depth);
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
/* (NB: in fact, current impl. of exec_vp() never returns NULL) */
|
|
Julius Milan |
c90eb8 |
- log("Failed to generate backtrace, reducing depth to %u",
|
|
Julius Milan |
c90eb8 |
+ log_warning("Failed to generate backtrace, reducing depth to %u",
|
|
Julius Milan |
c90eb8 |
bt_depth);
|
|
Julius Milan |
c90eb8 |
free(bt);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -443,7 +443,7 @@ int dump_suid_policy()
|
|
Julius Milan |
c90eb8 |
FILE *f = fopen(filename, "r");
|
|
Julius Milan |
c90eb8 |
if (!f)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Can't open %s", filename);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Can't open %s", filename);
|
|
Julius Milan |
c90eb8 |
return suid_dump_policy;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -452,7 +452,7 @@ int dump_suid_policy()
|
|
Julius Milan |
c90eb8 |
if (c != EOF)
|
|
Julius Milan |
c90eb8 |
suid_dump_policy = c - '0';
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- //log("suid dump policy is: %i", suid_dump_policy);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("suid dump policy is: %i", suid_dump_policy);
|
|
Julius Milan |
c90eb8 |
return suid_dump_policy;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/lib/ignored_problems.c b/src/lib/ignored_problems.c
|
|
Julius Milan |
c90eb8 |
index 7afc80a..d007598 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/lib/ignored_problems.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/lib/ignored_problems.c
|
|
Julius Milan |
c90eb8 |
@@ -176,7 +176,7 @@ void ignored_problems_add(ignored_problems_t *set, const char *problem_id)
|
|
Julius Milan |
c90eb8 |
* already emitted good explanatory message. This message
|
|
Julius Milan |
c90eb8 |
* explains what the previous failure causes.
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
- VERB1 log("Can't add problem '%s' to ignored problems:"
|
|
Julius Milan |
c90eb8 |
+ VERB1 log_warning("Can't add problem '%s' to ignored problems:"
|
|
Julius Milan |
c90eb8 |
" can't open the problem", problem_id);
|
|
Julius Milan |
c90eb8 |
return;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -195,7 +195,7 @@ void ignored_problems_remove_row(ignored_problems_t *set, const char *problem_id
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
INITIALIZE_LIBABRT();
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- VERB1 log("Going to remove problem '%s' from ignored problems", problem_id);
|
|
Julius Milan |
c90eb8 |
+ VERB1 log_warning("Going to remove problem '%s' from ignored problems", problem_id);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
FILE *orig_fp;
|
|
Julius Milan |
c90eb8 |
if (!ignored_problems_file_contains(set, problem_id, uuid, duphash, &orig_fp, "r"))
|
|
Julius Milan |
c90eb8 |
diff --git a/src/lib/kernel.c b/src/lib/kernel.c
|
|
Julius Milan |
c90eb8 |
index 8cd431c..d8ba920 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/lib/kernel.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/lib/kernel.c
|
|
Julius Milan |
c90eb8 |
@@ -71,7 +71,7 @@ static void record_oops(GList **oops_list, const struct abrt_koops_line_info* li
|
|
Julius Milan |
c90eb8 |
free(version);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- VERB3 if (rv == 0) log("Dropped oops: too short");
|
|
Julius Milan |
c90eb8 |
+ VERB3 if (rv == 0) log_warning("Dropped oops: too short");
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* In some comparisons, we skip 1st letter, to avoid dealing with
|
|
Julius Milan |
c90eb8 |
@@ -567,9 +567,9 @@ int koops_hash_str_ext(char result[SHA1_RESULT_LEN*2 + 1], const char *oops_buf,
|
|
Julius Milan |
c90eb8 |
hash_str = sr_thread_get_duphash(thread, frame_count, NULL,
|
|
Julius Milan |
c90eb8 |
duphash_flags|SR_DUPHASH_NOHASH);
|
|
Julius Milan |
c90eb8 |
if (hash_str)
|
|
Julius Milan |
c90eb8 |
- log("Generating duphash: '%s'", hash_str);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Generating duphash: '%s'", hash_str);
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log("Nothing useful for duphash");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Nothing useful for duphash");
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
free(hash_str);
|
|
Julius Milan |
c90eb8 |
diff --git a/src/lib/problem_api.c b/src/lib/problem_api.c
|
|
Julius Milan |
c90eb8 |
index aaf127a..045bcac 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/lib/problem_api.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/lib/problem_api.c
|
|
Julius Milan |
c90eb8 |
@@ -90,7 +90,7 @@ static int add_dirname_to_GList(struct dump_dir *dd, void *arg)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
if (!dir_has_correct_permissions(dd->dd_dirname, DD_PERM_DAEMONS))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Ignoring '%s': invalid owner, group or mode", dd->dd_dirname);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Ignoring '%s': invalid owner, group or mode", dd->dd_dirname);
|
|
Julius Milan |
c90eb8 |
/*Do not break*/
|
|
Julius Milan |
c90eb8 |
return 0;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-analyze-backtrace.c b/src/plugins/abrt-action-analyze-backtrace.c
|
|
Julius Milan |
c90eb8 |
index 9cd6f6f..622d795 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-analyze-backtrace.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-analyze-backtrace.c
|
|
Julius Milan |
c90eb8 |
@@ -87,8 +87,8 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
* instead of a backtrace.
|
|
Julius Milan |
c90eb8 |
* and component only. This is not supposed to happen often.
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
- log(_("Backtrace parsing failed for %s"), dump_dir_name);
|
|
Julius Milan |
c90eb8 |
- log("%d:%d: %s", location.line, location.column, location.message);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Backtrace parsing failed for %s"), dump_dir_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%d:%d: %s", location.line, location.column, location.message);
|
|
Julius Milan |
c90eb8 |
struct strbuf *emptybt = strbuf_new();
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
char *executable = dd_load_text(dd, FILENAME_EXECUTABLE);
|
|
Julius Milan |
c90eb8 |
@@ -137,7 +137,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
hash_str = sr_thread_get_duphash(crash_thread, 3, component,
|
|
Julius Milan |
c90eb8 |
SR_DUPHASH_NOHASH);
|
|
Julius Milan |
c90eb8 |
- log("Generating duphash: %s", hash_str);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Generating duphash: %s", hash_str);
|
|
Julius Milan |
c90eb8 |
free(hash_str);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -147,7 +147,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
free(hash_str);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log(_("Crash thread not found"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Crash thread not found"));
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
/* Compute the backtrace rating. */
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-analyze-core.in b/src/plugins/abrt-action-analyze-core.in
|
|
Julius Milan |
c90eb8 |
index 448bdab..9aca379 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-analyze-core.in
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-analyze-core.in
|
|
Julius Milan |
c90eb8 |
@@ -17,16 +17,16 @@ _ = lambda x: gettext.gettext(x)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
verbose = 0
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
-def log(s):
|
|
Julius Milan |
c90eb8 |
+def log_warning(s):
|
|
Julius Milan |
c90eb8 |
sys.stderr.write("%s\n" % s)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
def log1(message):
|
|
Julius Milan |
c90eb8 |
if verbose > 0:
|
|
Julius Milan |
c90eb8 |
- log(message)
|
|
Julius Milan |
c90eb8 |
+ log_warning(message)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
def log2(message):
|
|
Julius Milan |
c90eb8 |
if verbose > 1:
|
|
Julius Milan |
c90eb8 |
- log(message)
|
|
Julius Milan |
c90eb8 |
+ log_warning(message)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
def error_msg(s):
|
|
Julius Milan |
c90eb8 |
sys.stderr.write("%s\n" % s)
|
|
Julius Milan |
c90eb8 |
@@ -69,7 +69,7 @@ def extract_info_from_core(coredump_name):
|
|
Julius Milan |
c90eb8 |
#SEP = 3
|
|
Julius Milan |
c90eb8 |
EXECUTABLE = 4
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log(_("Analyzing coredump '%s'") % coredump_name)
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Analyzing coredump '%s'") % coredump_name)
|
|
Julius Milan |
c90eb8 |
eu_unstrip_OUT = Popen(["eu-unstrip","--core=%s" % coredump_name, "-n"], stdout=PIPE, bufsize=-1, universal_newlines=True).communicate()[0]
|
|
Julius Milan |
c90eb8 |
# parse eu_unstrip_OUT and return the list of build_ids
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -107,7 +107,7 @@ def extract_info_from_core(coredump_name):
|
|
Julius Milan |
c90eb8 |
# "Please split kernel debuginfo packages so that VDSO debuginfos are separate" -
|
|
Julius Milan |
c90eb8 |
# we might want to remove this special-casing later.
|
|
Julius Milan |
c90eb8 |
if b_ids_line[BUILD_ID] == '-':
|
|
Julius Milan |
c90eb8 |
- log(_("Missing build id: %s" % b_ids_line[EXECUTABLE]))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Missing build id: %s" % b_ids_line[EXECUTABLE]))
|
|
Julius Milan |
c90eb8 |
elif ((len(b_ids_line) == EXECUTABLE) or (b_ids_line[EXECUTABLE] not in ["linux-vdso.so.1", "linux-gate.so.1"])):
|
|
Julius Milan |
c90eb8 |
build_id = b_ids_line[BUILD_ID].split('@')[0]
|
|
Julius Milan |
c90eb8 |
build_ids.add(build_id)
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-analyze-oops.c b/src/plugins/abrt-action-analyze-oops.c
|
|
Julius Milan |
c90eb8 |
index f74ebcb..d7fe688 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-analyze-oops.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-analyze-oops.c
|
|
Julius Milan |
c90eb8 |
@@ -73,7 +73,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
if (!res || !drop_notreportable_oopses)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
/* Let users know that they can configure ABRT to drop these oopses. */
|
|
Julius Milan |
c90eb8 |
- log("Preserving oops '%s' because DropNotReportableOopses is 'no'", dump_dir_name);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Preserving oops '%s' because DropNotReportableOopses is 'no'", dump_dir_name);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
dd_save_text(dd, FILENAME_NOT_REPORTABLE,
|
|
Julius Milan |
c90eb8 |
_("The backtrace does not contain enough meaningful function frames "
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-find-bodhi-update b/src/plugins/abrt-action-find-bodhi-update
|
|
Julius Milan |
c90eb8 |
index d75ac87..3bb96fc 100755
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-find-bodhi-update
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-find-bodhi-update
|
|
Julius Milan |
c90eb8 |
@@ -130,7 +130,7 @@ if __name__ == "__main__":
|
|
Julius Milan |
c90eb8 |
bug_id = proc.stdout.read().rstrip().decode("utf-8", "ignore")
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if bug_id:
|
|
Julius Milan |
c90eb8 |
- log(_("Duplicate bugzilla bug '#{0}' was found").format(bug_id))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Duplicate bugzilla bug '#{0}' was found").format(bug_id))
|
|
Julius Milan |
c90eb8 |
else:
|
|
Julius Milan |
c90eb8 |
log1(_("There is no bugzilla bug with 'abrt_hash:{0}'").format(duphash_content))
|
|
Julius Milan |
c90eb8 |
sys.exit(RETURN_OK)
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-generate-backtrace.c b/src/plugins/abrt-action-generate-backtrace.c
|
|
Julius Milan |
c90eb8 |
index 30ce446..78d7795 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-generate-backtrace.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-generate-backtrace.c
|
|
Julius Milan |
c90eb8 |
@@ -84,7 +84,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
if (!backtrace)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
backtrace = xstrdup("");
|
|
Julius Milan |
c90eb8 |
- log("get_backtrace() returns NULL, broken core/gdb?");
|
|
Julius Milan |
c90eb8 |
+ log_warning("get_backtrace() returns NULL, broken core/gdb?");
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
free(debuginfo_dirs);
|
|
Julius Milan |
c90eb8 |
free_abrt_conf_data();
|
|
Julius Milan |
c90eb8 |
@@ -100,7 +100,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
/* Don't be completely silent. gdb run takes a few seconds,
|
|
Julius Milan |
c90eb8 |
* it is useful to let user know it (maybe) worked.
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
- log(_("Backtrace is generated and saved, %u bytes"), (int)strlen(backtrace));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Backtrace is generated and saved, %u bytes"), (int)strlen(backtrace));
|
|
Julius Milan |
c90eb8 |
free(backtrace);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
return 0;
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-generate-core-backtrace.c b/src/plugins/abrt-action-generate-core-backtrace.c
|
|
Julius Milan |
c90eb8 |
index 791af83..8bb6f96 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-generate-core-backtrace.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-generate-core-backtrace.c
|
|
Julius Milan |
c90eb8 |
@@ -78,7 +78,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
char *gdb_output = get_backtrace(dump_dir_name, exec_timeout_sec, NULL);
|
|
Julius Milan |
c90eb8 |
if (!gdb_output)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("Error: GDB did not return any data"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Error: GDB did not return any data"));
|
|
Julius Milan |
c90eb8 |
return 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -92,7 +92,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (!success)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("Error: %s"), error_message);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Error: %s"), error_message);
|
|
Julius Milan |
c90eb8 |
free(error_message);
|
|
Julius Milan |
c90eb8 |
return 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-list-dsos b/src/plugins/abrt-action-list-dsos
|
|
Julius Milan |
c90eb8 |
index 4ffa184..adb228a 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-list-dsos
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-list-dsos
|
|
Julius Milan |
c90eb8 |
@@ -7,7 +7,7 @@ import os
|
|
Julius Milan |
c90eb8 |
import getopt
|
|
Julius Milan |
c90eb8 |
import rpm
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
-def log(s):
|
|
Julius Milan |
c90eb8 |
+def log_warning(s):
|
|
Julius Milan |
c90eb8 |
sys.stderr.write("%s\n" % s)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
def error_msg(s):
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-trim-files.c b/src/plugins/abrt-action-trim-files.c
|
|
Julius Milan |
c90eb8 |
index 09ba662..3651a0b 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-trim-files.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-trim-files.c
|
|
Julius Milan |
c90eb8 |
@@ -108,7 +108,7 @@ static double get_dir_size(const char *dirname,
|
|
Julius Milan |
c90eb8 |
GList *cur = preserve_files_list;
|
|
Julius Milan |
c90eb8 |
while (cur)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- //log("'%s' ? '%s'", fullname, *pp);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("'%s' ? '%s'", fullname, *pp);
|
|
Julius Milan |
c90eb8 |
if (strcmp(fullname, (char*)cur->data) == 0)
|
|
Julius Milan |
c90eb8 |
goto next;
|
|
Julius Milan |
c90eb8 |
cur = cur->next;
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-action-ureport b/src/plugins/abrt-action-ureport
|
|
Julius Milan |
c90eb8 |
index 37595f1..a7484c8 100755
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-action-ureport
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-action-ureport
|
|
Julius Milan |
c90eb8 |
@@ -144,7 +144,7 @@ if __name__ == "__main__":
|
|
Julius Milan |
c90eb8 |
if reported_to and reported_to != "":
|
|
Julius Milan |
c90eb8 |
bugs = get_bugzilla_reports(reported_to)
|
|
Julius Milan |
c90eb8 |
if bugs:
|
|
Julius Milan |
c90eb8 |
- log(_("A bug was already filed about this problem:"))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("A bug was already filed about this problem:"))
|
|
Julius Milan |
c90eb8 |
bugs = sorted(bugs)
|
|
Julius Milan |
c90eb8 |
for bug in bugs:
|
|
Julius Milan |
c90eb8 |
print(bug)
|
|
Julius Milan |
c90eb8 |
@@ -154,7 +154,7 @@ if __name__ == "__main__":
|
|
Julius Milan |
c90eb8 |
else:
|
|
Julius Milan |
c90eb8 |
log1("'%s/reported_to' doesn't exist", dirname)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log(_("uReport was already sent, not sending it again"))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("uReport was already sent, not sending it again"))
|
|
Julius Milan |
c90eb8 |
sys.exit(0)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if report_type == "CCpp" and not core_backtrace_exists:
|
|
Julius Milan |
c90eb8 |
@@ -175,7 +175,7 @@ if __name__ == "__main__":
|
|
Julius Milan |
c90eb8 |
watch = os.getenv("uReport_WatchReportedBugs") or ""
|
|
Julius Milan |
c90eb8 |
if exitcode == 70 and watch.lower() in ["yes", "on", "1"]:
|
|
Julius Milan |
c90eb8 |
if reported_to and reported_to != "" and get_bugzilla_reports(reported_to):
|
|
Julius Milan |
c90eb8 |
- log(_("Adding you to CC List of the existing bugzilla bug"))
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Adding you to CC List of the existing bugzilla bug"))
|
|
Julius Milan |
c90eb8 |
run_event("watch_Bugzilla", dirname)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
email = os.getenv("uReport_ContactEmail")
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-dump-journal-xorg.c b/src/plugins/abrt-dump-journal-xorg.c
|
|
Julius Milan |
c90eb8 |
index 677183a..374519b 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-dump-journal-xorg.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-dump-journal-xorg.c
|
|
Julius Milan |
c90eb8 |
@@ -64,7 +64,7 @@ static GList *abrt_journal_extract_xorg_crashes(abrt_journal_t *journal)
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
while (abrt_journal_next(journal) > 0);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("Found crashes: %d", g_list_length(crash_info_list));
|
|
Julius Milan |
c90eb8 |
+ log_warning("Found crashes: %d", g_list_length(crash_info_list));
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
return crash_info_list;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-dump-oops.c b/src/plugins/abrt-dump-oops.c
|
|
Julius Milan |
c90eb8 |
index a348923..4348c4f 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-dump-oops.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-dump-oops.c
|
|
Julius Milan |
c90eb8 |
@@ -169,7 +169,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
unsigned errors = 0;
|
|
Julius Milan |
c90eb8 |
if (opts & OPT_u)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Updating problem directory");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Updating problem directory");
|
|
Julius Milan |
c90eb8 |
switch (g_list_length(oops_list))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
case 0:
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/abrt-watch-log.c b/src/plugins/abrt-watch-log.c
|
|
Julius Milan |
c90eb8 |
index 0587faf..f93df96 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/abrt-watch-log.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/abrt-watch-log.c
|
|
Julius Milan |
c90eb8 |
@@ -29,16 +29,16 @@ static bool memstr(void *buf, unsigned size, const char *str)
|
|
Julius Milan |
c90eb8 |
int len = strlen(str);
|
|
Julius Milan |
c90eb8 |
while ((int)size >= len)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- //log("LOOKING FOR:'%s'", str);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("LOOKING FOR:'%s'", str);
|
|
Julius Milan |
c90eb8 |
char *first = memchr(buf, (unsigned char)str[0], size - len + 1);
|
|
Julius Milan |
c90eb8 |
if (!first)
|
|
Julius Milan |
c90eb8 |
break;
|
|
Julius Milan |
c90eb8 |
- //log("FOUND:'%.66s'", first);
|
|
Julius Milan |
c90eb8 |
+ //log_warning("FOUND:'%.66s'", first);
|
|
Julius Milan |
c90eb8 |
first++;
|
|
Julius Milan |
c90eb8 |
if (len <= 1 || strncmp(first, str + 1, len - 1) == 0)
|
|
Julius Milan |
c90eb8 |
return true;
|
|
Julius Milan |
c90eb8 |
size -= (first - (char*)buf);
|
|
Julius Milan |
c90eb8 |
- //log("SKIP TO:'%.66s' %d chars", first, (int)(first - (char*)buf));
|
|
Julius Milan |
c90eb8 |
+ //log_warning("SKIP TO:'%.66s' %d chars", first, (int)(first - (char*)buf));
|
|
Julius Milan |
c90eb8 |
buf = first;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
return false;
|
|
Julius Milan |
c90eb8 |
@@ -117,7 +117,7 @@ static void run_scanner_prog(int fd, struct stat *statbuf, GList *match_list, ch
|
|
Julius Milan |
c90eb8 |
*/
|
|
Julius Milan |
c90eb8 |
if (lseek(fd, 0, SEEK_CUR) <= cur_pos)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Warning, '%s' did not process its input", prog[0]);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Warning, '%s' did not process its input", prog[0]);
|
|
Julius Milan |
c90eb8 |
lseek(fd, statbuf->st_size, SEEK_SET);
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/bodhi.c b/src/plugins/bodhi.c
|
|
Julius Milan |
c90eb8 |
index cd1d431..84ed4d8 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/bodhi.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/bodhi.c
|
|
Julius Milan |
c90eb8 |
@@ -416,7 +416,7 @@ static GHashTable *bodhi_query_list(const char *query, const char *release)
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
free(bodhi_url_bugs);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
-// log("%s", post_state->body);
|
|
Julius Milan |
c90eb8 |
+// log_warning("%s", post_state->body);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
json_object *json = json_tokener_parse(post_state->body);
|
|
Julius Milan |
c90eb8 |
if (is_error(json))
|
|
Julius Milan |
c90eb8 |
@@ -569,13 +569,13 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
if (query->buf[query->len - 1] == '&')
|
|
Julius Milan |
c90eb8 |
query->buf[query->len - 1] = '\0';
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log(_("Searching for updates"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Searching for updates"));
|
|
Julius Milan |
c90eb8 |
GHashTable *update_hash_tbl = bodhi_query_list(query->buf, release);
|
|
Julius Milan |
c90eb8 |
strbuf_free(query);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if (!update_hash_tbl || !g_hash_table_size(update_hash_tbl))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log(_("No updates for this package found"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("No updates for this package found"));
|
|
Julius Milan |
c90eb8 |
/*if (update_hash_tbl) g_hash_table_unref(update_hash_tbl);*/
|
|
Julius Milan |
c90eb8 |
return 0;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -604,7 +604,7 @@ int main(int argc, char **argv)
|
|
Julius Milan |
c90eb8 |
if (!q->len)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
/*strbuf_free(q);*/
|
|
Julius Milan |
c90eb8 |
- log(_("Local version of the package is newer than available updates"));
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Local version of the package is newer than available updates"));
|
|
Julius Milan |
c90eb8 |
return 0;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/src/plugins/oops-utils.c b/src/plugins/oops-utils.c
|
|
Julius Milan |
c90eb8 |
index 49a7d50..2e53cfc 100644
|
|
Julius Milan |
c90eb8 |
--- a/src/plugins/oops-utils.c
|
|
Julius Milan |
c90eb8 |
+++ b/src/plugins/oops-utils.c
|
|
Julius Milan |
c90eb8 |
@@ -29,7 +29,7 @@ int abrt_oops_process_list(GList *oops_list, const char *dump_location, const ch
|
|
Julius Milan |
c90eb8 |
int oops_cnt = g_list_length(oops_list);
|
|
Julius Milan |
c90eb8 |
if (oops_cnt != 0)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Found oopses: %d", oops_cnt);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Found oopses: %d", oops_cnt);
|
|
Julius Milan |
c90eb8 |
if ((flags & ABRT_OOPS_PRINT_STDOUT))
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
int i = 0;
|
|
Julius Milan |
c90eb8 |
@@ -38,7 +38,7 @@ int abrt_oops_process_list(GList *oops_list, const char *dump_location, const ch
|
|
Julius Milan |
c90eb8 |
char *kernel_bt = (char*)g_list_nth_data(oops_list, i++);
|
|
Julius Milan |
c90eb8 |
char *tainted_short = kernel_tainted_short(kernel_bt);
|
|
Julius Milan |
c90eb8 |
if (tainted_short)
|
|
Julius Milan |
c90eb8 |
- log("Kernel is tainted '%s'", tainted_short);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Kernel is tainted '%s'", tainted_short);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
free(tainted_short);
|
|
Julius Milan |
c90eb8 |
printf("\nVersion: %s", kernel_bt);
|
|
Julius Milan |
c90eb8 |
@@ -46,10 +46,10 @@ int abrt_oops_process_list(GList *oops_list, const char *dump_location, const ch
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
if (dump_location != NULL)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("Creating problem directories");
|
|
Julius Milan |
c90eb8 |
+ log_warning("Creating problem directories");
|
|
Julius Milan |
c90eb8 |
errors = abrt_oops_create_dump_dirs(oops_list, dump_location, analyzer, flags);
|
|
Julius Milan |
c90eb8 |
if (errors)
|
|
Julius Milan |
c90eb8 |
- log("%d errors while dumping oopses", errors);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%d errors while dumping oopses", errors);
|
|
Julius Milan |
c90eb8 |
/*
|
|
Julius Milan |
c90eb8 |
* This marker in syslog file prevents us from
|
|
Julius Milan |
c90eb8 |
* re-parsing old oopses. The only problem is that we
|
|
Julius Milan |
c90eb8 |
@@ -75,7 +75,7 @@ int abrt_oops_process_list(GList *oops_list, const char *dump_location, const ch
|
|
Julius Milan |
c90eb8 |
int n = unreported_cnt > 30 ? 30 : unreported_cnt;
|
|
Julius Milan |
c90eb8 |
n = n * n;
|
|
Julius Milan |
c90eb8 |
if (n > 9)
|
|
Julius Milan |
c90eb8 |
- log(_("Sleeping for %d seconds"), n);
|
|
Julius Milan |
c90eb8 |
+ log_warning(_("Sleeping for %d seconds"), n);
|
|
Julius Milan |
c90eb8 |
abrt_oops_signaled_sleep(n); /* max 15 mins */
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
diff --git a/tests/koops-parser.at b/tests/koops-parser.at
|
|
Julius Milan |
c90eb8 |
index 8d45558..4f9087a 100644
|
|
Julius Milan |
c90eb8 |
--- a/tests/koops-parser.at
|
|
Julius Milan |
c90eb8 |
+++ b/tests/koops-parser.at
|
|
Julius Milan |
c90eb8 |
@@ -23,9 +23,9 @@ int run_test(const struct test_struct *test)
|
|
Julius Milan |
c90eb8 |
version = NULL;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
if (version)
|
|
Julius Milan |
c90eb8 |
- log("version %s", version);
|
|
Julius Milan |
c90eb8 |
+ log_warning("version %s", version);
|
|
Julius Milan |
c90eb8 |
else
|
|
Julius Milan |
c90eb8 |
- log("version was not found");
|
|
Julius Milan |
c90eb8 |
+ log_warning("version was not found");
|
|
Julius Milan |
c90eb8 |
free(version);
|
|
Julius Milan |
c90eb8 |
fclose(fp);
|
|
Julius Milan |
c90eb8 |
return version == NULL;
|
|
Julius Milan |
c90eb8 |
@@ -69,19 +69,19 @@ int run_test(const struct test_struct *test, int flags)
|
|
Julius Milan |
c90eb8 |
case TEST_NOT_TAINTED:
|
|
Julius Milan |
c90eb8 |
if (tnt)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("%s '%s'", test->filename, tnt);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s '%s'", test->filename, tnt);
|
|
Julius Milan |
c90eb8 |
ret = 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
break;
|
|
Julius Milan |
c90eb8 |
case TEST_TAINTED:
|
|
Julius Milan |
c90eb8 |
if (!tnt)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("%s", test->filename);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s", test->filename);
|
|
Julius Milan |
c90eb8 |
ret = 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
else if (strcmp(tnt, test->expected_results) != 0)
|
|
Julius Milan |
c90eb8 |
{
|
|
Julius Milan |
c90eb8 |
- log("%s '%s'", test->filename, tnt);
|
|
Julius Milan |
c90eb8 |
+ log_warning("%s '%s'", test->filename, tnt);
|
|
Julius Milan |
c90eb8 |
ret = 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
};
|
|
Julius Milan |
c90eb8 |
@@ -105,12 +105,12 @@ int main(void)
|
|
Julius Milan |
c90eb8 |
{ EXAMPLE_PFX"/koops-tainted-insane", "ABCDEFGHIJKLMNOPQRSTUVWXYZ" },
|
|
Julius Milan |
c90eb8 |
};
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("test not tainted");
|
|
Julius Milan |
c90eb8 |
+ log_warning("test not tainted");
|
|
Julius Milan |
c90eb8 |
int ret = 0;
|
|
Julius Milan |
c90eb8 |
for (int i = 0; i < ARRAY_SIZE(not_tainted); ++i)
|
|
Julius Milan |
c90eb8 |
ret |= run_test(¬_tainted[i], TEST_NOT_TAINTED);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("test tainted");
|
|
Julius Milan |
c90eb8 |
+ log_warning("test tainted");
|
|
Julius Milan |
c90eb8 |
for (int i = 0; i < ARRAY_SIZE(tainted); ++i)
|
|
Julius Milan |
c90eb8 |
ret |= run_test(&tainted[i], TEST_TAINTED);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -139,7 +139,7 @@ int run_test(const struct test_struct *test)
|
|
Julius Milan |
c90eb8 |
if (!strcmp(hash_oops1, hash_oops2))
|
|
Julius Milan |
c90eb8 |
return 0;
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- log("'%s' != '%s'", hash_oops1, hash_oops2);
|
|
Julius Milan |
c90eb8 |
+ log_warning("'%s' != '%s'", hash_oops1, hash_oops2);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
return 1;
|
|
Julius Milan |
c90eb8 |
}
|
|
Julius Milan |
c90eb8 |
@@ -194,7 +194,7 @@ int run_test(const struct test_struct *test)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
int result = !(oops_list && !strcmp((char *)oops_list->data, oops_expected));
|
|
Julius Milan |
c90eb8 |
if (result)
|
|
Julius Milan |
c90eb8 |
- log("Obtained:\n'%s'\nExpected:\n'%s'", (char *)oops_list->data, oops_expected);
|
|
Julius Milan |
c90eb8 |
+ log_warning("Obtained:\n'%s'\nExpected:\n'%s'", (char *)oops_list->data, oops_expected);
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
g_list_free_full(oops_list, free);
|
|
Julius Milan |
c90eb8 |
free(oops_expected_bck);
|
|
Julius Milan |
c90eb8 |
diff --git a/tests/runtests/upload-ftp/ftpserver.py b/tests/runtests/upload-ftp/ftpserver.py
|
|
Julius Milan |
c90eb8 |
index 1cab50a..753b4ae 100644
|
|
Julius Milan |
c90eb8 |
--- a/tests/runtests/upload-ftp/ftpserver.py
|
|
Julius Milan |
c90eb8 |
+++ b/tests/runtests/upload-ftp/ftpserver.py
|
|
Julius Milan |
c90eb8 |
@@ -456,7 +456,7 @@ class _FileReadWriteError(OSError):
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
# --- loggers
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
-def log(msg):
|
|
Julius Milan |
c90eb8 |
+def log_warning(msg):
|
|
Julius Milan |
c90eb8 |
"""Log messages intended for the end user."""
|
|
Julius Milan |
c90eb8 |
print msg
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
@@ -2554,9 +2554,9 @@ class FTPHandler(object, asynchat.async_chat):
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
# --- logging wrappers
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
- def log(self, msg):
|
|
Julius Milan |
c90eb8 |
+ def log_warning(self, msg):
|
|
Julius Milan |
c90eb8 |
"""Log a message, including additional identifying session data."""
|
|
Julius Milan |
c90eb8 |
- log("[%s]@%s:%s %s" % (self.username, self.remote_ip,
|
|
Julius Milan |
c90eb8 |
+ log_warning("[%s]@%s:%s %s" % (self.username, self.remote_ip,
|
|
Julius Milan |
c90eb8 |
self.remote_port, msg))
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
def logline(self, msg):
|
|
Julius Milan |
c90eb8 |
@@ -3744,7 +3744,7 @@ class FTPServer(object, asyncore.dispatcher):
|
|
Julius Milan |
c90eb8 |
poll_fun = asyncore.poll
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
if count is None:
|
|
Julius Milan |
c90eb8 |
- log("Starting FTP server")
|
|
Julius Milan |
c90eb8 |
+ log_warning("Starting FTP server")
|
|
Julius Milan |
c90eb8 |
try:
|
|
Julius Milan |
c90eb8 |
try:
|
|
Julius Milan |
c90eb8 |
while asyncore.socket_map or _scheduler._tasks:
|
|
Julius Milan |
c90eb8 |
@@ -3753,7 +3753,7 @@ class FTPServer(object, asyncore.dispatcher):
|
|
Julius Milan |
c90eb8 |
except (KeyboardInterrupt, SystemExit, asyncore.ExitNow):
|
|
Julius Milan |
c90eb8 |
pass
|
|
Julius Milan |
c90eb8 |
finally:
|
|
Julius Milan |
c90eb8 |
- log("Shutting down FTP server")
|
|
Julius Milan |
c90eb8 |
+ log_warning("Shutting down FTP server")
|
|
Julius Milan |
c90eb8 |
cls.close_all()
|
|
Julius Milan |
c90eb8 |
else:
|
|
Julius Milan |
c90eb8 |
while (asyncore.socket_map or _scheduler._tasks) and count > 0:
|
|
Julius Milan |
c90eb8 |
@@ -3786,7 +3786,7 @@ class FTPServer(object, asyncore.dispatcher):
|
|
Julius Milan |
c90eb8 |
handler = self.handler(sock, self)
|
|
Julius Milan |
c90eb8 |
if not handler.connected:
|
|
Julius Milan |
c90eb8 |
return
|
|
Julius Milan |
c90eb8 |
- log("[]%s:%s Connected." % addr[:2])
|
|
Julius Milan |
c90eb8 |
+ log_warning("[]%s:%s Connected." % addr[:2])
|
|
Julius Milan |
c90eb8 |
ip = addr[0]
|
|
Julius Milan |
c90eb8 |
self.ip_map.append(ip)
|
|
Julius Milan |
c90eb8 |
|
|
Julius Milan |
c90eb8 |
--
|
|
Julius Milan |
c90eb8 |
2.9.5
|
|
Julius Milan |
c90eb8 |
|