I realize that qcow[2] encryption is a feature we have deprecated and will remove support for running it with the QEMU system emulators in this cycle. We do still need to make sure it continues to work for the sake of letting people run qemu-img convert to retrieve their data though.
Some of the other patches I'm working on which introduce a cypto cipher API touch this qcow2 code, thus I wanted to be able to test that it doesn't break anything. I found that qemu-iotests didn't have any coverage of the qcow2 encryption code. For added fun, I then discovered that qemu-io doesn't check if an encryption key is required, so ends up writing plain text to the files instead of cipher, and returning cipher text for reads, instead of plain text. IOW qemu-io will corrupt encrypted qcow2 files on write. This series adds some asserts that will protect against this kind of mistake, adds support for getting passwords to qemu-io (in the same manner that qemu-img supports), and finally adds a test case for reading/writing encrypted qcow2. Daniel P. Berrange (5): qcow2/qcow: protect against uninitialized encryption key util: move read_password method out of qemu-img into osdep/oslib util: allow \n to terminate password input qemu-io: prompt for encryption keys when required tests: add test case for encrypted qcow2 read/write block/qcow.c | 10 +++-- block/qcow2-cluster.c | 3 +- block/qcow2.c | 18 ++++++--- include/qemu/osdep.h | 2 + qemu-img.c | 93 +--------------------------------------------- qemu-io.c | 21 +++++++++++ tests/qemu-iotests/131 | 69 ++++++++++++++++++++++++++++++++++ tests/qemu-iotests/131.out | 46 +++++++++++++++++++++++ tests/qemu-iotests/group | 1 + util/oslib-posix.c | 67 +++++++++++++++++++++++++++++++++ util/oslib-win32.c | 24 ++++++++++++ 11 files changed, 252 insertions(+), 102 deletions(-) create mode 100755 tests/qemu-iotests/131 create mode 100644 tests/qemu-iotests/131.out -- 2.1.0