shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-03-10 16:58:47 +03:00
Code Releases Activity

cov: ensure there is either vg or error_vg

Browse Source 
Make it explicitely visible to coverity that
_process_pvs_in_vg() is not passing  NULL vg.
This commit is contained in:
Zdenek Kabelac 2023-04-26 13:38:22 +02:00
parent 871d9f379f
commit fae0ed8f10
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tools/toollib.c
View File

@ -4525,7 +4525,8 @@ static int _process_pvs_in_vgs(struct cmd_context *cmd, uint32_t read_flags,
error_flags = 0; error_flags = 0;
vg = vg_read(cmd, vg_name, vg_uuid, read_flags, lockd_state, &error_flags, &error_vg); vg = vg_read(cmd, vg_name, vg_uuid, read_flags, lockd_state, &error_flags, &error_vg);
if (_ignore_vg(cmd, error_flags, error_vg, vg_name, NULL, read_flags, &skip, &notfound)) { if (_ignore_vg(cmd, error_flags, error_vg, vg_name, NULL, read_flags, &skip, &notfound) ||
(!vg && !error_vg)) {
stack; stack;
ret_max = ECMD_FAILED; ret_max = ECMD_FAILED;
report_log_ret_code(ret_max); report_log_ret_code(ret_max);
@ -4535,7 +4536,7 @@ static int _process_pvs_in_vgs(struct cmd_context *cmd, uint32_t read_flags,
} }
if (notfound) if (notfound)
goto endvg; goto endvg;
/* /*
* Don't call "continue" when skip is set, because we need to remove * Don't call "continue" when skip is set, because we need to remove
* error_vg->pvs entries from devices list. * error_vg->pvs entries from devices list.