Fix missing break

Bug introduced with addition of internal error default case.
Seem like this code is not used.
TODO: add coverage test.

diff --git a/WHATS_NEW b/WHATS_NEW
index 3b333995ab9cb90a09a36a6ce080d6a6c1317ce4..d0bd6c939e7146dfd3d1fc7207db74dad34d91c5 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.94 - 
 ====================================
+  Fix missing break in _format_pvsegs (2.02.92).
   Test seg pointer for non-null it in raid_target_percent error path.
   Check for errors in _init_tags() during config loading.
   Always check result of _set_vg_name() in lvcreate.

diff --git a/lib/metadata/lv.c b/lib/metadata/lv.c
index 42dd5a198416aa35cb70eee30f6ff945ba72aee0..747bf9e73bd9529ee72f5ff28772be6f2ed8c947 100644 (file)
--- a/lib/metadata/lv.c
+++ b/lib/metadata/lv.c
@@ -53,6 +53,7 @@ static char *_format_pvsegs(struct dm_pool *mem, const struct lv_segment *seg,
                case AREA_UNASSIGNED:
                        name = "unassigned";
                        extent = 0;
+                       break;
                default:
                        log_error(INTERNAL_ERROR "Unknown area segtype.");
                        return NULL;