From 9fe13294a98cb005c9fb0038d01b77c0d9cbfd12 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Wed, 25 Jun 2014 23:38:49 -0400 Subject: [PATCH] coredump+coredumpctl: add COREDUMP_FILENAME, use in coredumpctl --- src/journal/coredump.c | 22 ++-- src/journal/coredumpctl.c | 301 +++++++++++++++++++++------------------------- 2 files changed, 153 insertions(+), 170 deletions(-) diff --git a/src/journal/coredump.c b/src/journal/coredump.c index afd69cd..bb6dcba 100644 --- a/src/journal/coredump.c +++ b/src/journal/coredump.c @@ -465,13 +465,13 @@ int main(int argc, char* argv[]) { _cleanup_free_ char *core_pid = NULL, *core_uid = NULL, *core_gid = NULL, *core_signal = NULL, *core_timestamp = NULL, *core_comm = NULL, *core_exe = NULL, *core_unit = NULL, *core_session = NULL, *core_message = NULL, *core_cmdline = NULL, *coredump_data = NULL, - *coredump_filename = NULL, *core_slice = NULL, *core_cgroup = NULL, *core_owner_uid = NULL, - *exe = NULL, *comm = NULL; + *core_slice = NULL, *core_cgroup = NULL, *core_owner_uid = NULL, + *exe = NULL, *comm = NULL, *filename = NULL; const char *info[_INFO_LEN]; _cleanup_close_ int coredump_fd = -1; - struct iovec iovec[17]; + struct iovec iovec[18]; off_t coredump_size; int r, j = 0; uid_t uid, owner_uid; @@ -548,15 +548,15 @@ int main(int argc, char* argv[]) { if (arg_storage != COREDUMP_STORAGE_NONE) arg_storage = COREDUMP_STORAGE_EXTERNAL; - r = save_external_coredump(info, uid, &coredump_filename, &coredump_fd, &coredump_size); + r = save_external_coredump(info, uid, &filename, &coredump_fd, &coredump_size); if (r < 0) goto finish; - r = maybe_remove_external_coredump(coredump_filename, coredump_size); + r = maybe_remove_external_coredump(filename, coredump_size); if (r < 0) goto finish; - log_info("Detected coredump of the journal daemon itself, diverted to %s.", coredump_filename); + log_info("Detected coredump of the journal daemon itself, diverted to %s.", filename); goto finish; } @@ -647,7 +647,7 @@ int main(int argc, char* argv[]) { IOVEC_SET_STRING(iovec[j++], "PRIORITY=2"); /* Always stream the coredump to disk, if that's possible */ - r = save_external_coredump(info, uid, &coredump_filename, &coredump_fd, &coredump_size); + r = save_external_coredump(info, uid, &filename, &coredump_fd, &coredump_size); if (r < 0) /* skip whole core dumping part */ goto log; @@ -656,9 +656,15 @@ int main(int argc, char* argv[]) { * now, as later on we will lack the privileges for * it. However, we keep the fd to it, so that we can still * process it and log it. */ - r = maybe_remove_external_coredump(coredump_filename, coredump_size); + r = maybe_remove_external_coredump(filename, coredump_size); if (r < 0) goto finish; + if (r == 0) { + const char *coredump_filename; + + coredump_filename = strappenda("COREDUMP_FILENAME=", filename); + IOVEC_SET_STRING(iovec[j++], coredump_filename); + } /* Now, let's drop privileges to become the user who owns the * segfaulted process and allocate the coredump memory under diff --git a/src/journal/coredumpctl.c b/src/journal/coredumpctl.c index 48e6341..08d8cdf 100644 --- a/src/journal/coredumpctl.c +++ b/src/journal/coredumpctl.c @@ -37,6 +37,7 @@ #include "macro.h" #include "journal-internal.h" #include "copy.h" +#include "compress.h" static enum { ACTION_NONE, @@ -280,55 +281,6 @@ static int retrieve(const void *data, return 0; } -#define filename_escape(s) xescape((s), "./ ") - -static int make_coredump_path(sd_journal *j, char **ret) { - _cleanup_free_ char - *pid = NULL, *boot_id = NULL, *tstamp = NULL, *comm = NULL, - *p = NULL, *b = NULL, *t = NULL, *c = NULL; - const void *d; - size_t l; - char *fn; - - assert(j); - assert(ret); - - SD_JOURNAL_FOREACH_DATA(j, d, l) { - retrieve(d, l, "COREDUMP_COMM", &comm); - retrieve(d, l, "COREDUMP_PID", &pid); - retrieve(d, l, "COREDUMP_TIMESTAMP", &tstamp); - retrieve(d, l, "_BOOT_ID", &boot_id); - } - - if (!pid || !comm || !tstamp || !boot_id) { - log_error("Failed to retrieve necessary fields to find coredump on disk."); - return -ENOENT; - } - - p = filename_escape(pid); - if (!p) - return log_oom(); - - t = filename_escape(tstamp); - if (!t) - return log_oom(); - - c = filename_escape(comm); - if (!t) - return log_oom(); - - b = filename_escape(boot_id); - if (!b) - return log_oom(); - - fn = strjoin("/var/lib/systemd/coredump/core.", c, ".", b, ".", p, ".", t, NULL); - if (!fn) - return log_oom(); - - *ret = fn; - return 0; -} - static void print_field(FILE* file, sd_journal *j) { _cleanup_free_ char *value = NULL; const void *d; @@ -349,12 +301,14 @@ static void print_field(FILE* file, sd_journal *j) { static int print_list(FILE* file, sd_journal *j, int had_legend) { _cleanup_free_ char *pid = NULL, *uid = NULL, *gid = NULL, - *sgnl = NULL, *exe = NULL, *comm = NULL, *cmdline = NULL; + *sgnl = NULL, *exe = NULL, *comm = NULL, *cmdline = NULL, + *filename = NULL; const void *d; size_t l; usec_t t; char buf[FORMAT_TIMESTAMP_MAX]; int r; + bool present; assert(file); assert(j); @@ -367,9 +321,10 @@ static int print_list(FILE* file, sd_journal *j, int had_legend) { retrieve(d, l, "COREDUMP_EXE", &exe); retrieve(d, l, "COREDUMP_COMM", &comm); retrieve(d, l, "COREDUMP_CMDLINE", &cmdline); + retrieve(d, l, "COREDUMP_FILENAME", &filename); } - if (!pid && !uid && !gid && !sgnl && !exe && !comm && !cmdline) { + if (!pid && !uid && !gid && !sgnl && !exe && !comm && !cmdline && !filename) { log_warning("Empty coredump log entry"); return -EINVAL; } @@ -381,22 +336,25 @@ static int print_list(FILE* file, sd_journal *j, int had_legend) { } format_timestamp(buf, sizeof(buf), t); + present = filename && access(filename, F_OK) == 0; if (!had_legend && !arg_no_legend) - fprintf(file, "%-*s %*s %*s %*s %*s %s\n", + fprintf(file, "%-*s %*s %*s %*s %*s %*s %s\n", FORMAT_TIMESTAMP_WIDTH, "TIME", 6, "PID", 5, "UID", 5, "GID", 3, "SIG", + 1, "PRESENT", "EXE"); - fprintf(file, "%-*s %*s %*s %*s %*s %s\n", + fprintf(file, "%-*s %*s %*s %*s %*s %*s %s\n", FORMAT_TIMESTAMP_WIDTH, buf, 6, strna(pid), 5, strna(uid), 5, strna(gid), 3, strna(sgnl), + 1, present ? "*" : "", strna(exe ?: (comm ?: cmdline))); return 0; @@ -408,8 +366,8 @@ static int print_info(FILE *file, sd_journal *j, bool need_space) { *sgnl = NULL, *exe = NULL, *comm = NULL, *cmdline = NULL, *unit = NULL, *user_unit = NULL, *session = NULL, *boot_id = NULL, *machine_id = NULL, *hostname = NULL, - *coredump = NULL, *slice = NULL, *cgroup = NULL, - *owner_uid = NULL, *message = NULL, *timestamp = NULL; + *slice = NULL, *cgroup = NULL, *owner_uid = NULL, + *message = NULL, *timestamp = NULL, *filename = NULL; const void *d; size_t l; int r; @@ -432,6 +390,7 @@ static int print_info(FILE *file, sd_journal *j, bool need_space) { retrieve(d, l, "COREDUMP_SLICE", &slice); retrieve(d, l, "COREDUMP_CGROUP", &cgroup); retrieve(d, l, "COREDUMP_TIMESTAMP", ×tamp); + retrieve(d, l, "COREDUMP_FILENAME", &filename); retrieve(d, l, "_BOOT_ID", &boot_id); retrieve(d, l, "_MACHINE_ID", &machine_id); retrieve(d, l, "_HOSTNAME", &hostname); @@ -546,9 +505,8 @@ static int print_info(FILE *file, sd_journal *j, bool need_space) { if (hostname) fprintf(file, " Hostname: %s\n", hostname); - if (make_coredump_path(j, &coredump) >= 0) - if (access(coredump, F_OK) >= 0) - fprintf(file, " Coredump: %s\n", coredump); + if (filename && access(filename, F_OK) == 0) + fprintf(file, " Coredump: %s\n", filename); if (message) { _cleanup_free_ char *m = NULL; @@ -619,66 +577,131 @@ static int dump_list(sd_journal *j) { return 0; } -static int dump_core(sd_journal* j) { - const void *data; - size_t len, ret; +static int save_core(sd_journal *j, int fd, char **path, bool *unlink_temp) { + const char *data; + _cleanup_free_ char *filename = NULL; + size_t len; int r; - assert(j); + assert((fd >= 0) != !!path); + assert(!!path == !!unlink_temp); - /* We want full data, nothing truncated. */ - sd_journal_set_data_threshold(j, 0); + /* Prefer uncompressed file to journal (probably cached) to + * compressed file (probably uncached). */ + r = sd_journal_get_data(j, "COREDUMP_FILENAME", (const void**) &data, &len); + if (r < 0 && r != -ENOENT) + log_warning("Failed to retrieve COREDUMP_FILENAME: %s", strerror(-r)); + else if (r == 0) + retrieve(data, len, "COREDUMP_FILENAME", &filename); - r = focus(j); - if (r < 0) - return r; - - print_info(output ? stdout : stderr, j, false); - - if (on_tty() && !output) { - log_error("Refusing to dump core to tty."); - return -ENOTTY; + if (filename && access(filename, R_OK) < 0) { + log_debug("File %s is not readable: %m", filename); + free(filename); + filename = NULL; } - r = sd_journal_get_data(j, "COREDUMP", (const void**) &data, &len); - if (r == ENOENT) { - _cleanup_free_ char *fn = NULL; - _cleanup_close_ int fd = -1; + if (filename && !endswith(filename, ".xz")) { + *path = filename; + filename = NULL; - r = make_coredump_path(j, &fn); - if (r < 0) - return r; + return 0; + } else { + _cleanup_close_ int fdt = -1; + char *temp = NULL; - fd = open(fn, O_RDONLY|O_CLOEXEC|O_NOCTTY); if (fd < 0) { - if (errno == ENOENT) + temp = strdup("/var/tmp/coredump-XXXXXX"); + if (!temp) + return log_oom(); + + fdt = mkostemp_safe(temp, O_WRONLY|O_CLOEXEC); + if (fdt < 0) { + log_error("Failed to create temporary file: %m"); + return -errno; + } + log_debug("Created temporary file %s", temp); + + fd = fdt; + } + + r = sd_journal_get_data(j, "COREDUMP", (const void**) &data, &len); + if (r == 0) { + ssize_t sz; + + assert(len >= 9); + data += 9; + len -= 9; + + sz = write(fdt, data, len); + if (sz < 0) { + log_error("Failed to write temporary file: %m"); + r = -errno; + goto error; + } + if (sz != (ssize_t) len) { + log_error("Short write to temporary file."); + r = -EIO; + goto error; + } + } else if (filename) { + _cleanup_close_ int fdf; + + fdf = open(filename, O_RDONLY | O_CLOEXEC); + if (fdf < 0) { + log_error("Failed to open %s: %m", filename); + r = -errno; + goto error; + } + + r = decompress_stream(fdf, fd, -1); + if (r < 0) { + log_error("Failed to decompress %s: %s", filename, strerror(-r)); + goto error; + } + } else { + if (r == -ENOENT) log_error("Coredump neither in journal file nor stored externally on disk."); else - log_error("Failed to open coredump file: %s", strerror(-r)); + log_error("Failed to retrieve COREDUMP field: %s", strerror(-r)); + goto error; + } - return -errno; + if (temp) { + *path = temp; + *unlink_temp = true; } - r = copy_bytes(fd, output ? fileno(output) : STDOUT_FILENO, (off_t) -1); - if (r < 0) { - log_error("Failed to stream coredump: %s", strerror(-r)); - return r; + return 0; + +error: + if (temp) { + unlink(temp); + log_debug("Removed temporary file %s", temp); } + return r; + } +} - } else if (r < 0) { - log_error("Failed to retrieve COREDUMP field: %s", strerror(-r)); +static int dump_core(sd_journal* j) { + int r; + + assert(j); + + r = focus(j); + if (r < 0) return r; - } else { - assert(len >= 9); - data = (const uint8_t*) data + 9; - len -= 9; - - ret = fwrite(data, len, 1, output ?: stdout); - if (ret != 1) { - log_error("Dumping coredump failed: %m (%zu)", ret); - return -errno; - } + print_info(output ? stdout : stderr, j, false); + + if (on_tty() && !output) { + log_error("Refusing to dump core to tty."); + return -ENOTTY; + } + + r = save_core(j, output ? fileno(output) : STDOUT_FILENO, NULL, NULL); + if (r < 0) { + log_error("Coredump retrieval failed: %s", strerror(-r)); + return r; } r = sd_journal_previous(j); @@ -690,11 +713,9 @@ static int dump_core(sd_journal* j) { static int run_gdb(sd_journal *j) { - _cleanup_free_ char *exe = NULL, *coredump = NULL; - char temp[] = "/var/tmp/coredump-XXXXXX"; - bool unlink_temp = false; - const char *path; - const void *data; + _cleanup_free_ char *exe = NULL, *coredump = NULL, *path = NULL; + bool unlink_path = false; + const char *data; siginfo_t st; size_t len; pid_t pid; @@ -702,8 +723,6 @@ static int run_gdb(sd_journal *j) { assert(j); - sd_journal_set_data_threshold(j, 0); - r = focus(j); if (r < 0) return r; @@ -717,9 +736,9 @@ static int run_gdb(sd_journal *j) { return r; } - assert(len >= 13); - data = (const uint8_t*) data + 13; - len -= 13; + assert(len > strlen("COREDUMP_EXE=")); + data += strlen("COREDUMP_EXE="); + len -= strlen("COREDUMP_EXE="); exe = strndup(data, len); if (!exe) @@ -735,57 +754,10 @@ static int run_gdb(sd_journal *j) { return -ENOENT; } - r = sd_journal_get_data(j, "COREDUMP", (const void**) &data, &len); - if (r == -ENOENT) { - - r = make_coredump_path(j, &coredump); - if (r < 0) - return r; - - if (access(coredump, R_OK) < 0) { - if (errno == ENOENT) - log_error("Coredump neither in journal file nor stored externally on disk."); - else - log_error("Failed to access coredump file: %m"); - - return -errno; - } - - path = coredump; - - } else if (r < 0) { - log_error("Failed to retrieve COREDUMP field: %s", strerror(-r)); + r = save_core(j, -1, &path, &unlink_path); + if (r < 0) { + log_error("Failed to retrieve core: %s", strerror(-r)); return r; - - } else { - _cleanup_close_ int fd = -1; - ssize_t sz; - - assert(len >= 9); - data = (const uint8_t*) data + 9; - len -= 9; - - fd = mkostemp_safe(temp, O_WRONLY|O_CLOEXEC); - if (fd < 0) { - log_error("Failed to create temporary file: %m"); - return -errno; - } - - unlink_temp = true; - - sz = write(fd, data, len); - if (sz < 0) { - log_error("Failed to write temporary file: %m"); - r = -errno; - goto finish; - } - if (sz != (ssize_t) len) { - log_error("Short write to temporary file."); - r = -EIO; - goto finish; - } - - path = temp; } pid = fork(); @@ -810,8 +782,10 @@ static int run_gdb(sd_journal *j) { r = st.si_code == CLD_EXITED ? st.si_status : 255; finish: - if (unlink_temp) - unlink(temp); + if (unlink_path) { + log_debug("Removed temporary file %s", path); + unlink(path); + } return r; } @@ -846,6 +820,9 @@ int main(int argc, char *argv[]) { goto end; } + /* We want full data, nothing truncated. */ + sd_journal_set_data_threshold(j, 0); + SET_FOREACH(match, matches, it) { r = sd_journal_add_match(j, match, strlen(match)); if (r != 0) { -- 2.7.4