coverity: validate retval from get_only_segment_using_this_lv

diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index 282dd4f151f7bd7abf1f127be423e8f805a91131..b3ed5131adf22fc98b44528cecbcdd0654a7cb75 100644 (file)
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -781,7 +781,8 @@ int lv_info_with_seg_status(struct cmd_context *cmd,
        if (lv_is_used_cache_pool(lv)) {
                /* INFO is not set as cache-pool cannot be active.
                 * STATUS is collected from cache LV */
-               lv_seg = get_only_segment_using_this_lv(lv);
+               if (!(lv_seg = get_only_segment_using_this_lv(lv)))
+                       return_0;
                (void) _lv_info(cmd, lv_seg->lv, 1, NULL, lv_seg, &status->seg_status, 0, 0);
                return 1;
        }

diff --git a/lib/display/display.c b/lib/display/display.c
index 18e49f977e857562e85faf07981eb56f23571d33..9a928d92d34269a2eca1f18c6812531bdf8af2ab 100644 (file)
--- a/lib/display/display.c
+++ b/lib/display/display.c
@@ -517,8 +517,8 @@ int lvdisplay_full(struct cmd_context *cmd,
                log_print("LV Pool metadata       %s", seg->metadata_lv->name);
                log_print("LV Pool data           %s", seg_lv(seg, 0)->name);
        } else if (lv_is_cache_origin(lv)) {
-               log_print("LV origin of Cache LV  %s",
-                         get_only_segment_using_this_lv(lv)->lv->name);
+               if ((seg = get_only_segment_using_this_lv(lv)))
+                       log_print("LV origin of Cache LV  %s", seg->lv->name);
        } else if (lv_is_cache(lv)) {
                seg = first_seg(lv);
                if (inkernel && !lv_cache_status(lv, &cache_status))