chiark / gitweb /
systemd-coredump: Log crashes without coredumps on failure
authorAndrew Cook <ariscop@gmail.com>
Wed, 4 Sep 2013 13:27:52 +0000 (23:27 +1000)
committerHarald Hoyer <harald@redhat.com>
Fri, 6 Sep 2013 08:09:55 +0000 (10:09 +0200)
Make a best-effort attempt to store information about crashes during
failure, currently if these are encountered the crash is completely
silenced.

ideally coredumpctl would show if a coredump is available.

src/journal/coredump.c

index a7d3c34fe4c7881eecbc0a62e617122433e19708..68c353fe833445e9963d46e15a476d173c27a99a 100644 (file)
@@ -241,7 +241,7 @@ int main(int argc, char* argv[]) {
         coredump_data = malloc(coredump_bufsize);
         if (!coredump_data) {
                 r = log_oom();
-                goto finish;
+                goto finalize;
         }
 
         memcpy(coredump_data, "COREDUMP=", 9);
@@ -261,12 +261,12 @@ int main(int argc, char* argv[]) {
 
                 if(coredump_size > COREDUMP_MAX) {
                         log_error("Coredump too large, ignoring");
-                        goto finish;
+                        goto finalize;
                 }
 
                 if (!GREEDY_REALLOC(coredump_data, coredump_bufsize, coredump_size + 1)) {
                         r = log_oom();
-                        goto finish;
+                        goto finalize;
                 }
         }
 
@@ -274,6 +274,7 @@ int main(int argc, char* argv[]) {
         iovec[j].iov_len = coredump_size;
         j++;
 
+finalize:
         r = sd_journal_sendv(iovec, j);
         if (r < 0)
                 log_error("Failed to send coredump: %s", strerror(-r));