diff options
| author | Peter Rajnoha <prajnoha@redhat.com> | 2022-05-05 11:02:32 +0200 |
|---|---|---|
| committer | Peter Rajnoha <prajnoha@redhat.com> | 2022-05-05 11:13:39 +0200 |
| commit | 7ec0726ce331a4dc1ab5dc5e1b9ae503e5d758da (patch) | |
| tree | bfd05d8f7f84b05803fd28fe6013165aa5fe1d18 /tools/toollib.c | |
| parent | ff6022d400430df2f51160ce9eec4010ac55696c (diff) | |
| download | lvm2-7ec0726ce331a4dc1ab5dc5e1b9ae503e5d758da.tar.gz | |
toollib: fix segfault when handling selection with historical LVs
When processing historical LVs inside process_each_lv_in_vg for
selection, we need to use dummy "_historical_lv" for select_match_lv.
This is because a historical LV is not an actual LV, but only a tiny
representation with subset of original properties that we recorded
(name, uuid...).
To use the same processing functions we use for full-fledged non-historical
LVs, we need to use the prefilled "_historical_lv" structure which has all
the other missing properties hard-coded.
Diffstat (limited to 'tools/toollib.c')
| -rw-r--r-- | tools/toollib.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/toollib.c b/tools/toollib.c index 545407c2f..d77092d89 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -3447,13 +3447,14 @@ int process_each_lv_in_vg(struct cmd_context *cmd, struct volume_group *vg, process_lv = 1; } - process_lv = process_lv && select_match_lv(cmd, handle, vg, lvl->lv) && _select_matches(handle); + _historical_lv.this_glv = glvl->glv; + _historical_lv.name = glvl->glv->historical->name; + + process_lv = process_lv && select_match_lv(cmd, handle, vg, &_historical_lv) && _select_matches(handle); if (!process_lv) continue; - _historical_lv.this_glv = glvl->glv; - _historical_lv.name = glvl->glv->historical->name; log_very_verbose("Processing historical LV %s in VG %s.", glvl->glv->historical->name, vg->name); ret = process_single_lv(cmd, &_historical_lv, handle); |