This patch removes 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()

Chrissie
Index: services/cfg.c
===================================================================
--- services/cfg.c      (revision 2654)
+++ services/cfg.c      (working copy)
@@ -547,10 +547,7 @@
        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);
 }
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to