diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2012-02-27 11:13:48 +0000 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2012-02-27 11:13:48 +0000 |
| commit | 9737943c4ce27b5cd8483cd64358550b67bdb85c (patch) | |
| tree | a7b1cf11d05b38648d73db98f7ce73bd30a4d470 | |
| parent | d2e33bed9435410f1cf9510f511a69212d2f299e (diff) | |
| download | lvm2-9737943c4ce27b5cd8483cd64358550b67bdb85c.tar.gz lvm2-9737943c4ce27b5cd8483cd64358550b67bdb85c.tar.xz lvm2-9737943c4ce27b5cd8483cd64358550b67bdb85c.zip | |
Fix missing break
Bug introduced with addition of internal error default case.
Seem like this code is not used.
TODO: add coverage test.
| -rw-r--r-- | WHATS_NEW | 1 | ||||
| -rw-r--r-- | lib/metadata/lv.c | 1 |
2 files changed, 2 insertions, 0 deletions
@@ -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 42dd5a19..747bf9e7 100644 --- 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; |