From 73d6bf290e3674d267d308f205fd898bc92a4f21 Mon Sep 17 00:00:00 2001
From: Jonathan Earl Brassow <jbrassow@redhat.com>
Date: Thu, 1 Dec 2011 00:04:21 +0000
Subject: Do not allow users to change permissions on RAID sub-LVs.

---
 tools/lvchange.c | 8 ++++++++
 1 file changed, 8 insertions(+)

(limited to 'tools/lvchange.c')

diff --git a/tools/lvchange.c b/tools/lvchange.c
index 2a99c455..ac761779 100644
--- a/tools/lvchange.c
+++ b/tools/lvchange.c
@@ -43,6 +43,14 @@ static int lvchange_permission(struct cmd_context *cmd,
 		return 0;
 	}
 
+	/* Not allowed to change permissions on RAID sub-LVs directly */
+	if ((lv->status & RAID_META) || (lv->status & RAID_IMAGE)) {
+		log_error("Cannot change permissions of RAID %s \"%s\"",
+			  (lv->status & RAID_IMAGE) ? "image" :
+			  "metadata area", lv->name);
+		return 0;
+	}
+
 	if (lv_access & LVM_WRITE) {
 		lv->status |= LVM_WRITE;
 		log_verbose("Setting logical volume \"%s\" read/write",
-- 
cgit