From a22f051d0428b39f5253cc4e65a94a719e56418e Mon Sep 17 00:00:00 2001 From: Christine Caulfield Date: Fri, 12 Feb 2010 07:46:02 +0000 Subject: [PATCH] Remove a double list_del() when a tracking CFG client shuts down without calling cfg_track_stop. This caused corosync to crash. The extra list_empty() check is redundant too because it also happens in remove_ci_from_shutdown() git-svn-id: http://svn.fedorahosted.org/svn/corosync/trunk@2655 fd59a12c-fef9-0310-b244-a6a79926bd2f --- services/cfg.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/services/cfg.c b/services/cfg.c index 6c7eff10..08028f24 100644 --- a/services/cfg.c +++ b/services/cfg.c @@ -547,10 +547,7 @@ int cfg_lib_exit_fn (void *conn) struct cfg_info *ci = (struct cfg_info *)api->ipc_private_data_get (conn); ENTER(); - if (!list_empty(&ci->list)) { - list_del(&ci->list); - remove_ci_from_shutdown(ci); - } + remove_ci_from_shutdown(ci); LEAVE(); return (0); }