[libvirt] [PATCH] storage: minor fix in luks encrypted volume creation
Andrea Bolognani
abologna at redhat.com
Wed Jun 6 15:46:54 UTC 2018
On Wed, 2018-06-06 at 17:30 +0200, Ján Tomko wrote:
[...]
> > + if (!enc->secrets) {
>
> You can use enc->secrets != 1, to match the error message.
>
> But the > 1 cause will be caught anyway later in
> storageBackendCreateQemuImgCheckEncryption which cannot be simply called
> before here because the code is overly complicated.
I'd rather the check be here, where we raise the error message,
instead of relying on the fact that the other function will never
change in a way that will cause it not to check for >1.
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list