diff options
author | Jonathan Earl Brassow <jbrassow@redhat.com> | 2011-12-01 00:04:21 +0000 |
---|---|---|
committer | Jonathan Earl Brassow <jbrassow@redhat.com> | 2011-12-01 00:04:21 +0000 |
commit | 73d6bf290e3674d267d308f205fd898bc92a4f21 (patch) | |
tree | 0db76d1e3a0ceaa7caad384ffaf999f284a00471 /tools/lvchange.c | |
parent | 8de2ef4f3bebc75e580bbcb86121e0b5025ed935 (diff) | |
download | lvm2-73d6bf290e3674d267d308f205fd898bc92a4f21.tar.gz lvm2-73d6bf290e3674d267d308f205fd898bc92a4f21.tar.xz lvm2-73d6bf290e3674d267d308f205fd898bc92a4f21.zip |
Do not allow users to change permissions on RAID sub-LVs.
Diffstat (limited to 'tools/lvchange.c')
-rw-r--r-- | tools/lvchange.c | 8 |
1 files changed, 8 insertions, 0 deletions
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", |