d3a4de
commit 1ce4508c92493cf03ea1b3c42e83540b387884fa
d3a4de
Author: Antonio Torres <antorres@redhat.com>
d3a4de
Date:   Fri Jul 2 07:12:48 2021 -0400
d3a4de
Subject: [PATCH] debug: don't set resource hard limit to zero
d3a4de
d3a4de
    Setting the resource hard limit to zero is irreversible, meaning if it
d3a4de
    is set to zero then there is no way to set it higher. This means
d3a4de
    enabling core dump is not possible, since setting a new resource limit
d3a4de
    for RLIMIT_CORE would fail. By only setting the soft limit to zero, we
d3a4de
    can disable and enable core dumps without failures.
d3a4de
d3a4de
    This fix is present in both main and 3.0.x upstream branches.
d3a4de
    
d3a4de
    Ticket in RHEL Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1977572
d3a4de
    Signed-off-by: Antonio Torres antorres@redhat.com
d3a4de
---
d3a4de
 src/lib/debug.c | 2 +-
d3a4de
 1 file changed, 1 insertion(+), 1 deletion(-)
d3a4de
d3a4de
diff --git a/src/lib/debug.c b/src/lib/debug.c
d3a4de
index 576bcb2a65..6330c9cb66 100644
d3a4de
--- a/src/lib/debug.c
d3a4de
+++ b/src/lib/debug.c
d3a4de
@@ -599,7 +599,7 @@ int fr_set_dumpable(bool allow_core_dumps)
d3a4de
 		struct rlimit no_core;
d3a4de
 
d3a4de
 		no_core.rlim_cur = 0;
d3a4de
-		no_core.rlim_max = 0;
d3a4de
+		no_core.rlim_max = core_limits.rlim_max;
d3a4de
 
d3a4de
 		if (setrlimit(RLIMIT_CORE, &no_core) < 0) {
d3a4de
 			fr_strerror_printf("Failed disabling core dumps: %s", fr_syserror(errno));