From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Aring Date: Wed, 5 Oct 2022 15:23:11 -0400 Subject: [Cluster-devel] [PATCHv2 dlm-tool 1/2] dlm_controld: be sure we close logging at last Message-ID: <20221005192312.4130838-1-aahringo@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit I currently try to debug the following: (gdb) bt 0 _int_malloc (av=av at entry=0x7fd353ac2bc0 , bytes=bytes at entry=8192) at malloc.c:3755 1 0x00007fd3537a04a6 in __libc_calloc (n=n at entry=1, elem_size=elem_size at entry=8192) at malloc.c:3445 2 0x00007fd353792bd7 in __GI___open_memstream (bufloc=bufloc at entry=0x7ffc4edd2ea0, sizeloc=sizeloc at entry=0x7ffc4edd2ea8) at memstream.c:83 3 0x00007fd35382cba4 in __GI___vsyslog_chk (pri=163, flag=1, fmt=0x5560190522da "%s", ap=0x7ffc4edd2f90) at ../misc/syslog.c:167 4 0x00007fd35382d1e3 in __syslog_chk (pri=pri at entry=3, flag=flag at entry=1, fmt=fmt at entry=0x5560190522da "%s") at ../misc/syslog.c:129 5 0x000055601904e114 in syslog (__fmt=0x5560190522da "%s", __pri=3) at /usr/include/bits/syslog.h:31 6 log_level (name_in=, level_in=, fmt=0x55601905243e "abandoned lockspace %s") at logging.c:166 7 0x000055601903a91e in loop () at main.c:1597 8 main (argc=, argv=) at main.c:2161 We see that the last thing in dlm_controld was log_level() then it crashed internal handling of libc and syslog(). (gdb) f 6 6 log_level (name_in=, level_in=, fmt=0x55601905243e "abandoned lockspace %s") at logging.c:166 166 syslog(level, "%s", log_str); We see that log_level() was called with a format string of "abandoned lockspace %s" and we only do that after leaving the main loop, dlm_controld was going to shutdown and crashed. The reason is that at this time the syslog logging was already closed by closelog() and we still tried to call syslog() and libc doesn't like it. We should be sure closing the log functionality is the last thing to do when exiting dlm_controld. This patch is doing that so that dlm_controld should not crash anymore. Reported-by: Barry Marson --- v2: - remove sob. dlm_controld/main.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlm_controld/main.c b/dlm_controld/main.c index 80fe14bd..7cf6348e 100644 --- a/dlm_controld/main.c +++ b/dlm_controld/main.c @@ -1604,12 +1604,14 @@ static int loop(void) close_plocks(); close_cpg_daemon(); clear_configfs(); - close_logging(); close_cluster(); close_cluster_cfg(); list_for_each_entry(ls, &lockspaces, list) log_error("abandoned lockspace %s", ls->name); + + /* must be end */ + close_logging(); return rv; } -- 2.31.1