diff options
| author | Peter Rajnoha <prajnoha@redhat.com> | 2015-07-08 15:08:39 +0200 |
|---|---|---|
| committer | Peter Rajnoha <prajnoha@redhat.com> | 2015-07-08 15:08:39 +0200 |
| commit | 6b48233f2548621c42129b8f0e2911e05c206660 (patch) | |
| tree | 97c1a2090b3bad307e770d169b53731c82177548 | |
| parent | 3ec4813ba22adedb8914089b6f17fe779d76ee77 (diff) | |
| download | lvm2-6b48233f2548621c42129b8f0e2911e05c206660.tar.gz | |
coverity: fix NULL check in lv->lvid.s
tools/polldaemon.c:457: array_null: Comparing an array to null is not useful: "lv->lvid.s"
The lv->lvid.s is never NULL. The check was supposed to be *lv->lvid.s
to check if the string is not empty.
| -rw-r--r-- | tools/polldaemon.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/polldaemon.c b/tools/polldaemon.c index ab9e4747e..bbe46411e 100644 --- a/tools/polldaemon.c +++ b/tools/polldaemon.c @@ -454,7 +454,7 @@ static int _lvmpolld_init_poll_vg(struct cmd_context *cmd, const char *vgname, if (!id.display_name && !lpdp->parms->aborting) continue; - if (!lv->lvid.s) { + if (!*lv->lvid.s) { log_print_unless_silent("Missing LV uuid within: %s/%s", id.vg_name, id.lv_name); continue; } |