From: Ben Pfaff <blp@nicira.com>
Date: Tue, 11 May 2010 17:56:10 +0000 (-0700)
Subject: daemon: Allow monitored daemon to dump core no more than once.
X-Git-Tag: v1.0.0~14
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=7c2dd4c64823397d82d7d1c4e44f6a420df52aa6;p=sliver-openvswitch.git

daemon: Allow monitored daemon to dump core no more than once.

If the monitored daemon dumps core frequently, then this can quickly
exhaust the host's disk space.  This commit limits core dumps to at most
one per monitored session (typically, once per boot).
---

diff --git a/lib/daemon.c b/lib/daemon.c
index 081912b12..292be1465 100644
--- a/lib/daemon.c
+++ b/lib/daemon.c
@@ -353,6 +353,18 @@ monitor_daemon(pid_t daemon_pid)
             free(s);
 
             if (should_restart(status)) {
+                if (WCOREDUMP(status)) {
+                    /* Disable further core dumps to save disk space. */
+                    struct rlimit r;
+
+                    r.rlim_cur = 0;
+                    r.rlim_max = 0;
+                    if (setrlimit(RLIMIT_CORE, &r) == -1) {
+                        VLOG_WARN("failed to disable core dumps: %s",
+                                  strerror(errno));
+                    }
+                }
+
                 VLOG_ERR("%s, restarting", status_msg);
                 daemon_pid = fork_and_wait_for_startup(&daemonize_fd);
                 if (!daemon_pid) {