diff options
author | David Teigland <teigland@redhat.com> | 2020-02-11 12:59:39 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2020-02-11 13:01:13 -0600 |
commit | cba06012acc589888ef88221f1a580b5b81b4100 (patch) | |
tree | e4b1db1c56edb8dcd6a9af952561abe79bed2f78 | |
parent | 892a182975049ebe732860c30c866fe2de5b7717 (diff) | |
download | lvm2-cba06012acc589888ef88221f1a580b5b81b4100.tar.gz |
writecache: check if cachevol is writable
before trying to initialize it (since wipe_lv
does not return an error if it fails to write.)
-rw-r--r-- | tools/lvconvert.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tools/lvconvert.c b/tools/lvconvert.c index cd9a3e8f9..aa2dca7f5 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -5351,6 +5351,11 @@ static int _writecache_zero(struct cmd_context *cmd, struct logical_volume *lv) }; int ret; + if (!(lv->status & LVM_WRITE)) { + log_error("Cannot initialize readonly LV %s", display_lvname(lv)); + return 0; + } + if (test_mode()) return 1; |