Просмотр исходного кода

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

Christine Caulfield 16 лет назад
Родитель
Сommit
a22f051d04
1 измененных файлов с 1 добавлено и 4 удалено
  1. 1 4
      services/cfg.c

+ 1 - 4
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);
 }