From b2a326b511950c301fceb7f4901671af53f0e8af Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Mon, 28 Sep 2020 23:12:48 +0200 Subject: [PATCH] libdm: validate thin-pool before sending messages Alhtough lvm2 does validation on its side, ensure DM code is not sending messages to failed thin pool. --- libdm/libdm-deptree.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/libdm/libdm-deptree.c b/libdm/libdm-deptree.c index ffd084e52..59616eb3a 100644 --- a/libdm/libdm-deptree.c +++ b/libdm/libdm-deptree.c @@ -1600,6 +1600,15 @@ static int _node_send_messages(struct dm_tree_node *dnode, if (!have_messages || !send) return 1; /* transaction_id is matching */ + if (stp.fail || stp.read_only || stp.needs_check) { + log_error("Cannot send messages to thin pool %s%s%s%s.", + _node_name(dnode), + stp.fail ? " in failed state" : "", + stp.read_only ? " with read only metadata" : "", + stp.needs_check ? " which needs check first" : ""); + return 0; + } + dm_list_iterate_items(tmsg, &seg->thin_messages) { if (!(_thin_pool_node_message(dnode, tmsg))) return_0;