From: Jonathan Earl Brassow <jbrassow@redhat.com>
Date: Thu, 1 Dec 2011 00:04:21 +0000 (+0000)
Subject: Do not allow users to change permissions on RAID sub-LVs.
X-Git-Tag: v2_02_91~226
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=73d6bf290e3674d267d308f205fd898bc92a4f21;p=lvm2.git

Do not allow users to change permissions on RAID sub-LVs.
---

diff --git a/tools/lvchange.c b/tools/lvchange.c
index 2a99c455b..ac7617793 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",