diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2012-06-21 12:43:31 +0200 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2012-08-23 14:37:52 +0200 |
commit | fd417db274e9a75f33f661b475aeebfd3d9eae45 (patch) | |
tree | 0a141abbc4fca181dd8ebd93859a7723caceee0f /tools | |
parent | 15891d366d34fa6ea7235ec803ee0c5445a17101 (diff) | |
download | lvm2-fd417db274e9a75f33f661b475aeebfd3d9eae45.tar.gz lvm2-fd417db274e9a75f33f661b475aeebfd3d9eae45.tar.xz lvm2-fd417db274e9a75f33f661b475aeebfd3d9eae45.zip |
check: add internal errors for unexpected paths
Adding couple INTERNAL_ERROR reports for unwanted parameters:
Ensure the 'top' metadata node cannot be NULL for lvmetad.
Make obvious vginfo2 cannot be NULL.
Report internal error if handler and vg is undefined.
Check for handle in poll_vg().
Ensure seg is not NULL in dev_manager_transient().
Report missing read_ahead for _lv_read_ahead_single().
Check for report handler in dm_report_object().
Check missing VG in _vgreduce_single().
Diffstat (limited to 'tools')
-rw-r--r-- | tools/polldaemon.c | 5 | ||||
-rw-r--r-- | tools/vgreduce.c | 5 |
2 files changed, 10 insertions, 0 deletions
diff --git a/tools/polldaemon.c b/tools/polldaemon.c index fe55c3b6..f57484ea 100644 --- a/tools/polldaemon.c +++ b/tools/polldaemon.c @@ -261,6 +261,11 @@ static int _poll_vg(struct cmd_context *cmd, const char *vgname, const char *name; int finished; + if (!parms) { + log_error(INTERNAL_ERROR "Handle is undefined."); + return ECMD_FAILED; + } + dm_list_iterate_items(lvl, &vg->lvs) { lv = lvl->lv; if (!(lv->status & parms->lv_type)) diff --git a/tools/vgreduce.c b/tools/vgreduce.c index 96966513..975b9eab 100644 --- a/tools/vgreduce.c +++ b/tools/vgreduce.c @@ -124,6 +124,11 @@ static int _vgreduce_single(struct cmd_context *cmd, struct volume_group *vg, int r = ECMD_FAILED; const char *name = pv_dev_name(pv); + if (!vg) { + log_error(INTERNAL_ERROR "VG is NULL."); + return ECMD_FAILED; + } + if (pv_pe_alloc_count(pv)) { log_error("Physical volume \"%s\" still in use", name); return ECMD_FAILED; |