cache: fix lvdisplay --maps

'lvdisplay -m' tried to go through NULL policy settings,
when such policy was not defined for CachedLV.

Patch is fixing display of cache-pool without defined settings,
as this is now a valid pool and we mostly want users to define
these settings when actually really caching a LV.

diff --git a/WHATS_NEW b/WHATS_NEW
index 2f1ec8aeed09ec4bd07bc839934682aa96542a75..12368dd679a5ba9b63575380d49f787e204e901a 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.172 - 
 ===============================
+  Fix lvdisplay --maps for cache pool without policy settings.
   Avoid using origin_only manipulation with cached device.
   Support aborting of flushing cache LV.
   Reenable conversion of data and metadata thin-pool volumes to raid.

diff --git a/lib/cache_segtype/cache.c b/lib/cache_segtype/cache.c
index da23dae3ca9e00705812d82384f1615b88074a54..7461b21f5ea0e158cab8dfc5eb7a2a4a6c628df0 100644 (file)
--- a/lib/cache_segtype/cache.c
+++ b/lib/cache_segtype/cache.c
@@ -55,7 +55,8 @@ static void _cache_display(const struct lv_segment *seg)
        log_print("  Mode\t\t%s", get_cache_mode_name(pool_seg));
        log_print("  Policy\t\t%s", pool_seg->policy_name);
 
-       if ((n = pool_seg->policy_settings->child))
+       if (pool_seg->policy_settings &&
+           (n = pool_seg->policy_settings->child))
                dm_config_write_node(n, _cache_out_line, NULL);
 
        log_print(" ");