]> xenbits.xensource.com Git - qemu-xen.git/commitdiff
qcow2: don't permit changing encryption parameters
authorDaniel P. Berrange <berrange@redhat.com>
Fri, 3 Nov 2017 14:39:02 +0000 (14:39 +0000)
committerKevin Wolf <kwolf@redhat.com>
Fri, 17 Nov 2017 12:35:59 +0000 (13:35 +0100)
Currently if trying to change encryption parameters on a qcow2 image, qemu-img
will abort. We already explicitly check for attempt to change encrypt.format
but missed other parameters like encrypt.key-secret. Rather than list each
parameter, just blacklist changing of all parameters with a 'encrypt.' prefix.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
block/qcow2.c

index b3d66a0e88c675c0ab971d5c6723a8e3672b3d47..92e5d548e399c016f69ffb34b7e934ab7cc590ac 100644 (file)
@@ -4069,6 +4069,9 @@ static int qcow2_amend_options(BlockDriverState *bs, QemuOpts *opts,
                 error_report("Changing the encryption format is not supported");
                 return -ENOTSUP;
             }
+        } else if (g_str_has_prefix(desc->name, "encrypt.")) {
+            error_report("Changing the encryption parameters is not supported");
+            return -ENOTSUP;
         } else if (!strcmp(desc->name, BLOCK_OPT_CLUSTER_SIZE)) {
             cluster_size = qemu_opt_get_size(opts, BLOCK_OPT_CLUSTER_SIZE,
                                              cluster_size);