[lvm-devel] [PATCH 3/4] lvconvert: _lvconvert_splitsnapshot check if lv is snapshot type

Wu Guanghao wuguanghao3 at huawei.com
Fri Sep 11 09:42:16 UTC 2020


cow may not be a snapshot type, the return value of origin_from_cow(cow) may be NULL

Signed-off-by: Wu Guanghao <wuguanghao3 at huawei.com>
Signed-off-by: Zhiqiang Liu <liuzhiqiang26 at huawei.com>
---
 tools/lvconvert.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tools/lvconvert.c b/tools/lvconvert.c
index 0adc61e36..f6a5572c7 100644
--- a/tools/lvconvert.c
+++ b/tools/lvconvert.c
@@ -1825,6 +1825,9 @@ static int _lvconvert_splitsnapshot(struct cmd_context *cmd, struct logical_volu
 	struct volume_group *vg = cow->vg;
 	const char *cow_name = display_lvname(cow);

+	if (!lv_is_snapshot(cow))
+		return 0;
+
 	if (lv_is_virtual_origin(origin_from_cow(cow))) {
 		log_error("Unable to split off snapshot %s with virtual origin.", cow_name);
 		return 0;
-- 




More information about the lvm-devel mailing list