diff options
author | Heinz Mauelshagen <heinzm@redhat.com> | 2016-08-16 23:57:09 +0200 |
---|---|---|
committer | Heinz Mauelshagen <heinzm@redhat.com> | 2016-08-16 23:57:09 +0200 |
commit | c0a0eedf2e572b99e40384e68e3d5f8fba6f0ae5 (patch) | |
tree | ad3e2f7eaac2eee547a112c062a457664e752f5b | |
parent | 73df2aedf9788dcf2dbf09f20f8783c6d2108e75 (diff) | |
download | lvm2-c0a0eedf2e572b99e40384e68e3d5f8fba6f0ae5.tar.gz |
vgsplit: fix regression processing thin external origins
a579ba2ac27d fixed a regression causing a segfault if no external
origin existed but broke the logic leading to erroneous error
messages and creations of split off exported VGs in case the
external origin and the pool LVs were allocated on different PVs.
- resolves rhbz1367459
-rw-r--r-- | tools/vgsplit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/vgsplit.c b/tools/vgsplit.c index 93161c17d..52f603f9a 100644 --- a/tools/vgsplit.c +++ b/tools/vgsplit.c @@ -331,7 +331,7 @@ static int _move_thins(struct volume_group *vg_from, data_lv = seg_lv(first_seg(seg->pool_lv), 0); /* Ignore, if no allocations on PVs of @vg_to */ - if (!lv_is_on_pvs(data_lv, &vg_to->pvs) || + if (!lv_is_on_pvs(data_lv, &vg_to->pvs) && (seg->external_lv && !lv_is_on_pvs(seg->external_lv, &vg_to->pvs))) continue; |