Gitweb:
http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=24001a08ab9273...
Commit: 24001a08ab92737b32968aeba5f6b2b9c036c480
Parent: 3b5afac9b433dd26f4461d4f59401bbb299219ec
Author: Zdenek Kabelac <zkabelac(a)redhat.com>
AuthorDate: Thu Aug 21 15:38:18 2014 +0200
Committer: Zdenek Kabelac <zkabelac(a)redhat.com>
CommitterDate: Tue Aug 26 14:13:06 2014 +0200
cleanup: check pv_count is not 0
Since we already detect writemostly_ARG is non-zero
make it obvious pv_count will also be non-zero in this case.
---
tools/lvchange.c | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/tools/lvchange.c b/tools/lvchange.c
index 42769c2..0981f2c 100644
--- a/tools/lvchange.c
+++ b/tools/lvchange.c
@@ -740,9 +740,8 @@ static int lvchange_writemostly(struct logical_volume *lv)
if (arg_count(cmd, writebehind_ARG))
raid_seg->writebehind = arg_uint_value(cmd, writebehind_ARG, 0);
- if (arg_count(cmd, writemostly_ARG)) {
+ if ((pv_count = arg_count(cmd, writemostly_ARG))) {
/* writemostly can be specified more than once */
- pv_count = arg_count(cmd, writemostly_ARG);
pv_names = dm_pool_alloc(lv->vg->vgmem, sizeof(char *) * pv_count);
if (!pv_names)
return_0;