From d330e660ee622a2707e1feab05d69a06386f9067 Mon Sep 17 00:00:00 2001
From: Milan Broz <mbroz@redhat.com>
Date: Thu, 15 Mar 2007 14:00:30 +0000
Subject: [PATCH] Fix vgremove to require at least one vg argument.

---
 WHATS_NEW        | 1 +
 tools/vgremove.c | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/WHATS_NEW b/WHATS_NEW
index 61c0d7efc..5ee9523d9 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.24 -
 ====================================
+  Fix vgremove to require at least one vg argument.
   Fix reading of striped LVs in LVM1 format.
   Flag nolocking as clustered so clvmd startup sees clustered LVs. (2.02.10)
   Add a few missing pieces of vgname command line validation.
diff --git a/tools/vgremove.c b/tools/vgremove.c
index f350f1c54..9aca5207b 100644
--- a/tools/vgremove.c
+++ b/tools/vgremove.c
@@ -87,6 +87,11 @@ int vgremove(struct cmd_context *cmd, int argc, char **argv)
 {
 	int ret;
 
+	if (!argc) {
+		log_error("Please enter one or more volume group paths");
+		return EINVALID_CMD_LINE;
+	}
+
 	if (!lock_vol(cmd, ORPHAN, LCK_VG_WRITE)) {
 		log_error("Can't get lock for orphan PVs");
 		return ECMD_FAILED;