From 131ca0eb9582dd183ed40400caf4fc832f43cb16 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Tue, 9 Mar 2021 18:23:42 +0100 Subject: [PATCH] activation: use existing LV as best effort Returning NULL for lv_committed is basically instant crash, so instead try with passed LV instead. It shouldn't matter as this is internall error path anyway, but coverity should be happier. --- lib/metadata/metadata.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c index f82ada8be..71e59207d 100644 --- a/lib/metadata/metadata.c +++ b/lib/metadata/metadata.c @@ -4335,7 +4335,7 @@ char *tags_format_and_copy(struct dm_pool *mem, const struct dm_list *tagsl) const struct logical_volume *lv_committed(const struct logical_volume *lv) { struct volume_group *vg; - struct logical_volume *found_lv; + const struct logical_volume *found_lv; if (!lv) return NULL; @@ -4348,7 +4348,7 @@ const struct logical_volume *lv_committed(const struct logical_volume *lv) if (!(found_lv = find_lv_in_vg_by_lvid(vg, &lv->lvid))) { log_error(INTERNAL_ERROR "LV %s (UUID %s) not found in committed metadata.", display_lvname(lv), lv->lvid.s); - return NULL; + found_lv = lv; /* Use uncommitted LV as best effort */ } return found_lv;