shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-30 17:18:21 +03:00
Code Releases Activity

reporter: restore report type to initial value after processing report_for_selection

Browse Source 
If we are executing 'report_for_selection' to do an internal report
just for the selection itself (not for display), we can call it more
than once. In that case, we are reusing the same selection handle
(e.g. inside 'process_each_lv_in_vg').

The selection handle has 'report_type' field which is a union of all
report types needed for the report based on selection fields we actually
use.

The 'report_type' is further clarified based on checks and rules inside
'_get_final_report_type' which 'report_for_selection' calls. Then, this
final report type unambiguously identifies proper branch to take in
'report_all_in_{pv,vg,lv}' that is called next.

If the 'report_for_selection' is called more than once with the same
selection handle, we need to make sure that we always restore the report
type to its initial value, so all the rules inside 'report_for_selection'
are applied correctly next time.

This patch fixes the missing restoration of the 'report_type' value in
'selection_handle' that is reused for recurring 'report_for_selection'
calls.

An example scenario where this failed was with selecting an LV for
removal with "lvremove --select" while using a field in the selection
that required extra DM info or DM status call for the LV (that is,
"Logical Volume Device Info Fields" and "Logical Volume Device Status Fields"
as visible in 'lvs -S help').
This commit is contained in:
Peter Rajnoha 2023-05-30 21:14:37 +02:00
parent 0591131a53
commit e2ecc6c763
No known key found for this signature in database
GPG Key ID: E776664036DF84AB
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
WHATS_NEW
View File

@ -1,5 +1,6 @@
version 2.03.22 -
=================================
Fix failing -S|--select for non-reporting cmds if using LV info/status fields.
version 2.03.21 - 21st April 2023
=================================

7
tools/reporter.c
View File

@ -633,6 +633,7 @@ int report_for_selection(struct cmd_context *cmd,
struct logical_volume *lv)
{
struct selection_handle *sh = parent_handle->selection_handle;
report_type_t initial_report_type = sh->report_type;
struct report_args args = {0};
struct single_report_args *single_args = &args.single_args[REPORT_IDX_SINGLE];
int do_lv_info, do_lv_seg_status;
@ -648,8 +649,10 @@ int report_for_selection(struct cmd_context *cmd,
&sh->report_type))
return_0;
if (!(handle = init_processing_handle(cmd, parent_handle)))
if (!(handle = init_processing_handle(cmd, parent_handle))) {
sh->report_type = initial_report_type;
return_0;
}
/*
* We're already reporting for select so override
@ -695,6 +698,8 @@ int report_for_selection(struct cmd_context *cmd,
break;
}
sh->report_type = initial_report_type;
/*
* Keep the selection handle provided from the caller -
* do not destroy it - the caller will still use it to