diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2015-11-17 13:17:59 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2015-11-17 19:01:25 +0100 |
| commit | cad3568def7f82ae58c3585a94ceb377bb155167 (patch) | |
| tree | fd688eb3f397b85aa95c37f76393e13eaf6e81b1 | |
| parent | 51dfba002b05cc65346b539b4b030f0c2dfffeaf (diff) | |
| download | lvm2-cad3568def7f82ae58c3585a94ceb377bb155167.tar.gz | |
raid: drop unneeded NULL test
Skip testing target_pvs for NULL, we already
dereference it in many other places.
If check would ever be needed - it needs to be
in front of _raid_extract_images().
| -rw-r--r-- | lib/metadata/raid_manip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/metadata/raid_manip.c b/lib/metadata/raid_manip.c index 4b2cb610e..b99b6713b 100644 --- a/lib/metadata/raid_manip.c +++ b/lib/metadata/raid_manip.c @@ -913,7 +913,7 @@ static int _raid_extract_images(struct logical_volume *lv, uint32_t new_count, display_lvname(seg_lv(seg, s))); } else { /* Conditions for second pass */ - if (!target_pvs || !lv_is_on_pvs(seg_lv(seg, s), target_pvs) || + if (!lv_is_on_pvs(seg_lv(seg, s), target_pvs) || !lv_is_on_pvs(seg_metalv(seg, s), target_pvs)) continue; |